[PATCH v4] powerpc: 64K page support for kexec
miltonm at bga.com
Sun Apr 29 15:35:33 EST 2007
On Apr 27, 2007, at 1:23 PM, Haren Myneni wrote:
> Luke Browning wrote:
>> On Fri, 2007-04-27 at 11:59 -0500, Milton Miller wrote:
>>> (panic is a function, so you are checking that the staticly linked
>>> non-weak function is available. If you want to check on when
>>> if its a panic kdump or not, you need to decode the flag and pass
>>> it to kexec_sequence, pass it back here, and update all platforms
>>> for the new parameter).
>>> But I still say the check while clearing the table is too late,
>>> it should be a debug scan before clearing any mappings.
>> How about the following as an alternative.
>> BUG_ON((hpte_v & 0x4000000000000000UL) && (crashing_cpus == -1));
>> BUG_ON((size == MMU_PAGE_16G) && (crashing_cpus == -1));
>> BUG_ON((size == MMU_PAGE_64K_AP) && (crashing_cpus == -1));
> should be crashing_cpu - contains the panic cpu ID
and that only exists (or is even declared) when CONFIG_KEXEC,
whereas this code is based on MMU.
>> I don't have time to work on a multi-platform solution.
We could move the comment to fix this next to the MMU definitions,
and drop the BUG. For that matter, we probably don't need to
define or save slots for the sizes we don't support. In other
words, tie adding support to including support for this function.
More information about the Linuxppc-dev