On 02/28/2012 08:20 PM, Christoph Hellwig wrote:
On Tue, Feb 28, 2012 at 07:17:41PM -0800, Alex Elder wrote:
This series cleans up some code involving ceph's virtual extended
attributes. Three of them define some simple macros are set up to
help ensure the attributes are defined in a consistent way. One
makes the size of certain constant values get defined at startup
time rather than repeatedly, and the remaining two are some very
small changes made for clarity.
Is there any reason you can't use the generic_*xattr helpers that
parse attribute names and use the handler array in the superblock?
We should. And I started to look at that but decided to at least
start moving the code in a better, more regular direction. I've
got it on a list but haven't gotten around to it. Lots of this
stuff is just the result of taking my first walk through some
of this code.
I'm still vaguely planning on getting all remaining filesystems
converted over to it.
OK. I'll look into converting this stuff soon.
-Alex
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
