Subject: Re: [patch 4/4] mISDN: misc timerdev fixes

From: Andi Kleen <[email-masked]>
Date: Tue, 23 Sep 2008 16:14:28 +0200

On Tue, Sep 23, 2008 at 04:30:05AM -0700, Andrew Morton wrote:
> On Tue, 23 Sep 2008 06:40:26 -0400 Christoph Hellwig <[email-masked]> wrote:
> 
> > On Mon, Sep 22, 2008 at 02:51:03PM -0700, [email-masked] wrote:
> > > From: Andi Kleen <[email-masked]>
> > > 
> > > - Remove noop VFS stubs. The VFS does that on a NULL pointer anyways.
> > 
> > > -static loff_t
> > > -mISDN_llseek(struct file *filep, loff_t offset, int orig)
> > > -{
> > > -	return -ESPIPE;
> > > -}
> > 
> > >  static struct file_operations mISDN_fops = {
> > > -	.llseek		= mISDN_llseek,
> > 
> > This is wrong.  no llseek means we use default_llseek, which is
> > different from returning -ESPIPE.
> 
> so.. this?

No, Christoph was wrong on that one. The original patch is ok.
He would have been right if that was a file system, but it isn't.

-Andi