Skip to content

Increase NFS rsize/wsize for netboot tutorial #1796

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

ndanyluk
Copy link

rsize and wsize normally default to 1048576 (1MB) for NFS mounts, but for an NFS root device they default to 4096 (4KB). This can lead to a significant performance degradation if the user expects the normal defaults.

rsize and wsize normally default to 1048576 (1MB) for NFS mounts, but for an NFS root device they default to 4096 (4KB).  This can lead to a significant performance degradation if the user expects the normal defaults.
@JamesH65
Copy link
Contributor

@timg236 Do you know anything about this? I have no idea.

@timg236
Copy link
Contributor

timg236 commented Mar 11, 2021

I've never tried it

@ndanyluk
Copy link
Author

@JamesH65 @timg236 rsize and wsize are parameters that govern the negotiated packet size for NFS filesystems. They cap at 1MB (the param is in bytes). With a typical NFS mount on a Pi, this defaults to 1MB. However, the NFSROOT kernel directive has different default values. In the case of the kernel NFSROOT, these parameters default to 4KB which fragments every read/write to the NFS root since each packet will be capped at 4KB rather than a max at 1MB.

NFSROOT kernel.org documentation

@timg236
Copy link
Contributor

timg236 commented Mar 18, 2021

I think it would be better to put that text in there explaining that to improve performance you can add those options

@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

In light of #1911 is this PR still relevant? If there is supporting third-party documentation outside of this repo, and there hasn't been much recent progress I'd be inclined to close it? This feels very much non-Raspberry Pi specific material and therefore outside the scope of the more lightweight documentation set?

@aallan
Copy link
Contributor

aallan commented Jun 8, 2021

After some consideration I'm closing.

@aallan aallan closed this Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants