DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: enhance build instructions on windows
@ 2024-10-25 14:15 Andre Muezerie
  2024-10-25 14:15 ` [PATCH] doc: enhanced build instructions on Windows Andre Muezerie
  0 siblings, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-25 14:15 UTC (permalink / raw)
  Cc: dev, Andre Muezerie

* Updated about old Meson issue
* Added instruction to get developer prompt

Andre Muezerie (1):
  doc: enhanced build instructions on Windows

 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 16 +++++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)

-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH] doc: enhanced build instructions on Windows
  2024-10-25 14:15 [PATCH] doc: enhance build instructions on windows Andre Muezerie
@ 2024-10-25 14:15 ` Andre Muezerie
  2024-10-25 14:50   ` Thomas Monjalon
  2024-11-11 16:14   ` [PATCH v6] doc: modernize build instructions on Windows Andre Muezerie
  0 siblings, 2 replies; 18+ messages in thread
From: Andre Muezerie @ 2024-10-25 14:15 UTC (permalink / raw)
  To: Thomas Monjalon, Dmitry Kozlyuk, Tyler Retzlaff; +Cc: dev, Andre Muezerie

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 16 +++++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420..60db768 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c..c5fad81 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,9 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+Meson version 0.58 was unusable with LLVM toolchain
+because of an `issue <https://github.com/mesonbuild/meson/issues/8981>`_, but
+more recent versions are working fine.
 
 
 Install the Backend
@@ -129,13 +128,20 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'. When doing so, it's recommended
+to specify the Target Architecture (-arch) and the Host Architecture (-host_arch).
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
 .. code-block:: console
 
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -host_arch=amd64 -arch=amd64
+
+Compile the code from the developer prompt.
+
+.. code-block:: console
+
    cd C:\Users\me\dpdk
    meson setup -Denable_stdatomic=true build
    meson compile -C build
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH] doc: enhanced build instructions on Windows
  2024-10-25 14:15 ` [PATCH] doc: enhanced build instructions on Windows Andre Muezerie
@ 2024-10-25 14:50   ` Thomas Monjalon
  2024-10-25 19:30     ` [PATCH v2 0/2] doc: enhance build instructions on windows Andre Muezerie
  2024-11-11 16:14   ` [PATCH v6] doc: modernize build instructions on Windows Andre Muezerie
  1 sibling, 1 reply; 18+ messages in thread
From: Thomas Monjalon @ 2024-10-25 14:50 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: Dmitry Kozlyuk, Tyler Retzlaff, dev

Hello and welcome,

25/10/2024 16:15, Andre Muezerie:
> Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
> ---
> --- a/doc/guides/windows_gsg/build_dpdk.rst
> +++ b/doc/guides/windows_gsg/build_dpdk.rst
> @@ -72,10 +72,9 @@ A good option to choose is the MSI installer for both meson and ninja together::
>  
>  	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
>  
> -Required version is Meson 0.57.
> -
> -Versions starting from 0.58 are unusable with LLVM toolchain
> -because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
> +Meson version 0.58 was unusable with LLVM toolchain
> +because of an `issue <https://github.com/mesonbuild/meson/issues/8981>`_, but
> +more recent versions are working fine.

Please could we be more explicit about which version is working fine?




^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v2 0/2] doc: enhance build instructions on windows
  2024-10-25 14:50   ` Thomas Monjalon
@ 2024-10-25 19:30     ` Andre Muezerie
  2024-10-25 19:30       ` [PATCH v2 1/2] doc: enhanced build instructions on Windows Andre Muezerie
  2024-10-25 19:30       ` [PATCH v2 2/2] doc: provide Meson version that works Andre Muezerie
  0 siblings, 2 replies; 18+ messages in thread
From: Andre Muezerie @ 2024-10-25 19:30 UTC (permalink / raw)
  To: dev; +Cc: Andre Muezerie

* Updated about old Meson issue
* Added instruction to get developer prompt

---
v2:
 - provide Meson version that works

Andre Muezerie (2):
  doc: enhanced build instructions on Windows
  doc: provide Meson version that works

 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 16 +++++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)

-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v2 1/2] doc: enhanced build instructions on Windows
  2024-10-25 19:30     ` [PATCH v2 0/2] doc: enhance build instructions on windows Andre Muezerie
@ 2024-10-25 19:30       ` Andre Muezerie
  2024-10-26 21:03         ` Dmitry Kozlyuk
  2024-10-25 19:30       ` [PATCH v2 2/2] doc: provide Meson version that works Andre Muezerie
  1 sibling, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-25 19:30 UTC (permalink / raw)
  To: dev; +Cc: Andre Muezerie

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 16 +++++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420..60db768 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c..c5fad81 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,9 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+Meson version 0.58 was unusable with LLVM toolchain
+because of an `issue <https://github.com/mesonbuild/meson/issues/8981>`_, but
+more recent versions are working fine.
 
 
 Install the Backend
@@ -129,13 +128,20 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'. When doing so, it's recommended
+to specify the Target Architecture (-arch) and the Host Architecture (-host_arch).
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
 .. code-block:: console
 
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -host_arch=amd64 -arch=amd64
+
+Compile the code from the developer prompt.
+
+.. code-block:: console
+
    cd C:\Users\me\dpdk
    meson setup -Denable_stdatomic=true build
    meson compile -C build
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v2 2/2] doc: provide Meson version that works
  2024-10-25 19:30     ` [PATCH v2 0/2] doc: enhance build instructions on windows Andre Muezerie
  2024-10-25 19:30       ` [PATCH v2 1/2] doc: enhanced build instructions on Windows Andre Muezerie
@ 2024-10-25 19:30       ` Andre Muezerie
  2024-10-26 20:45         ` Dmitry Kozlyuk
  1 sibling, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-25 19:30 UTC (permalink / raw)
  To: dev; +Cc: Andre Muezerie

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 doc/guides/windows_gsg/build_dpdk.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index c5fad81..76bfc22 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -74,7 +74,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 Meson version 0.58 was unusable with LLVM toolchain
 because of an `issue <https://github.com/mesonbuild/meson/issues/8981>`_, but
-more recent versions are working fine.
+more recent versions are working fine. Meson version 1.5.2 was tested and worked.
 
 
 Install the Backend
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v2 2/2] doc: provide Meson version that works
  2024-10-25 19:30       ` [PATCH v2 2/2] doc: provide Meson version that works Andre Muezerie
@ 2024-10-26 20:45         ` Dmitry Kozlyuk
  0 siblings, 0 replies; 18+ messages in thread
From: Dmitry Kozlyuk @ 2024-10-26 20:45 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: dev

2024-10-25 12:30 (UTC-0700), Andre Muezerie:
> Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
> ---
>  doc/guides/windows_gsg/build_dpdk.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
> index c5fad81..76bfc22 100644
> --- a/doc/guides/windows_gsg/build_dpdk.rst
> +++ b/doc/guides/windows_gsg/build_dpdk.rst
> @@ -74,7 +74,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
>  
>  Meson version 0.58 was unusable with LLVM toolchain
>  because of an `issue <https://github.com/mesonbuild/meson/issues/8981>`_, but
> -more recent versions are working fine.
> +more recent versions are working fine. Meson version 1.5.2 was tested and worked.
>  
>  
>  Install the Backend

In order to address reviewer feedback on a single patch,
you are supposed to send an updated version of that patch,
not a series of the original patch (as v2 1/2)
and a patch with amendments (as v2 2/2, i.e. this patch).
IOW, you should have sent a single v2 patch with the proper text.

