-
Notifications
You must be signed in to change notification settings - Fork 133
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
WARNING: failed to getxattr #431
Comments
e84681f should fix this. It seems |
I'm not seeing speed improvements when using
Is there any way to further troubleshoot this and verify that the extended attributes are working? |
So the warnings have stopped, but now the xattr's are no longer being written to the files? |
@SeeSpotRun this is what I'm seeing, yes. |
Strange, because e84681f doesn't actually change the "payload" line which is the https://github.com/sahib/rmlint/blame/develop/lib/xattr.c#L185 Has something else changed? Can you manage to write xattrs for any files? |
Hmm, do you have a specific test scenario I can execute to give you the info needed to isolate this? |
The following should be enough to test:
If you have different filesystem types maybe try it on each of them. |
Alright, this worked perfectly (APFS volume). I found that the attributes WERE indeed stored correctly at least in this test run:
So I looked again at the previous run file attributes. When I first tested, I'm going to do ANOTHER run now and see if the time improves, but I fear that it won't... Is there another test (or argument I'm missing) I can perform during the run that will verify as to whether the extended attributes are not only present, but also being COMPARED/USED vs generating new ones DURING |
Not at present. I suspect what you are seeing is same as #462. Non-duplicate files are generally not fully hashed because their uniqueness is determined either by their unique file length, or because they differ from other same-length files somewhere in the first few bytes or kilobytes. I'm working on an option for #462 and this may also resolve your problem. |
Hmm, perhaps you're right. I'm extremely discombobulated right now because on this last run it flew through the process and comparison of around 6TB of files within just about half an hour. For some reason the first three runs did NOT do this and ran multiple days cumulatively (approximately 20 hours a run), but this time it clearly used the extended attribute data... By the way, are you aware of whether or not extended attributes work properly with ZFS on macOS yet? I assume that it does since they seem to work fine otherwise, but I thought I'd pose the question before digging into a protracted test. |
No idea |
Can you tell me what I can run outside of |
You can just do:
|
This suppresses this unhelpful warning on macOS: WARNING: failed to getxattr for some_file: Attribute not found Also, return the system call result (0 or -1) from rm_xattr_is_fail instead of the errno value. Although not currently used, this return value is more idiomatic. Fixes #431
Running "rmlint --xattr" is throwing these warnings...
WARNING: failed to getxattr for blah blah blah: Attribute not found
I assume these warnings are normal when running for the first time as related extended attributes are not there yet but I expect these warnings to stop on subsequent executions on the same set of files but they do not.
I even checked the files with "xattr -l " and it correctly shows the rmlint user.rmlint.blake2b.cksum and user.rmlint.blake2b.mtime extended attributes which means that rmlint was able to successfully write the extended attributes for the files but then why rmlint is throwing these warnings?
This is on macOS Mojave with APFS filesystem. rmlint was installed using brew.
The text was updated successfully, but these errors were encountered: