patch 'doc: add Linux capability to access physical addresses' has been queued to stable release 22.11.2

Xueming Li xuemingl at nvidia.com
Sun Apr 9 17:45:29 CEST 2023


Hi,

FYI, your patch has been queued to stable release 22.11.2

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 04/11/23. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://git.dpdk.org/dpdk-stable/log/?h=22.11-staging

This queued commit can be viewed at:
https://git.dpdk.org/dpdk-stable/log/?h=22.11-staging/commit/d4dc7b1d1b751d30e2bef01ca8df7d8b2e626fd5

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From d4dc7b1d1b751d30e2bef01ca8df7d8b2e626fd5 Mon Sep 17 00:00:00 2001
From: Dmitry Kozlyuk <dmitry.kozliuk at gmail.com>
Date: Fri, 20 Jan 2023 00:24:24 +0300
Subject: [PATCH] doc: add Linux capability to access physical addresses
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit 50b567c66da268bcc7206dab92f690148d57ad8f ]

DAC_READ_SEARCH or DAC_OVERRIDE capability is required to access
/proc/self/pagemap, but the Linux guide mentioned neither one.
Recommend DAC_READ_SEARCH as less impactful.

Fixes: 979bb5d493fb ("doc: add more instructions for running as non-root")

Reported-by: Boris Ouretskey <borisusun at gmail.com>
Reported-by: Isaac Boukris <iboukris at gmail.com>
Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk at gmail.com>
---
 .mailmap                             | 2 ++
 doc/guides/linux_gsg/enable_func.rst | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/.mailmap b/.mailmap
index c09043056c..5dc95a9d07 100644
--- a/.mailmap
+++ b/.mailmap
@@ -169,6 +169,7 @@ Björn Töpel <bjorn.topel at intel.com>
 Bo Chen <box.c.chen at intel.com>
 Boleslav Stankevich <boleslav.stankevich at oktetlabs.ru>
 Boon Ang <bang at vmware.com>
+Boris Ouretskey <borisusun at gmail.com>
 Boris Pismenny <borisp at mellanox.com>
 Brandon Lo <blo at iol.unh.edu>
 Brendan Ryan <brendan.ryan at intel.com>
@@ -528,6 +529,7 @@ Ilya Maximets <i.maximets at ovn.org> <i.maximets at samsung.com>
 Ilya V. Matveychikov <matvejchikov at gmail.com>
 Ilyes Ben Hamouda <ilyes.ben_hamouda at 6wind.com>
 Intiyaz Basha <intiyaz.basha at caviumnetworks.com>
+Isaac Boukris <iboukris at gmail.com>
 Itsuro Oda <oda at valinux.co.jp>
 Ivan Boule <ivan.boule at 6wind.com>
 Ivan Dyukov <i.dyukov at samsung.com>
diff --git a/doc/guides/linux_gsg/enable_func.rst b/doc/guides/linux_gsg/enable_func.rst
index 829084d80e..2344d97403 100644
--- a/doc/guides/linux_gsg/enable_func.rst
+++ b/doc/guides/linux_gsg/enable_func.rst
@@ -55,12 +55,12 @@ Refer to the `documentation <https://www.kernel.org/doc/Documentation/vm/hugetlb
 If the driver requires using physical addresses (PA),
 the executable file must be granted additional capabilities:
 
-* ``SYS_ADMIN`` to read ``/proc/self/pagemaps``
+* ``DAC_READ_SEARCH`` and ``SYS_ADMIN`` to read ``/proc/self/pagemaps``
 * ``IPC_LOCK`` to lock hugepages in memory
 
 .. code-block:: console
 
-   setcap cap_ipc_lock,cap_sys_admin+ep <executable>
+   setcap cap_dac_read_search,cap_ipc_lock,cap_sys_admin+ep <executable>
 
 If physical addresses are not accessible,
 the following message will appear during EAL initialization::
-- 
2.25.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2023-04-09 21:45:42.349381500 +0800
+++ 0140-doc-add-Linux-capability-to-access-physical-addresse.patch	2023-04-09 21:45:38.819042200 +0800
@@ -1 +1 @@
-From 50b567c66da268bcc7206dab92f690148d57ad8f Mon Sep 17 00:00:00 2001
+From d4dc7b1d1b751d30e2bef01ca8df7d8b2e626fd5 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit 50b567c66da268bcc7206dab92f690148d57ad8f ]
@@ -11 +13,0 @@
-Cc: stable at dpdk.org
@@ -22 +24 @@
-index cac02a6f48..0859104404 100644
+index c09043056c..5dc95a9d07 100644
@@ -25 +27 @@
-@@ -171,6 +171,7 @@ Björn Töpel <bjorn.topel at intel.com>
+@@ -169,6 +169,7 @@ Björn Töpel <bjorn.topel at intel.com>
@@ -33 +35 @@
-@@ -534,6 +535,7 @@ Ilya Maximets <i.maximets at ovn.org> <i.maximets at samsung.com>
+@@ -528,6 +529,7 @@ Ilya Maximets <i.maximets at ovn.org> <i.maximets at samsung.com>


More information about the stable mailing list