As for this documentation paragraph: please drop the mention of Meson 0.58
and simply state that Meson 1.5.2 or newer is recommended.
The warning was put in place because the bug lasted many Meson versions.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v2 1/2] doc: enhanced build instructions on Windows
  2024-10-25 19:30       ` [PATCH v2 1/2] doc: enhanced build instructions on Windows Andre Muezerie
@ 2024-10-26 21:03         ` Dmitry Kozlyuk
  2024-10-28 19:45           ` [PATCH v3] doc: enhance " Andre Muezerie
  0 siblings, 1 reply; 18+ messages in thread
From: Dmitry Kozlyuk @ 2024-10-26 21:03 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: dev

2024-10-25 12:30 (UTC-0700), Andre Muezerie:
> -Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
> +Open a 'Visual Studio Developer Command Prompt'. When doing so, it's recommended
> +to specify the Target Architecture (-arch) and the Host Architecture (-host_arch).

It seems that Microsoft documentation [1] uses "target" and "host" terms
for what Meson documentation [2] calls "host" and "build" respectively.
This deserves a notice to avoid confusion.

[1]: https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022
[2]: https://mesonbuild.com/Cross-compilation.html

Why manually specifying architectures is recommended?
If it is mandatory, please use imperative so that no one skips this step.
However, Meson has cross-compilation features, don't they work
(cross-compilation from Windows has never been considered yet)?

>  The developer prompt will configure the environment
>  to select the appropriate compiler, linker and SDK paths
>  required to build with Visual Studio 2022.
>  
>  .. code-block:: console
>  
> +    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -host_arch=amd64 -arch=amd64
> +
> +Compile the code from the developer prompt.
> +
> +.. code-block:: console
> +
>     cd C:\Users\me\dpdk
>     meson setup -Denable_stdatomic=true build
>     meson compile -C build


^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v3] doc: enhance build instructions on Windows
  2024-10-26 21:03         ` Dmitry Kozlyuk
@ 2024-10-28 19:45           ` Andre Muezerie
  2024-10-30  9:09             ` Dmitry Kozlyuk
  0 siblings, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-28 19:45 UTC (permalink / raw)
  To: dev; +Cc: Andre Muezerie

Enhance build instructions on Windows

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
v3:
 - Recommended Meson version to use and removed mention about old bug.
 - Set VS developer prompt to produce 64 bit binaries if needed.
v2:
 - provide Meson version that works

 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 19 ++++++++++++++-----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420..60db768 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c..c81a818 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+Meson version 1.5.2 or newer is recommended.
 
 
 Install the Backend
@@ -129,11 +126,23 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'.
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
+Windows 32-bit is currently not supported, so if your Visual Studio environment defaults
+to 32-bits you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
+For more details about the Developer Prompt options look at the `Visual Studio Developer
+Command Prompt and Developer PowerShell
+<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
+
+.. code-block:: console
+
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
+
+Compile the code from the developer prompt.
+
 .. code-block:: console
 
    cd C:\Users\me\dpdk
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v3] doc: enhance build instructions on Windows
  2024-10-28 19:45           ` [PATCH v3] doc: enhance " Andre Muezerie
@ 2024-10-30  9:09             ` Dmitry Kozlyuk
  2024-10-30 15:19               ` [PATCH v4] doc: modernize " Andre Muezerie
  0 siblings, 1 reply; 18+ messages in thread
From: Dmitry Kozlyuk @ 2024-10-30  9:09 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: dev

2024-10-28 12:45 (UTC-0700), Andre Muezerie:
> Enhance build instructions on Windows

Commit messages should explain why the previous state had been problematic
and how the new state solves the issues. For example:

	doc: modernize build instructions on Windows

	Meson 0.57 was an unstable version and is now outdated.
	The referenced bug in Meson 0.58 is fixed in stable releases.
	Recommend the latest stable release which is tested.

	...likewise about command prompt changes...

[...]
> @@ -129,11 +126,23 @@ Depending on the distribution, paths in this file may need adjustments.
>  Option 3. Native Build on Windows using MSVC
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  
> -Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
> +Open a 'Visual Studio Developer Command Prompt'.
>  The developer prompt will configure the environment
>  to select the appropriate compiler, linker and SDK paths
>  required to build with Visual Studio 2022.
>  
> +Windows 32-bit is currently not supported, so if your Visual Studio environment defaults
> +to 32-bits you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
> +For more details about the Developer Prompt options look at the `Visual Studio Developer
> +Command Prompt and Developer PowerShell
> +<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
> +
> +.. code-block:: console
> +
> +    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
> +
> +Compile the code from the developer prompt.
> +
>  .. code-block:: console
>  
>     cd C:\Users\me\dpdk

It will be unclear to the reader to which machine "32-bit" refers.
Suggestion:

	Building DPDK applications that run on 32-bit Windows is currently not
	supported. If your Visual Studio environment defaults to producing
	32-bit binaries...

With the above corrections,
Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v4] doc: modernize build instructions on Windows
  2024-10-30  9:09             ` Dmitry Kozlyuk
