doc: fix references to /dev/huge
Checks
Commit Message
change /dev/huge to /dev/hugepages
Bugzilla ID: 492
Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
---
doc/guides/faq/faq.rst | 2 +-
doc/guides/nics/thunderx.rst | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Comments
23/06/2020 07:55, Sarosh Arif:
> change /dev/huge to /dev/hugepages
>
> Bugzilla ID: 492
> Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
Please could you explain why /dev/hugepages must be used,
in the commit log for the record?
FYI, a similar change was done in the website by Stephen.
On Mon, Jul 6, 2020 at 1:44 AM Thomas Monjalon <thomas@monjalon.net> wrote:
> 23/06/2020 07:55, Sarosh Arif:
> > change /dev/huge to /dev/hugepages
> >
> > Bugzilla ID: 492
> > Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
>
> Please could you explain why /dev/hugepages must be used,
> in the commit log for the record?
>
The path /dev/huge does not exist, the correct path is /dev/hugepages, so
for the sake of accuracy in the documentation it should be changed.
>
> FYI, a similar change was done in the website by Stephen.
>
This bug was submitted by Stephen on bugzilla, I took notice of it and
submitted a patch to correct it. This bug is not yet corrected on the
website.
On 05-Jul-20 9:44 PM, Thomas Monjalon wrote:
> 23/06/2020 07:55, Sarosh Arif:
>> change /dev/huge to /dev/hugepages
>>
>> Bugzilla ID: 492
>> Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
>
> Please could you explain why /dev/hugepages must be used,
> in the commit log for the record?
>
> FYI, a similar change was done in the website by Stephen.
>
/dev/hugepages is the default path used by systemd (check
dev-hugepages.mount unit) so this is the correct path.
06/07/2020 10:01, Sarosh Arif:
> On Mon, Jul 6, 2020 at 1:44 AM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> > 23/06/2020 07:55, Sarosh Arif:
> > > change /dev/huge to /dev/hugepages
> > >
> > > Bugzilla ID: 492
> > > Signed-off-by: Sarosh Arif <sarosh.arif@emumba.com>
> >
> > Please could you explain why /dev/hugepages must be used,
> > in the commit log for the record?
> >
>
> The path /dev/huge does not exist, the correct path is /dev/hugepages, so
> for the sake of accuracy in the documentation it should be changed.
The directory can be created.
As Anatoly said, the reason is that it is the default path
on most systems.
> >
> > FYI, a similar change was done in the website by Stephen.
> >
>
> This bug was submitted by Stephen on bugzilla, I took notice of it and
> submitted a patch to correct it. This bug is not yet corrected on the
> website.
It is fixed in the website.
Applied with more explanations.
@@ -18,7 +18,7 @@ The number of pages allocated can be seen by executing the following command::
Once all the pages are mmapped by an application, they stay that way.
If you start a test application with less than the maximum, then you have free pages.
-When you stop and restart the test application, it looks to see if the pages are available in the ``/dev/huge`` directory and mmaps them.
+When you stop and restart the test application, it looks to see if the pages are available in the ``/dev/hugepages`` directory and mmaps them.
If you look in the directory, you will see ``n`` number of 2M pages files. If you specified 1024, you will see 1024 page files.
These are then placed in memory segments to get contiguous memory.
@@ -156,7 +156,7 @@ This section provides instructions to configure SR-IOV with Linux OS.
-netdev tap,id=net0,ifname=tap0,script=/etc/qemu-ifup_thunder \
-device virtio-net-device,netdev=net0 \
-serial stdio \
- -mem-path /dev/huge
+ -mem-path /dev/hugepages
#. Enable **VFIO-NOIOMMU** mode (optional):