linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jens Axboe <jens.axboe@oracle.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Arjan van de Ven <arjan@infradead.org>,
	akpm@osdl.org, linux-kernel@vger.kernel.org,
	linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH] fs: Correct SuS compliance for open of large file  without options
Date: Thu, 27 Sep 2007 17:08:54 +0200	[thread overview]
Message-ID: <20070927150853.GA10154@kernel.dk> (raw)
In-Reply-To: <20070927154432.302ec5a7@the-village.bc.nu>

On Thu, Sep 27 2007, Alan Cox wrote:
> > > Its a change of a specific error return from the wrong error to the right
> > > one, nothing more. Fixing the returned error gives us correct behaviour
> > > according to the standards and other systems.
> > 
> > It may still break applications. Waving some standard at them if they
> > complain is unlikely to impress them.
> 
> And our existing behaviour may well break correctly written
> portable applications, and is incorrect as well.

It's been that way for ages, how likely do you think that is? Not very,
is my guess. Existing practice trumps standard description in my book.

> Testing so far says it doesn't break anything, which is no suprise if you
> apply about ten braincells to the case under consideration.

Well it's not my call, just seems like a really bad idea to change the
error value. You can't claim full coverage for such testing anyway, it's
one of those things that people will complain about two releases later
saying it broke app foo.

-- 
Jens Axboe

  reply	other threads:[~2007-09-27 15:08 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-27 13:29 [PATCH] fs: Correct SuS compliance for open of large file without options Alan Cox
2007-09-27 14:01 ` Arjan van de Ven
2007-09-27 14:19   ` Alan Cox
2007-09-27 14:35     ` Jens Axboe
2007-09-27 14:44       ` Alan Cox
2007-09-27 15:08         ` Jens Axboe [this message]
2007-09-27 15:19           ` Alan Cox
2007-09-27 15:59             ` Theodore Tso
2007-09-27 17:23               ` Andrew Morton
2007-09-27 17:59                 ` Greg KH
2007-09-27 18:37                   ` Theodore Tso
2007-09-27 18:45                     ` Matthew Wilcox
2007-09-27 21:34                     ` Greg KH
2007-09-27 22:27                       ` Kyle Moffett
2007-09-27 23:11                         ` Greg KH
2007-09-27 23:19                       ` Theodore Tso
2007-09-27 23:28                         ` Matthew Wilcox
2007-09-28  3:21                         ` Greg KH
     [not found]                         ` <20070927232857.GA12049@parisc-linux.org>
2007-09-28  2:21                           ` Theodore Tso
2007-09-28  3:22                           ` Greg KH
2007-09-27 23:41               ` Jens Axboe

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070927150853.GA10154@kernel.dk \
    --to=jens.axboe@oracle.com \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=arjan@infradead.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).