@ 2024-10-30 15:19               ` Andre Muezerie
  2024-10-30 15:48                 ` Stephen Hemminger
  0 siblings, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-30 15:19 UTC (permalink / raw)
  To: dmitry.kozliuk; +Cc: andremue, dev

Meson 0.57 was an unstable version and is now outdated.
The referenced bug in Meson 0.58 is fixed in stable releases.
Recommend the latest stable release which is tested.

Building DPDK applications that run on 32-bit Windows is
currently not supported. However, some Visual Studio environments
default to producing 32-bit binaries.
Recommend instructing the developer prompt to produce 64-bit binaries
when that is the case.

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
v4:
 - Clarified that "32-bit binaries" were refering to the Windows environment.
 - Added clarifying commit messages.
v3:
 - Recommended Meson version to use and removed mention about old bug.
 - Set VS developer prompt to produce 64 bit binaries if needed.
v2:
 - provide Meson version that works

 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 22 ++++++++++++++++------
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420..60db768 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c..ff89e95 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+Meson version 1.5.2 or newer is recommended.
 
 
 Install the Backend
@@ -129,13 +126,26 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'.
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
+Building DPDK applications that run on 32-bit Windows is currently not
+supported. If your Visual Studio environment defaults to producing
+32-bit binaries you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
+For more details about the Developer Prompt options look at the `Visual Studio Developer
+Command Prompt and Developer PowerShell
+<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
+
+.. code-block:: console
+
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
+
+Compile the code from the developer prompt.
+
 .. code-block:: console
 
    cd C:\Users\me\dpdk
    meson setup -Denable_stdatomic=true build
-   meson compile -C build
+   meson compile -C build
\ No newline at end of file
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v4] doc: modernize build instructions on Windows
  2024-10-30 15:19               ` [PATCH v4] doc: modernize " Andre Muezerie
@ 2024-10-30 15:48                 ` Stephen Hemminger
  2024-10-31  3:08                   ` [PATCH v5] " Andre Muezerie
  0 siblings, 1 reply; 18+ messages in thread
From: Stephen Hemminger @ 2024-10-30 15:48 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: dmitry.kozliuk, dev

On Wed, 30 Oct 2024 08:19:45 -0700
Andre Muezerie <andremue@linux.microsoft.com> wrote:

> Meson 0.57 was an unstable version and is now outdated.
> The referenced bug in Meson 0.58 is fixed in stable releases.
> Recommend the latest stable release which is tested.
> 
> Building DPDK applications that run on 32-bit Windows is
> currently not supported. However, some Visual Studio environments
> default to producing 32-bit binaries.
> Recommend instructing the developer prompt to produce 64-bit binaries
> when that is the case.
> 
> Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
> ---
> v4:
>  - Clarified that "32-bit binaries" were refering to the Windows environment.
>  - Added clarifying commit messages.
> v3:
>  - Recommended Meson version to use and removed mention about old bug.
>  - Set VS developer prompt to produce 64 bit binaries if needed.
> v2:
>  - provide Meson version that works
> 
>  .mailmap                              |  1 +
>  doc/guides/windows_gsg/build_dpdk.rst | 22 ++++++++++++++++------
>  2 files changed, 17 insertions(+), 6 deletions(-)
> 
> diff --git a/.mailmap b/.mailmap
> index 5290420..60db768 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
>  Anatoly Burakov <anatoly.burakov@intel.com>
>  Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
>  Anders Roxell <anders.roxell@linaro.org>
> +Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
>  Andrea Arcangeli <aarcange@redhat.com>
>  Andrea Grandi <andrea.grandi@intel.com>
>  Andre Richter <andre.o.richter@gmail.com>
> diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
> index 708875c..ff89e95 100644
> --- a/doc/guides/windows_gsg/build_dpdk.rst
> +++ b/doc/guides/windows_gsg/build_dpdk.rst
> @@ -72,10 +72,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
>  
>  	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
>  
> -Required version is Meson 0.57.
> -
> -Versions starting from 0.58 are unusable with LLVM toolchain
> -because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
> +Meson version 1.5.2 or newer is recommended.
>  
>  
>  Install the Backend
> @@ -129,13 +126,26 @@ Depending on the distribution, paths in this file may need adjustments.
>  Option 3. Native Build on Windows using MSVC
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  
> -Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
> +Open a 'Visual Studio Developer Command Prompt'.
>  The developer prompt will configure the environment
>  to select the appropriate compiler, linker and SDK paths
>  required to build with Visual Studio 2022.
>  
> +Building DPDK applications that run on 32-bit Windows is currently not
> +supported. If your Visual Studio environment defaults to producing
> +32-bit binaries you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
> +For more details about the Developer Prompt options look at the `Visual Studio Developer
> +Command Prompt and Developer PowerShell
> +<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
> +
> +.. code-block:: console
> +
> +    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
> +
> +Compile the code from the developer prompt.
> +
>  .. code-block:: console
>  
>     cd C:\Users\me\dpdk
>     meson setup -Denable_stdatomic=true build
> -   meson compile -C build
> +   meson compile -C build
> \ No newline at end of file

Please fix the file to have newline at end.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v5] doc: modernize build instructions on Windows
  2024-10-30 15:48                 ` Stephen Hemminger
