Hi Guilherme,
On 11/22/20 at 12:32pm, Guilherme Piccoli wrote:
> Hi Dave and Kairui, thanks for your responses! OK, if that makes sense
> to you I'm fine with it. I'd just recommend to test recent kernels in
> multiple distros with the minimum "range" to see if 64M is enough for
> crashkernel, maybe we'd need to bump that.
Giving the different kernel configs and the different userspace
initramfs setup it is hard to get an uniform value for all distributions,
but we can have an interface/kconfig-option for them to provide a value like this patch
is doing. And it could be improved like Kairui said about some known
kernel added extra values later, probably some more improvements if
doable.
Thanks
Dave
On 11/22/20 9:47 PM, Dave Young wrote:
> Hi Guilherme,
> On 11/22/20 at 12:32pm, Guilherme Piccoli wrote:
>> Hi Dave and Kairui, thanks for your responses! OK, if that makes sense
>> to you I'm fine with it. I'd just recommend to test recent kernels in
>> multiple distros with the minimum "range" to see if 64M is enough for
>> crashkernel, maybe we'd need to bump that.
>
> Giving the different kernel configs and the different userspace
> initramfs setup it is hard to get an uniform value for all distributions,
> but we can have an interface/kconfig-option for them to provide a value like this patch
> is doing. And it could be improved like Kairui said about some known
> kernel added extra values later, probably some more improvements if
> doable.
>
> Thanks
> Dave
>
Hi.
Are we going to move forward with implementing this for X86 and Arm ?
If other platform maintainers want to include this CONFIG option in
their configuration settings they have a starting point.
Thank you,
John.
( I am not currently on many of the included dist lists in this email,
so hopefully key contributors are included in this exchange )
Hi John,
On 01/21/21 at 09:32am, [email protected] wrote:
> On 11/22/20 9:47 PM, Dave Young wrote:
> > Hi Guilherme,
> > On 11/22/20 at 12:32pm, Guilherme Piccoli wrote:
> > > Hi Dave and Kairui, thanks for your responses! OK, if that makes sense
> > > to you I'm fine with it. I'd just recommend to test recent kernels in
> > > multiple distros with the minimum "range" to see if 64M is enough for
> > > crashkernel, maybe we'd need to bump that.
> >
> > Giving the different kernel configs and the different userspace
> > initramfs setup it is hard to get an uniform value for all distributions,
> > but we can have an interface/kconfig-option for them to provide a value like this patch
> > is doing. And it could be improved like Kairui said about some known
> > kernel added extra values later, probably some more improvements if
> > doable.
> >
> > Thanks
> > Dave
> >
>
> Hi.
>
> Are we going to move forward with implementing this for X86 and Arm ?
>
> If other platform maintainers want to include this CONFIG option in their
> configuration settings they have a starting point.
I would expect this become arch independent.
Saeed, Kairui, would any of you like to update the patch?
>
> Thank you,
>
> John.
>
> ( I am not currently on many of the included dist lists in this email, so
> hopefully key contributors are included in this exchange )
>
Thanks
Dave
On 01/22/21 at 09:22am, Dave Young wrote:
> Hi John,
>
> On 01/21/21 at 09:32am, [email protected] wrote:
> > On 11/22/20 9:47 PM, Dave Young wrote:
> > > Hi Guilherme,
> > > On 11/22/20 at 12:32pm, Guilherme Piccoli wrote:
> > > > Hi Dave and Kairui, thanks for your responses! OK, if that makes sense
> > > > to you I'm fine with it. I'd just recommend to test recent kernels in
> > > > multiple distros with the minimum "range" to see if 64M is enough for
> > > > crashkernel, maybe we'd need to bump that.
> > >
> > > Giving the different kernel configs and the different userspace
> > > initramfs setup it is hard to get an uniform value for all distributions,
> > > but we can have an interface/kconfig-option for them to provide a value like this patch
> > > is doing. And it could be improved like Kairui said about some known
> > > kernel added extra values later, probably some more improvements if
> > > doable.
> > >
> > > Thanks
> > > Dave
> > >
> >
> > Hi.
> >
> > Are we going to move forward with implementing this for X86 and Arm ?
> >
> > If other platform maintainers want to include this CONFIG option in their
> > configuration settings they have a starting point.
>
> I would expect this become arch independent.
Clarify a bit, it can be a general config option under arch/Kconfig and
just put the code in general arch independent part.
>
> Saeed, Kairui, would any of you like to update the patch?
>
> >
> > Thank you,
> >
> > John.
> >
> > ( I am not currently on many of the included dist lists in this email, so
> > hopefully key contributors are included in this exchange )
> >
>
> Thanks
> Dave