Re: [PATCH v5 2/2] powerpc/fadump: update documentation about 'fadump_append=' parameter

[msuchanek <msuchanek@suse.de>]

Hello,

On Wed, 12 Jul 2017 00:00:57 +0530
Hari Bathini <hbathini@linux.vnet.ibm.com> wrote:

> Hi Michal,
>=20
>=20
> Thanks for the review..
>=20
>=20
> On Monday 26 June 2017 05:45 PM, Michal Such=C3=A1nek wrote:
> > Hello,
> >
> > On Tue, 20 Jun 2017 21:14:08 +0530
> > Hari Bathini <hbathini@linux.vnet.ibm.com> wrote:
> > =20

> I would prefer documenting over a complex implementation. Actually, I
> am considering a simple approach of replacing every occurrence of
> "fadump_extra_args=3D" with "fadump_extra_args " in fadump capture
> kernel. The cmdline
>=20
>    "root=3D/dev/sda2 ro fadump_extra_args=3D"a b c" crashkernel=3D512M=20
> fadump_extra_args=3Dd"
>=20
> becomes
>=20
>    "root=3D/dev/sda2 ro fadump_extra_args "a b c" crashkernel=3D512M=20
> fadump_extra_args d"

which is totally broken

>=20
> 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