Re: [PATCH 0/5] Allow more than PAGESIZE data read in configfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On Wed, Oct 11, 2006 at 01:19:35PM -0700, Andrew Morton wrote:
> The patch deletes a pile of custom code from configfs and replaces it with
> calls to standard kernel infrastructure and fixes a shortcoming/bug in the
> process.  Migration over to the new interface is trivial and almost
> scriptable.
> Nice patch.  What's not to like about it??

	We're discussing it both online and offline because there are
concerns they have that do need to be addressed.
	What's not to like?  If you need more than 4K to display an
attribute, odds are you have more than one attribute in the file, which
is Wrong.  These guys, it turns out, don't, which is why I'm trying to
help them.  But I don't want to encourage others to do the wrong thing.
	I'm also unhappy with the lack of symmetry this creates on the
store/show() prototypes.  We're discussing whether the store() side
should support larger-than-a-page writes, and how to do it.



"Conservative, n.  A statesman who is enamoured of existing evils,
 as distinguished from the Liberal, who wishes to replace them
 with others."
	- Ambrose Bierce, The Devil's Dictionary

Joel Becker
Principal Software Developer
E-mail: [email protected]
Phone: (650) 506-8127
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at
Please read the FAQ at

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux