lkml.org 
[lkml]   [2006]   [Oct]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 0/5] Allow more than PAGESIZE data read in configfs
    On Tue, 10 Oct 2006 13:35:11 -0700
    Joel Becker <Joel.Becker@oracle.com> wrote:

    > On Tue, Oct 10, 2006 at 11:20:43AM -0700, Chandra Seetharaman wrote:
    > > Currently, maximum amount of data that can be read from a configfs
    > > attribute file is limited to PAGESIZE bytes. This is a limitation for
    > > some of the usages of configfs.
    >
    > NAK. This forces a complex and inappropriate interface on the
    > majority of users, and doesn't honor configfs' simplicity-first design.

    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??

    > I understand Chandra's concerns, but this patch isn't the right
    > way to do it.

    To do what? Fix the artificial PAGE_SIZE contraint? The patch would be
    justified on cleanup grounds even if nobody was hitting that limit.
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-10-11 22:23    [from the cache]
    ©2003-2011 Jasper Spaans