lkml.org 
[lkml]   [2010]   [Sep]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 00/15] change default_llseek action
    Date
    On Wednesday 15 September 2010, Valdis.Kletnieks@vt.edu wrote:
    > Show Details
    > On Tue, 14 Sep 2010 22:22:28 +0200, Arnd Bergmann said:
    >
    > > This changes all instances of struct file_operations in
    > > the kernel to have a .llseek operation and then changes
    > > the default to no_llseek, which returns -ESPIPE, which
    > > is what we had decided some time ago in a discussion
    > > with Christoph Hellwig.
    >
    > I don't suppose there's any clean way to throw a build error or a
    > printk_on_once() or something if we encounter an unconverted 'struct
    > file_operations', is there? I have this creeping fear that this patch will go
    > upstream during the merge window - as will 12 new staging/ drivers from authors
    > who didn't get the memo yet.
    >
    > Other than the "missed converting a new usage" issue, it looks OK to me.

    One advantage of the scripted approach is that I can run the script
    repeatedly, and send a patch after -rc1 fixing up all the new file
    operations.

    Moreover, any new drivers are by definition not regressions and staging
    drivers are expected to have issues. IIRC the most common llseek operation
    is 'noop_llseek', which usually means it should have been 'no_llseek' to
    start with, but since this has been in a released kernel, some random
    user application might expect a seek on that file to succeed without
    error.

    Arnd


    \
     
     \ /
      Last update: 2010-09-15 18:41    [from the cache]
    ©2003-2014 Jasper Spaans. Advertise on this site