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.
prev 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