[v1,1/1] usertools/devbind: add documentation for no-IOMMU mode

Message ID a1a8fce7e0eb311e7ca673867862a549270f5450.1733132101.git.anatoly.burakov@intel.com (mailing list archive)
State Accepted
Delegated to: Thomas Monjalon
Headers
Series [v1,1/1] usertools/devbind: add documentation for no-IOMMU mode |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation warning apply patch failure
ci/iol-testing warning apply patch failure
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/intel-Functional success Functional PASS

Commit Message

Burakov, Anatoly Dec. 2, 2024, 9:35 a.m. UTC
Support for `--noiommu-mode` flag was added, but documentation for it was
not. Add documentation for the flag.

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
 doc/guides/tools/devbind.rst | 5 +++++
 1 file changed, 5 insertions(+)
  

Comments

Burakov, Anatoly Dec. 4, 2024, 12:33 p.m. UTC | #1
On 12/2/2024 10:35 AM, Anatoly Burakov wrote:
> Support for `--noiommu-mode` flag was added, but documentation for it was
> not. Add documentation for the flag.
> 
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---

This update is also included in my devbind rewrite:

https://patches.dpdk.org/project/dpdk/list/?series=34098
  
Burakov, Anatoly Feb. 18, 2025, 9:28 a.m. UTC | #2
On 02/12/2024 10:35, Anatoly Burakov wrote:
> Support for `--noiommu-mode` flag was added, but documentation for it was
> not. Add documentation for the flag.
> 
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---
>   doc/guides/tools/devbind.rst | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
> index 841615570f..e010d4b7df 100644
> --- a/doc/guides/tools/devbind.rst
> +++ b/doc/guides/tools/devbind.rst
> @@ -62,6 +62,11 @@ OPTIONS
>         Use this flag to change ownership to the specified user and group, so that
>         devices bound to VFIO would be usable by unprivileged users.
>   
> +* ``--noiommu-mode``
> +
> +      When using vfio-pci driver on a system with no IOMMU, this flag should be used to
> +      enable unsafe no-IOMMU mode for vfio-pci driver.
> +
>   
>   .. warning::
>   

Ping.
  
Bruce Richardson Feb. 18, 2025, 9:46 a.m. UTC | #3
On Mon, Dec 02, 2024 at 09:35:01AM +0000, Anatoly Burakov wrote:
> Support for `--noiommu-mode` flag was added, but documentation for it was
> not. Add documentation for the flag.
> 
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---
>  doc/guides/tools/devbind.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
> index 841615570f..e010d4b7df 100644
> --- a/doc/guides/tools/devbind.rst
> +++ b/doc/guides/tools/devbind.rst
> @@ -62,6 +62,11 @@ OPTIONS
>        Use this flag to change ownership to the specified user and group, so that
>        devices bound to VFIO would be usable by unprivileged users.
>  
> +* ``--noiommu-mode``
> +
> +      When using vfio-pci driver on a system with no IOMMU, this flag should be used to
> +      enable unsafe no-IOMMU mode for vfio-pci driver.
> +
>  
>  .. warning::

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
  
Thomas Monjalon March 19, 2025, 4:02 a.m. UTC | #4
18/02/2025 10:46, Bruce Richardson:
> On Mon, Dec 02, 2024 at 09:35:01AM +0000, Anatoly Burakov wrote:
> > Support for `--noiommu-mode` flag was added, but documentation for it was
> > not. Add documentation for the flag.
> > 
> > Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks.
  

Patch

diff --git a/doc/guides/tools/devbind.rst b/doc/guides/tools/devbind.rst
index 841615570f..e010d4b7df 100644
--- a/doc/guides/tools/devbind.rst
+++ b/doc/guides/tools/devbind.rst
@@ -62,6 +62,11 @@  OPTIONS
       Use this flag to change ownership to the specified user and group, so that
       devices bound to VFIO would be usable by unprivileged users.
 
+* ``--noiommu-mode``
+
+      When using vfio-pci driver on a system with no IOMMU, this flag should be used to
+      enable unsafe no-IOMMU mode for vfio-pci driver.
+
 
 .. warning::