On Wed, Sep 06, 2023 at 05:59:03PM +0200, Bernd Schubert wrote:
file_remove_privs might call into notify_change(), which requires to hold an exclusive lock.
Looks good:
Reviewed-by: Christoph Hellwig hch@lst.de
FYI, I'd be really curious about benchmarking this against you version that checks xattrs for shared locked writes on files that have xattrs but not security ones or setuid bits. On the one hand being able to do the shared lock sounds nice, on the other hand even just looking up the xattrs will probably make it slower at least for smaller I/O.