public inbox for linux-ext4@vger.kernel.org
 help / color / mirror / Atom feed
From: Jim Meyering <jim@meyering.net>
To: Ted Ts'o <tytso@mit.edu>
Cc: "Pádraig Brady" <P@draigBrady.com>,
	8411@debbugs.gnu.org, linux-ext4@vger.kernel.org
Subject: Re: bug#8411: due to missing sync even on 2.6.39, cp fails to copy an odd file
Date: Sun, 03 Apr 2011 12:15:17 +0200	[thread overview]
Message-ID: <87wrjbd4ju.fsf@rho.meyering.net> (raw)
In-Reply-To: <20110402230005.GL21075@thunk.org> (Ted Ts'o's message of "Sat, 2 Apr 2011 19:00:05 -0400")

Ted Ts'o wrote:
> On Sat, Apr 02, 2011 at 08:08:34PM +0200, Jim Meyering wrote:
>> From 0a6d128d0d17c1604245f1caafe6af73584a0bb8 Mon Sep 17 00:00:00 2001
>> From: Jim Meyering <meyering@redhat.com>
>> Date: Sat, 2 Apr 2011 19:59:30 +0200
>> Subject: [PATCH] copy: require fiemap sync also for 2.6.38 and 2.6.39 kernels
>>
>> * src/extent-scan.c (extent_need_sync): Require sync also for 2.6.38
>> and 2.6.39.  Without this, part of the cp/fiemap-empty test would fail
>> both on F15-to-be and rawhide.  For discussion and details, see:
>> http://thread.gmane.org/gmane.comp.gnu.coreutils.bugs/22190
>
> FYI, the following fix has been merged into mainline, which should fix
> the problem for 2.6.39 once it is finally released, at least for ext4.
> It was merged right before Linus released 2.6.39-rc1.  I'm assuming
> that Rawhide released a pre-2.6.39-rc1 kernel in the middle of the
> merge window.
>
> Some distro's will informally, but incorrectly, refer to such a
> release as "2.6.39-rc0".  I prefer the more technically correct
> 2.6.38-git18 (which is the first git tag in the Linux git repo which
> contained the patch below, as of March 25, 2011).  Unfortunately, RPM
> doesn't understand that 2.6.38-rc1 sorts before 2.6.38, while
> 2.6.38-git17 sorts *after* 2.6.38.  (Hence the incorrect, but
> convenient, use of 2.6.39-rc0.)

Right.  Rawhide's 2.6.39-0.rc0.git11.0.fc16.x86_64 kernel
is from March 22.

Good.  That means we needn't condemn 2.6.39.
This sort of uname-based kernel check is precisely
why we should minimize use of -rcN named kernels,
but if it affects only rawhide (and that only briefly),
I won't complain too loudly.

      parent reply	other threads:[~2011-04-03 10:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <87fwq0gay0.fsf@rho.meyering.net>
2011-04-02 13:29 ` bug#8411: due to missing sync even on 2.6.39, cp fails to copy an odd file Pádraig Brady
2011-04-02 13:50   ` Jim Meyering
2011-04-02 18:08     ` Jim Meyering
2011-04-02 23:00       ` Ted Ts'o
2011-04-03 10:12         ` Pádraig Brady
2011-04-03 10:26           ` Jim Meyering
2011-04-03 10:46           ` Theodore Tso
2011-04-03 10:15         ` Jim Meyering [this message]

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=87wrjbd4ju.fsf@rho.meyering.net \
    --to=jim@meyering.net \
    --cc=8411@debbugs.gnu.org \
    --cc=P@draigBrady.com \
    --cc=linux-ext4@vger.kernel.org \
    --cc=tytso@mit.edu \
    /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