On Wed, Jan 11, 2017 at 12:51 PM, Jan Kara <jack@xxxxxxx> wrote: > On Wed 11-01-17 11:29:28, Miklos Szeredi wrote: >> I know there's work on this for xfs, but could this be done in generic mm >> code? >> >> What are the obstacles? page->mapping and page->index are the obvious >> ones. > > Yes, these two are the main that come to my mind. Also you'd need to > somehow share the mapping->i_mmap tree so that unmap_mapping_range() works. > >> If that's too difficult is it maybe enough to share mappings between >> files while they are completely identical and clone the mapping when >> necessary? > > Well, but how would the page->mapping->host indirection work? Even if you > have identical contents of the mappings, you still need to be aware there > are several inodes behind them and you need to pick the right one > somehow... When do we actually need page->mapping->host? The only place where it's not available is page writeback. Then we can know that the original page was already cow-ed and after being cowed, the page belong only to a single inode. What then happens if the newly written data is cloned before being written back? We can either write back the page during the clone, so that only clean pages are ever shared. Or we can let dirty pages be shared between inodes. In that latter case the question is: do we care about which inode we use for writing back the data? Is the inode needed at all? I don't know enough about filesystem internals to see clearly what happens in such a situation. >> All COW filesystems would benefit, as well as layered ones: lots of >> fuse fs, and in some cases overlayfs too. >> >> Related: what can DAX do in the presence of cloned block? > > For DAX handling a block COW should be doable if that is what you are > asking about. Handling of blocks that can be written to while they are > shared will be rather difficult (you have problems with keeping dirty bits > in the radix tree consistent if nothing else). What happens if you do: - clone_file_range(A, off1, B, off2, len); - mmap both A and B using DAX. The mapping will contain the same struct page for two different mappings, no? Thanks, Miklos -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
