-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fs/littlefs: Add to get the full path #11729
Conversation
Signed-off-by: zhouliang3 <[email protected]>
@crafcat7 let's put littlefs patch in the same patch to avoid the ci break. |
Okay, I'll sort out the relevant changes and add them in |
Missing lfs_file_path |
623b017
to
9cd56b1
Compare
Please try to compile it again with the latest changes, I submitted an implementation of lfs_file_path for littlefs. |
It works now. Thanks. |
9cd56b1
to
0b112c0
Compare
Signed-off-by: chenrun1 <[email protected]>
0b112c0
to
bdbb825
Compare
@crafcat7 have you submitted the littlefs patch to the upstream? i don't think it's a good idea to maintain it in nuttx. |
I sorted out the new interfaces for obtaining paths and submitted them here. |
is this functionality mainly for #11724 ? |
The current implementation of filelock still relies on getting the path to the file to determine the target. So if we want to use filelock on a file on littlefs, we need a way to get the path to the target file. |
Yes, otherwise we need save path into file handle. |
Summary
Add FIOC_FILEPATH to LFS ioctl
Impact
LFS ioctl adds FIOC_FILEPATH for file paths
Testing
local test PASS