On 12/20/23 at 10:05am, Coiby Xu wrote:
On Tue, Dec 19, 2023 at 04:14:48PM +0800, Baoquan He wrote:
> On 12/19/23 at 01:31pm, Coiby Xu wrote:
> > Resolves:
https://issues.redhat.com/browse/RHEL-17451
> >
> > Explain what factors affect the default crashkernel value and ask users
> > to reset it manually if needed.
> >
> > Cc: Baoquan He <bhe(a)redhat.com>
> > Cc: Jie Li <jieli(a)redhat.com>
> > Signed-off-by: Coiby Xu <coxu(a)redhat.com>
> > ---
> > kdump.conf.5 | 15 ++++++++++++++-
> > 1 file changed, 14 insertions(+), 1 deletion(-)
> >
> > diff --git a/kdump.conf.5 b/kdump.conf.5
> > index 9117aa7b..0e6ff211 100644
> > --- a/kdump.conf.5
> > +++ b/kdump.conf.5
> > @@ -29,7 +29,20 @@ understand how this configuration file affects the behavior
of kdump.
> > .B auto_reset_crashkernel <yes|no>
> > .RS
> > determine whether to reset kernel crashkernel parameter to the default value
> > -or not when kexec-tools is updated or a new kernel is installed.
> > +or not when kexec-tools is updated or a new kernel is installed. The default
> > +crashkernel values are different for different architectures and also take
the
> ~~~ s/for/on/
I'm curious to ask why "on" is a better choice considering there are more
occurrences of "for different architectures" than "on ..."
in linux/Documentation,
Possibly personal preference, not strong opinion. Please take the one
you prefer.
grep -rI -B1 "different archi" ./ --include=*.rst
./process/5.Posting.rst- combinations of configuration options, use cross-compilers
to build for
./process/5.Posting.rst: different architectures, etc.
--
./process/howto.rst- - "Here is a patch that explains what I am trying to
describe."
./process/howto.rst: - "I tested it on 5 different architectures..."
--
./arch/arm/setup.rst- MEMC chip control register for Acorn Archimedes and Acorn
A5000
./arch/arm/setup.rst: based machines. May be used differently by different
architectures.
--
./arch/arm/setup.rst- Default sound setting on Acorn machines. May be used
differently by
./arch/arm/setup.rst: different architectures.
--
./trace/kprobes.rst-
./trace/kprobes.rst:Here are sample overhead figures (in usec) for different
architectures::
--
./userspace-api/unshare.rst-
./userspace-api/unshare.rst: d) Registration of system call number for different
architectures
> Other than this minor concern, this looks good to me, thx.
>
> Acked-by: Baoquan He <bhe(a)redhat.com>
Thanks for reviewing and ack'ing the patch!
You are welcome.