Re: [dpdk-dev] [PATCH v2] kernel/linux: fix kernel dir for meson
On 12/03, Bruce Richardson wrote: >On Tue, Dec 03, 2019 at 08:33:22PM +0800, Ye Xiaolong wrote: >> On 12/03, Bruce Richardson wrote: >> >On Tue, Dec 03, 2019 at 01:29:17PM +0800, Xiaolong Ye wrote: >> >> kernel_dir option in meson build is equivalent to RTE_KERNELDIR in make >> >> system, for cross-compilation case, users would specify it as local >> >> kernel src dir like >> >> >> >> //target-arm_glibc/linux-arm/linux-4.19.81/ >> >> >> >> Current meson build would fail to compile kernel module if user specify >> >> kernel_dir as above, this patch fixes this issue. >> >> >> >> Fixes: 317832f97c16 ("kernel/linux: fix modules install path") >> >> Cc: sta...@dpdk.org >> >> Cc: iryz...@nfware.com >> >> >> >> Signed-off-by: Xiaolong Ye >> >> --- >> >> >> >> V2 changes: >> >> >> >> 1. handle both normal and cross-compilation cases >> >> >> >We need to handle both, but they need to be handled without breaking the >> >currently working case where we pass in /lib/modules/$(uname -r)/ as the >> >kerneldir path. >> >> So you mean we should allow user to specify both /lib/modules/$(uname -r) and >> /lib/modules/$(uname -r)/build as kernel_dir for normal case? >> >That is up to you, but we need to still allow the former case so as to >avoid breaking backward compatibility for existing build setups. Therefore >I suggest supporting both is recommended. Make sense, I'll try a new version. Thanks, Xiaolong > >/Bruce
Re: [dpdk-dev] [PATCH v2] kernel/linux: fix kernel dir for meson
On Tue, Dec 03, 2019 at 08:33:22PM +0800, Ye Xiaolong wrote: > On 12/03, Bruce Richardson wrote: > >On Tue, Dec 03, 2019 at 01:29:17PM +0800, Xiaolong Ye wrote: > >> kernel_dir option in meson build is equivalent to RTE_KERNELDIR in make > >> system, for cross-compilation case, users would specify it as local > >> kernel src dir like > >> > >> //target-arm_glibc/linux-arm/linux-4.19.81/ > >> > >> Current meson build would fail to compile kernel module if user specify > >> kernel_dir as above, this patch fixes this issue. > >> > >> Fixes: 317832f97c16 ("kernel/linux: fix modules install path") > >> Cc: sta...@dpdk.org > >> Cc: iryz...@nfware.com > >> > >> Signed-off-by: Xiaolong Ye > >> --- > >> > >> V2 changes: > >> > >> 1. handle both normal and cross-compilation cases > >> > >We need to handle both, but they need to be handled without breaking the > >currently working case where we pass in /lib/modules/$(uname -r)/ as the > >kerneldir path. > > So you mean we should allow user to specify both /lib/modules/$(uname -r) and > /lib/modules/$(uname -r)/build as kernel_dir for normal case? > That is up to you, but we need to still allow the former case so as to avoid breaking backward compatibility for existing build setups. Therefore I suggest supporting both is recommended. /Bruce
Re: [dpdk-dev] [PATCH v2] kernel/linux: fix kernel dir for meson
On 12/03, Bruce Richardson wrote: >On Tue, Dec 03, 2019 at 01:29:17PM +0800, Xiaolong Ye wrote: >> kernel_dir option in meson build is equivalent to RTE_KERNELDIR in make >> system, for cross-compilation case, users would specify it as local >> kernel src dir like >> >> //target-arm_glibc/linux-arm/linux-4.19.81/ >> >> Current meson build would fail to compile kernel module if user specify >> kernel_dir as above, this patch fixes this issue. >> >> Fixes: 317832f97c16 ("kernel/linux: fix modules install path") >> Cc: sta...@dpdk.org >> Cc: iryz...@nfware.com >> >> Signed-off-by: Xiaolong Ye >> --- >> >> V2 changes: >> >> 1. handle both normal and cross-compilation cases >> >We need to handle both, but they need to be handled without breaking the >currently working case where we pass in /lib/modules/$(uname -r)/ as the >kerneldir path. So you mean we should allow user to specify both /lib/modules/$(uname -r) and /lib/modules/$(uname -r)/build as kernel_dir for normal case? Thanks, Xiaolong > >/Bruce
Re: [dpdk-dev] [PATCH v2] kernel/linux: fix kernel dir for meson
On Tue, Dec 03, 2019 at 01:29:17PM +0800, Xiaolong Ye wrote: > kernel_dir option in meson build is equivalent to RTE_KERNELDIR in make > system, for cross-compilation case, users would specify it as local > kernel src dir like > > //target-arm_glibc/linux-arm/linux-4.19.81/ > > Current meson build would fail to compile kernel module if user specify > kernel_dir as above, this patch fixes this issue. > > Fixes: 317832f97c16 ("kernel/linux: fix modules install path") > Cc: sta...@dpdk.org > Cc: iryz...@nfware.com > > Signed-off-by: Xiaolong Ye > --- > > V2 changes: > > 1. handle both normal and cross-compilation cases > We need to handle both, but they need to be handled without breaking the currently working case where we pass in /lib/modules/$(uname -r)/ as the kerneldir path. /Bruce