[dpdk-web] [PATCH] quick-start: use /dev/hugepages instead of /mnt/huge

[dpdk-web] [PATCH] quick-start: use /dev/hugepages instead of /mnt/huge

[Stephen Hemminger]

The standard mount location for hugepages on most Linux distributions
is /dev/hugepages. Make the example safe, so that if hugepages are
already mounted nothing needs to be done.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 content/doc/quick-start.md | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/content/doc/quick-start.md b/content/doc/quick-start.md
index a1c2a6cdf066..6b61b00ba166 100644
--- a/content/doc/quick-start.md
+++ b/content/doc/quick-start.md
@@ -22,14 +22,15 @@ To include the examples as part of the build,
 replace the meson command with:
 
 ```
+
   meson -Dexamples=all build
 ```
 
 Reserve huge pages memory.
 
 ```
-  mkdir -p /mnt/huge
-  mount -t hugetlbfs nodev /mnt/huge
+  mkdir -p /dev/hugepages
+  mountpoint -q /dev/hugepages || mount -t hugetlbfs nodev /dev/hugepages
   echo 64 > /sys/devices/system/node/node0/hugepages/hugepages-2048kB/nr_hugepages
 ```
 
-- 
2.26.2

Re: [dpdk-web] [PATCH] quick-start: use /dev/hugepages instead of /mnt/huge

[Ferruh Yigit]

On 6/24/2020 5:34 PM, Stephen Hemminger wrote:
> The standard mount location for hugepages on most Linux distributions
> is /dev/hugepages. Make the example safe, so that if hugepages are
> already mounted nothing needs to be done.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Re: [dpdk-web] [PATCH] quick-start: use /dev/hugepages instead of /mnt/huge

[Thomas Monjalon]

02/07/2020 13:40, Ferruh Yigit:
> On 6/24/2020 5:34 PM, Stephen Hemminger wrote:
> > The standard mount location for hugepages on most Linux distributions
> > is /dev/hugepages. Make the example safe, so that if hugepages are
> > already mounted nothing needs to be done.
> > 
> > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Removed the extra blank line and applied, thanks

Note: I didn't know "mountpoint", interesting.
Would you like to do the same in the doc guides?