Skip to content

docs(hipFile): Create new guides for setting up storage targets#8897

Open
riley-dixon wants to merge 3 commits into
developfrom
users/rildixon/hipfile-docs-add-drive-setup
Open

docs(hipFile): Create new guides for setting up storage targets#8897
riley-dixon wants to merge 3 commits into
developfrom
users/rildixon/hipfile-docs-add-drive-setup

Conversation

@riley-dixon

@riley-dixon riley-dixon commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Motivation

Adds guides for setting up a local NVMe, remote NVMeoF, and remote NFSoRDMA storage target to run hipFile against.

Technical Details

Adds three new install guides under docs/install/: setup-local-nvme.rst, setup-nvmeof.rst, and setup-nfsordma.rst, covering local NVMe partitioning, NVMeoF target/initiator setup, and NFS-over-RDMA export/mount respectively. Each follows the existing install-guide structure (prerequisites, numbered setup steps, verification) and is cross-linked from install.rst.

Issue Tracking

JIRA ID: AIHIPFILE-236 AIROCDOC-4110

Test Plan

N/A

Test Result

N/A

Submission Checklist

@riley-dixon
riley-dixon requested a review from a team July 20, 2026 23:04
@riley-dixon riley-dixon self-assigned this Jul 20, 2026
@riley-dixon
riley-dixon requested a review from a team as a code owner July 20, 2026 23:04
Copilot AI review requested due to automatic review settings July 20, 2026 23:04
@riley-dixon riley-dixon added documentation Improvements or additions to documentation project: hipfile labels Jul 20, 2026
@therock-pr-bot

Copy link
Copy Markdown

✅ All Policy Checks Passed

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled

🎉 All policy checks passed!

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@riley-dixon
riley-dixon force-pushed the users/rildixon/hipfile-docs-add-drive-setup branch from e935d55 to eec5a5c Compare July 20, 2026 23:06
@riley-dixon

Copy link
Copy Markdown
Contributor Author

Note: this PR is based on top of #7790

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds and refines hipFile documentation to help users set up supported storage targets (local NVMe, NVMeoF, and NFS-over-RDMA) and centralizes install instructions in the published docs site.

Changes:

  • Replaces the in-repo INSTALL.md content with a pointer to the official hipFile install docs.
  • Adds new install guides for local NVMe setup, NVMeoF target/initiator setup, and NFSoRDMA setup.
  • Updates reference/troubleshooting docs around ROCm requirements and NFSoRDMA behavior (HIPFILE_UNSUPPORTED_FILE_SYSTEMS).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
projects/hipfile/INSTALL.md Replaces lengthy install/build instructions with a link to the official docs site.
projects/hipfile/docs/troubleshooting/troubleshooting.rst Updates the documented ROCm minimum version for fastpath prerequisites.
projects/hipfile/docs/reference/hipFile-io-backends.rst Adds an explicit note about HIPFILE_UNSUPPORTED_FILE_SYSTEMS when using NFSoRDMA.
projects/hipfile/docs/reference/hipFile-environment-variables.rst Documents the NFSoRDMA requirement for HIPFILE_UNSUPPORTED_FILE_SYSTEMS.
projects/hipfile/docs/install/setup-nvmeof.rst New NVMeoF target/initiator setup and verification walkthrough.
projects/hipfile/docs/install/setup-nfsordma.rst New NFS-over-RDMA export/mount and verification walkthrough.
projects/hipfile/docs/install/setup-local-nvme.rst New local NVMe partition/format/mount and verification walkthrough.
projects/hipfile/docs/install/install.rst Updates ROCm doc link reference and adds a note about experimental NVMeoF/NFSoRDMA support.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread projects/hipfile/docs/install/setup-nvmeof.rst
Comment thread projects/hipfile/docs/install/setup-nvmeof.rst
Comment thread projects/hipfile/docs/install/setup-nfsordma.rst Outdated
@riley-dixon
riley-dixon force-pushed the users/rildixon/hipfile-docs-add-drive-setup branch from eec5a5c to 25f4641 Compare July 20, 2026 23:17
@riley-dixon
riley-dixon requested a review from a team as a code owner July 20, 2026 23:17
@gaoikawa gaoikawa changed the title hipFile: Create new guides for setting up storage targets docs(hipFile): Create new guides for setting up storage targets Jul 21, 2026
utilities that can be used instead. Before you begin, make sure hipFile is
installed. See :doc:`./install` for package options or :doc:`./build-from-source`
to build from source.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we should add the couple of limitations we have, like the fact that we don't support lvm volumes (yet!) and mdraid. Only because the lvm volume issue seems to come up a lot.

the transfer is fully zero-copy end-to-end depends on the NFS server's own
RDMA support for its storage backend; consult your NFS server documentation.

Configure the InfiniBand interface

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest replacing 'Infiniband' with 'RDMA' so it's more generic.

.. code:: shell

sudo mkdir /mnt/nfs
sudo mount -o rdma,port=20049 <server_ip>:<export_path> /mnt/nfs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious if here we want to talk about any tuning options, like 'nconnect'? @kurtmcmillan maybe has some direct experience with this.

- Version 31.40 or newer of the ``amdgpu-dkms`` driver
- ``nvme-cli`` installed on the initiator (client) system
- Network or fabric connectivity to an NVMeoF target. This guide's examples
use RDMA; TCP is also supported.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'TCP is also supported' -> is that actually true? I thought we needed RDMA for actual hipFile fastpath to work.

$ sudo journalctl -k | grep nvmet
nvmet_rdma: enabling port 1 (<target_ip>:<target_port>)

.. warning::

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make this stick out more, somehow - like a surrounded in a yellow box or something like that?

Perhaps @spolifroni-amd has a suggestion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation organization: ROCm project: hipfile

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants