Re: [PATCH v4 5/5] RISC-V: Add crash kernel support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Στις 2021-06-15 22:21, Rob Herring έγραψε:
On Tue, Jun 15, 2021 at 12:48 PM Geert Uytterhoeven
<geert@xxxxxxxxxxxxxx> wrote:

Hi Nick,

On Tue, Jun 15, 2021 at 8:29 PM Nick Kossifidis <mick@xxxxxxxxxxxx> wrote:
> Στις 2021-06-15 16:19, Geert Uytterhoeven έγραψε:
> > This does not match
> > https://github.com/devicetree-org/dt-schema/blob/master/schemas/chosen.yaml#L77:
> >
> >     $ref: types.yaml#/definitions/uint64-array
> >     maxItems: 2
> >     description:
> >       This property (currently used only on arm64) holds the memory
> > range,
> >       the address and the size, of the elf core header which mainly
> > describes
> >       the panicked kernel\'s memory layout as PT_LOAD segments of elf
> > format.
> >
> > Hence "linux,elfcorehdr" should be a property of the /chosen node,
> > instead of a memory node with a compatible value of "linux,elfcorehdr".
> >
>
> That's a binding for a property on the /chosen node, that as the text
> says it's defined for arm64 only and the code that handled it was also

That doesn't mean it must not be used on other architectures ;-)
Arm64 was just the first one to use it...

It is used on arm64 because memory is often passed by UEFI tables and
not with /memory node. As riscv is also supporting EFI, I'd think they
would do the same.


We've had this discussion before, riscv uses /memory for now and even if we switched to getting memory from ACPI/UEFI tables, the elf core header is passed from the crashed kernel to the kdump kernel, it has nothing to do with UEFI since the bootloader is the kernel itself. Am I missing something ?

Regards,
Nick



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux