Re: [dpdk-dev] [PATCH v5] build: kni cross-compilation support

On Tue, Feb 09, 2021 at 12:07:13PM +0000, Juraj Linkeš wrote:

quoted

-----Original Message-----
From: Bruce Richardson <redacted>
Sent: Tuesday, February 9, 2021 12:51 PM
To: Juraj Linkeš <redacted>
Cc: thomas@monjalon.net; Ruifeng.Wang@arm.com;
Honnappa.Nagarahalli@arm.com; jerinjacobk@gmail.com;
hemant.agrawal@nxp.com; ferruh.yigit@intel.com; aboyer@pensando.io;
dev@dpdk.org
Subject: Re: [PATCH v5] build: kni cross-compilation support

<snip>

quoted

quoted

+	description: 'Path to the kernel for building kernel modules.
+Headers must be in $kernel_dir. Modules will be installed in
+/lib/modules/$(uname -r)/extra/dpdk for native builds.')

While correct, the last sentence isn't really relevant to the build option, it's more
an additional note, and it seems a bit awkward. Maybe it can be shortened by
referencing "/lib/modules" rather than giving the full path.
Alternatively, maybe just putting the details in the documentation is a better
plan, and leaving the comment here as just "path to the kernel for building
kernel modules".

Having it in docs would be better, but where exactly? I tried looking, but didn't find a suitable place.

It probably should go in the GSG doc, but I agree that there is nowhere
obvious for it. I think we could do with the addition of some descriptions
of the various options in the GSG, but that is out of the scope of this
patch.