@ 2024-10-31  3:08                   ` Andre Muezerie
  2024-11-11 14:11                     ` Thomas Monjalon
  0 siblings, 1 reply; 18+ messages in thread
From: Andre Muezerie @ 2024-10-31  3:08 UTC (permalink / raw)
  To: stephen; +Cc: andremue, dev, dmitry.kozliuk

Meson 0.57 was an unstable version and is now outdated.
The referenced bug in Meson 0.58 is fixed in stable releases.
Recommend the latest stable release which is tested.

Building DPDK applications that run on 32-bit Windows is
currently not supported. However, some Visual Studio environments
default to producing 32-bit binaries.
Recommend instructing the developer prompt to produce 64-bit binaries
when that is the case.

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
v5:
 - Added new line at end of file build_dpdk.rst
v4:
 - Clarified that "32-bit binaries" was refering to the Windows environment.
 - Added clarifying commit messages.
v3:
 - Recommended Meson version to use and removed mention about old bug.
 - Set VS developer prompt to produce 64 bit binaries if needed.
v2:
 - provide Meson version that works

 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 20 +++++++++++++++-----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420..60db768 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c..cac6725 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+Meson version 1.5.2 or newer is recommended.
 
 
 Install the Backend
@@ -129,11 +126,24 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'.
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
+Building DPDK applications that run on 32-bit Windows is currently not
+supported. If your Visual Studio environment defaults to producing
+32-bit binaries you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
+For more details about the Developer Prompt options look at the `Visual Studio Developer
+Command Prompt and Developer PowerShell
+<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
+
+.. code-block:: console
+
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
+
+Compile the code from the developer prompt.
+
 .. code-block:: console
 
    cd C:\Users\me\dpdk
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v5] doc: modernize build instructions on Windows
  2024-10-31  3:08                   ` [PATCH v5] " Andre Muezerie
@ 2024-11-11 14:11                     ` Thomas Monjalon
  2024-11-11 15:59                       ` Andre Muezerie
  0 siblings, 1 reply; 18+ messages in thread
From: Thomas Monjalon @ 2024-11-11 14:11 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: stephen, dev, andremue, dev, dmitry.kozliuk

31/10/2024 04:08, Andre Muezerie:
> -Required version is Meson 0.57.
> -
> -Versions starting from 0.58 are unusable with LLVM toolchain
> -because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
> +Meson version 1.5.2 or newer is recommended.

I would prefer to know which minimal version is supported.
Giving the latest version is not so helpful.

PS: when sending a new version, please reply to the very first version
to avoid excessive indent in mail list, thanks.



^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v5] doc: modernize build instructions on Windows
  2024-11-11 14:11                     ` Thomas Monjalon
@ 2024-11-11 15:59                       ` Andre Muezerie
  0 siblings, 0 replies; 18+ messages in thread
