This repository has been archived by the owner on Feb 26, 2020. It is now read-only.
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Explicitly cast the sizeof in hostid_read() to prevent the following compiler warning on 32-bit systems. module/spl/spl-generic.c:490:10: error: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Werror=format] Signed-off-by: Brian Behlendorf <[email protected]>
- Loading branch information
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This commit produces the following error:
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually this happens to me with the parent commit too.
I don't know what is wrong.
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happens with all commits to the extent that I know. Behavior started happening, to the best of my knowledge, after a Fedora package update.
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just installed a fresh FC17 in a VM and am not seeing the errors you described above.
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know what the problem is. The problem is only when the spl sources I'm compiling are stored on a ZFS dataset. The problem does not happen if the file system where the spl checkout is, is an ext4 or a tmpfs file system.
Weird, huh?
e8267ac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've got a hunch this is related to issue openzfs/zfs#764 which we really need to fix. Currently inodes may be a bit to lazily updated in zfs which can confuse certain utilities.