On Fri, 23 Sept 2022 at 17:47, Christian Brauner <brauner@xxxxxxxxxx> wrote: > > On Fri, Sep 23, 2022 at 05:38:34PM +0200, Miklos Szeredi wrote: > It's basically just like when you copy a file betweed idmapped mounts: > > mount -o X-mount.idmap=<mapping1> --bind /source /source-idmapped > mount -o X-mount.idmap=<mapping2> --bind /target /target-idmapped > > cp /source-idmapped/file1 /target-idmapped > > where you need to take the source and target idmappings into account. > > So basically like what ovl_set_attr() is doing just for acls. But I But then before this patch the copy-up behavior was incorrect, right? In that case this needs to be advertised as a bug fix, rather than a cleanup. Thanks, Miklos