On Mon, Mar 10, 2014 at 03:37:16AM -0700, Christoph Hellwig wrote:
> I think the right fix is to stop abusing the iolock in filestreams.
> To me it seems like a look inside fstrm_item_t should be fine
> for what the filestreams code wants if I understand it correctly.
Seems like the iolock could be removed fairly easily by using either of
the two options:
a) reference count fstrm_item, and just grab a reference to it for each
child as well as the parent and insert it multiple times. Kill
->pip.
b) only allocate and insert fstrm_items for directories. Find the
directory by grabbing an entry off inode->i_dentry and then grabbing
the parent. There always should be a dentry around when we allocate
blocks, and if none we can just skip out of the filestreams
allocator if there's none. For the cases that matter there is.
Both mean that the race it tries to protect against using the iolock is
remove entirely, and the code becomes more efficient as well. Option a)
seems simple to implement, but b) will save a lot more memory and
operations when using the filestreams allocator.
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
