login
Header Space

 
 

Re: [PATCH 07/30] IGET: Stop BEFS from using iget() and read_inode()

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: David Howells <dhowells@...>
Cc: <hch@...>, <viro@...>, <torvalds@...>, <akpm@...>, <linux-kernel@...>, <linux-fsdevel@...>
Date: Monday, October 1, 2007 - 1:39 pm

If you're soliciting opinions, I think I tend to prefer the feel of the
code paths after the changes.  I don't know the benefits of the change
are worth the risk in unmaintained file systems, though.


This caught my eye.  Surely we can do better :).  It seems to happen a
few times in the patches, the instance in this patch was the first that
I noticed.

- z
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 00/30] Remove iget() and read_inode(), David Howells, (Mon Oct 1, 9:09 am)
[PATCH 18/30] IGET: Stop ISOFS from using read_inode(), David Howells, (Mon Oct 1, 9:10 am)
Re: [PATCH 07/30] IGET: Stop BEFS from using iget() and read..., Zach Brown, (Mon Oct 1, 1:39 pm)
[PATCH 03/30] IGET: Use iget_failed() in GFS2, David Howells, (Mon Oct 1, 9:09 am)
[PATCH 02/30] IGET: Use iget_failed() in AFS, David Howells, (Mon Oct 1, 9:09 am)
speck-geostationary