RFC: Patch for mountd to handle a database for exports
- Reply: Peter Eriksson : "Re: RFC: Patch for mountd to handle a database for exports"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Thu, 20 Jul 2023 22:50:15 UTC
Hi, Peter Eriksson has submitted an interesting patch that adds support for a database file for exports. My understanding is that this improves performance when used with the ZFS share property for exporting a large number of file systems. There are a couple of user visible issues that I'd like feedback from others. (Hopefully Peter can correct me if I get any of these wrong.) - The patch uses a single database file and a new "-D" option to specify it on the command line. --> I think it might be less confusing to just put the database file(s) in the exports list and identify them via a ".db" filename suffix. What do others think? The changes are #ifdef'd on USE_SHAREDB. I'm thinking that this change will be always built, so maybe USE_SHAREDB is not needed? (Obviously mountd(8)'s semantics will only changed if/when database file(s) are provided.) Once I have feedback on the above, I will put a patch up on phabricator. rick ps: I believe kevans@ has volunteered to shepperd the ZFS share property changes.