From: Andre Muezerie @ 2024-11-11 15:59 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: stephen, dev, dmitry.kozliuk

On Mon, Nov 11, 2024 at 03:11:03PM +0100, Thomas Monjalon wrote:
> 31/10/2024 04:08, Andre Muezerie:
> > -Required version is Meson 0.57.
> > -
> > -Versions starting from 0.58 are unusable with LLVM toolchain
> > -because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
> > +Meson version 1.5.2 or newer is recommended.
> 
> I would prefer to know which minimal version is supported.
> Giving the latest version is not so helpful.
> 
> PS: when sending a new version, please reply to the very first version
> to avoid excessive indent in mail list, thanks.
> 
> 
Sorry for the late reply. 

The Meson issue previously mentioned in the documentation took a significant 
amount of time to get fixed. The fix only showed up in the main tree on August 7: 
https://github.com/mesonbuild/meson/pull/11715#event-13793516174

Meson 1.5.2 was the first release containing that fix:
https://github.com/mesonbuild/meson/releases

That's the reason we added that specific version information to the patch, not
just because it was the newest. But I see how it looked like that was the reason.

I'll change the wording a little to make it more clear, and reply to the first version
as suggested.

Thanks for all the suggestions. They are greatly appreciated.

^ permalink raw reply	[flat|nested] 18+ messages in thread

* [PATCH v6] doc: modernize build instructions on Windows
  2024-10-25 14:15 ` [PATCH] doc: enhanced build instructions on Windows Andre Muezerie
  2024-10-25 14:50   ` Thomas Monjalon
@ 2024-11-11 16:14   ` Andre Muezerie
  2024-11-11 18:18     ` Thomas Monjalon
  2024-11-12  9:56     ` Dmitry Kozlyuk
  1 sibling, 2 replies; 18+ messages in thread
From: Andre Muezerie @ 2024-11-11 16:14 UTC (permalink / raw)
  To: andremue; +Cc: dev, dmitry.kozliuk, roretzla, thomas

Meson 0.57 was an unstable version and is now outdated.
The referenced bug in Meson 0.58 is fixed in stable releases.
Recommend the first version containing the fix: 1.5.2.

Building DPDK applications that run on 32-bit Windows is
currently not supported. However, some Visual Studio environments
default to producing 32-bit binaries.
Recommend instructing the developer prompt to produce 64-bit binaries
when that is the case.

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 .mailmap                              |  1 +
 doc/guides/windows_gsg/build_dpdk.rst | 20 +++++++++++++++-----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 5290420258..60db768b98 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,6 +98,7 @@ Anatolii Gerasymenko <anatolii.gerasymenko@intel.com>
 Anatoly Burakov <anatoly.burakov@intel.com>
 Anbarasan Murugesan <anbarasanx.murugesan@intel.com>
 Anders Roxell <anders.roxell@linaro.org>
+Andre Muezerie <andremue@linux.microsoft.com> <andremue@microsoft.com>
 Andrea Arcangeli <aarcange@redhat.com>
 Andrea Grandi <andrea.grandi@intel.com>
 Andre Richter <andre.o.richter@gmail.com>
diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index 708875c16f..2c7bf1a309 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -72,10 +72,7 @@ A good option to choose is the MSI installer for both meson and ninja together::
 
 	http://mesonbuild.com/Getting-meson.html#installing-meson-and-ninja-with-the-msi-installer%22
 
-Required version is Meson 0.57.
-
-Versions starting from 0.58 are unusable with LLVM toolchain
-because of a `Meson issue <https://github.com/mesonbuild/meson/issues/8981>`_.
+The minimal Meson supported version is 1.5.2.
 
 
 Install the Backend
@@ -129,11 +126,24 @@ Depending on the distribution, paths in this file may need adjustments.
 Option 3. Native Build on Windows using MSVC
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-Open a 'Developer PowerShell for VS 2022' prompt from the start menu.
+Open a 'Visual Studio Developer Command Prompt'.
 The developer prompt will configure the environment
 to select the appropriate compiler, linker and SDK paths
 required to build with Visual Studio 2022.
 
+Building DPDK applications that run on 32-bit Windows is currently not
+supported. If your Visual Studio environment defaults to producing
+32-bit binaries you can instruct the toolset to produce 64-bit binaries using "-arch" parameter.
+For more details about the Developer Prompt options look at the `Visual Studio Developer
+Command Prompt and Developer PowerShell
+<https://learn.microsoft.com/en-us/visualstudio/ide/reference/command-prompt-powershell?view=vs-2022>`_.
+
+.. code-block:: console
+
+    "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\Tools\VsDevCmd.bat" -arch=amd64
+
+Compile the code from the developer prompt.
+
 .. code-block:: console
 
    cd C:\Users\me\dpdk
-- 
2.34.1


^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v6] doc: modernize build instructions on Windows
  2024-11-11 16:14   ` [PATCH v6] doc: modernize build instructions on Windows Andre Muezerie
@ 2024-11-11 18:18     ` Thomas Monjalon
  2024-11-12  9:56     ` Dmitry Kozlyuk
  1 sibling, 0 replies; 18+ messages in thread
From: Thomas Monjalon @ 2024-11-11 18:18 UTC (permalink / raw)
  To: andremue, Andre Muezerie; +Cc: dev, dmitry.kozliuk, roretzla

11/11/2024 17:14, Andre Muezerie:
> +The minimal Meson supported version is 1.5.2.

OK looks good, thanks.



^ permalink raw reply	[flat|nested] 18+ messages in thread

* Re: [PATCH v6] doc: modernize build instructions on Windows
  2024-11-11 16:14   ` [PATCH v6] doc: modernize build instructions on Windows Andre Muezerie
  2024-11-11 18:18     ` Thomas Monjalon
@ 2024-11-12  9:56     ` Dmitry Kozlyuk
  1 sibling, 0 replies; 18+ messages in thread
From: Dmitry Kozlyuk @ 2024-11-12  9:56 UTC (permalink / raw)
  To: Andre Muezerie; +Cc: dev, roretzla, thomas

2024-11-11 08:14 (UTC-0800), Andre Muezerie:
> Meson 0.57 was an unstable version and is now outdated.
> The referenced bug in Meson 0.58 is fixed in stable releases.
> Recommend the first version containing the fix: 1.5.2.
> 
> Building DPDK applications that run on 32-bit Windows is
> currently not supported. However, some Visual Studio environments
> default to producing 32-bit binaries.
> Recommend instructing the developer prompt to produce 64-bit binaries
> when that is the case.
> 
> Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>

Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>

^ permalink raw reply	[flat|nested] 18+ messages in thread

end of thread, other threads:[~2024-11-12  9:56 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-10-25 14:15 [PATCH] doc: enhance build instructions on windows Andre Muezerie
2024-10-25 14:15 ` [PATCH] doc: enhanced build instructions on Windows Andre Muezerie
2024-10-25 14:50   ` Thomas Monjalon
2024-10-25 19:30     ` [PATCH v2 0/2] doc: enhance build instructions on windows Andre Muezerie
2024-10-25 19:30       ` [PATCH v2 1/2] doc: enhanced build instructions on Windows Andre Muezerie
2024-10-26 21:03         ` Dmitry Kozlyuk
2024-10-28 19:45           ` [PATCH v3] doc: enhance " Andre Muezerie
2024-10-30  9:09             ` Dmitry Kozlyuk
2024-10-30 15:19               ` [PATCH v4] doc: modernize " Andre Muezerie
2024-10-30 15:48                 ` Stephen Hemminger
2024-10-31  3:08                   ` [PATCH v5] " Andre Muezerie
2024-11-11 14:11                     ` Thomas Monjalon
2024-11-11 15:59                       ` Andre Muezerie
2024-10-25 19:30       ` [PATCH v2 2/2] doc: provide Meson version that works Andre Muezerie
2024-10-26 20:45         ` Dmitry Kozlyuk
2024-11-11 16:14   ` [PATCH v6] doc: modernize build instructions on Windows Andre Muezerie
2024-11-11 18:18     ` Thomas Monjalon
2024-11-12  9:56     ` Dmitry Kozlyuk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).