Dmitry Kozlyuk (2): eal/windows: do not install virt2phys header buildtools: enable chkincs on Windows buildtools/chkincs/meson.build | 5 ----- lib/eal/windows/include/meson.build | 1 - 2 files changed, 6 deletions(-) -- 2.29.3
The header was not intended to be a public one. DPDK users should use `rte_mem_virt2iova()` to translate addresses. Other virt2phys users should use the header from the driver instead. Fixes: 2a5d547a4a9b ("eal/windows: implement basic memory management") Cc: stable@dpdk.org Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com> --- lib/eal/windows/include/meson.build | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/eal/windows/include/meson.build b/lib/eal/windows/include/meson.build index b3534b025f..5fb1962ac7 100644 --- a/lib/eal/windows/include/meson.build +++ b/lib/eal/windows/include/meson.build @@ -5,6 +5,5 @@ includes += include_directories('.') headers += files( 'rte_os.h', - 'rte_virt2phys.h', 'rte_windows.h', ) -- 2.29.3
Independent header compilation test (chkincs) was disabled on Windows. The comment stated that the shebang line in the generator script was not working. Meson 0.57.0, currently recommended for Windows, successfully parses that line and invokes the script. Remove the OS restriction as its reason no longer applies. Fixes: 05050ac4ce99 ("build: add header includes check") Cc: bruce.richardson@intel.com Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com> --- buildtools/chkincs/meson.build | 5 ----- 1 file changed, 5 deletions(-) diff --git a/buildtools/chkincs/meson.build b/buildtools/chkincs/meson.build index 34dcd81ce1..5ffca89761 100644 --- a/buildtools/chkincs/meson.build +++ b/buildtools/chkincs/meson.build @@ -6,11 +6,6 @@ if not get_option('check_includes') subdir_done() endif -if is_windows - # for windows, the shebang line in the script won't work. - error('option "check_includes" is not supported on windows') -endif - gen_c_file_for_header = find_program('gen_c_file_for_header.py') gen_c_files = generator(gen_c_file_for_header, output: '@BASENAME@.c', -- 2.29.3
On Fri, Oct 01, 2021 at 07:37:26PM +0300, Dmitry Kozlyuk wrote:
> Independent header compilation test (chkincs) was disabled on Windows.
> The comment stated that the shebang line in the generator script was not
> working. Meson 0.57.0, currently recommended for Windows, successfully
> parses that line and invokes the script. Remove the OS restriction
> as its reason no longer applies.
>
> Fixes: 05050ac4ce99 ("build: add header includes check")
> Cc: bruce.richardson@intel.com
>
> Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
01/10/2021 18:37, Dmitry Kozlyuk:
> The header was not intended to be a public one.
> DPDK users should use `rte_mem_virt2iova()` to translate addresses.
> Other virt2phys users should use the header from the driver instead.
>
> Fixes: 2a5d547a4a9b ("eal/windows: implement basic memory management")
> Cc: stable@dpdk.org
>
> Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
> ---
> headers += files(
> 'rte_os.h',
> - 'rte_virt2phys.h',
> 'rte_windows.h',
> )
We should plan to rename it.
Files starting with rte_ are supposed to be public.
01/10/2021 18:37, Dmitry Kozlyuk:
> Dmitry Kozlyuk (2):
> eal/windows: do not install virt2phys header
> buildtools: enable chkincs on Windows
Applied, thanks.
2021-10-11 21:07 (UTC+0200), Thomas Monjalon:
> 01/10/2021 18:37, Dmitry Kozlyuk:
> > The header was not intended to be a public one.
> > DPDK users should use `rte_mem_virt2iova()` to translate addresses.
> > Other virt2phys users should use the header from the driver instead.
> >
> > Fixes: 2a5d547a4a9b ("eal/windows: implement basic memory management")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
> > ---
> > headers += files(
> > 'rte_os.h',
> > - 'rte_virt2phys.h',
> > 'rte_windows.h',
> > )
>
> We should plan to rename it.
> Files starting with rte_ are supposed to be public.
Is "rte_eal_" prefix OK for internal headers, like in "rte_eal_paging.h"?
I want to put NetUIO interface to EAL, but also to use it from bus/pci.