Hello, On Wed, 12 Jul 2017 00:00:57 +0530 Hari Bathini <hbath...@linux.vnet.ibm.com> wrote:
> Hi Michal, > > > Thanks for the review.. > > > On Monday 26 June 2017 05:45 PM, Michal Suchánek wrote: > > Hello, > > > > On Tue, 20 Jun 2017 21:14:08 +0530 > > Hari Bathini <hbath...@linux.vnet.ibm.com> wrote: > > > I would prefer documenting over a complex implementation. Actually, I > am considering a simple approach of replacing every occurrence of > "fadump_extra_args=" with "fadump_extra_args " in fadump capture > kernel. The cmdline > > "root=/dev/sda2 ro fadump_extra_args="a b c" crashkernel=512M > fadump_extra_args=d" > > becomes > > "root=/dev/sda2 ro fadump_extra_args "a b c" crashkernel=512M > fadump_extra_args d" which is totally broken > > in fadump capture kernel. This must take care of the pitfalls with > the current approach and also, > doesn't rely on parse_args() which was not designed for this scenario > to start with..? It was designed for parsing arguments. To handle replacing arguments you have to extend it. You need to get more information from it for this case. Best regards Michal
