On Fri, Nov 19, 2021 at 1:34 PM Pingfan Liu <piliu(a)redhat.com> wrote:
On Fri, Nov 19, 2021 at 11:22:57AM +0800, Coiby Xu wrote:
> The crashkernel=auto implementation in kernel space has been rejected
> upstream [1]. The current user space implementation [2] [3] ships a
> crashkernel.default but hasn't supported the swiotlb memory requirement,
> custom crashkernel value from user and fadump.
>
> The crashkernel.default implementation seems to be overly
> complex,
> - the default crashkernel value rarely changes. This is no need to ship
> the same crashkernel.default default for every kernel package of a
> architecture;
> - when deciding the value of crashkernel for a new kernel, the
> crashkernel.default of existing kernel is took into consideration
>
> We can simply let the kexec-tools maintain the default crashkernel
> values and provide an API for kdump-anacon-addon to query it. And for
> a newly installed kernel, we can simply call "kdumpctl reset-crashkernel
> KERNELPATH" to set its crashkernel value.
>
> For the unfulfilled requirements,
> - crashkernel is introduced to /etc/kdump.conf for the
user can set
> custom crashkernel value to tell kexec-tools to
manage crashkernel
> value automatically.
> - "kdumpctl reset-crashkernel" has been
written for the above
> purpose.
> - "kdumpctl fadump on/off" is added for
supporting fadump.
>
Overall, this series make sense to me. My major concern is about [3/11].
^^^ [2/11]. Sorry about the
mistake
Thanks,
Pingfan
>
> [1]
https://lore.kernel.org/linux-mm/20210507010432.IN24PudKT%25akpm@linux-fo...
> [2]
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1171
> [3]
https://lists.fedoraproject.org/archives/list/kexec@lists.fedoraproject.o...
>
>
> Coiby Xu (11):
> update default crashkernel value
> factor out kdump_get_arch_recommend_crashkernel
> provide get_default_crashkernel for kdump_anaconda_addon
> introduce crashkernel option to kdump.conf
> add a helper function to write a config value to kdump.conf
> add a helper function to read kernel cmdline parameter from grubby
> --info
> rewrite reset_crashkernel to provide more features for the user and to
> be called by kernel installation hook
> allow to add extra memory to crashkernel string
> Reserve extra memory when SME or SEV is active
> provide kdumpctl fadump on/off
> use "kdumpctl reset-crashkernel KERNELIMAGE" in kernel installation
> hook
>
> 92-crashkernel.install | 135 +----------------------------
> kdump-lib-initramfs.sh | 9 ++
> kdump-lib.sh | 95 +++++++++++++++------
> kdump.conf | 6 ++
> kdump.conf.5 | 7 ++
> kdumpctl | 188 ++++++++++++++++++++++++++++++++++++-----
> kdumpctl.8 | 16 ++--
> 7 files changed, 271 insertions(+), 185 deletions(-)
>
> --
> 2.31.1
>
_______________________________________________
kexec mailing list -- kexec(a)lists.fedoraproject.org
To unsubscribe send an email to kexec-leave(a)lists.fedoraproject.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines:
https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/kexec@lists.fedoraproject.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure