public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Oleg Drokin <green@namesys.com>
To: Anton Altaparmakov <aia21@cantab.net>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Andrew Morton <akpm@digeo.com>,
	mason@suse.com, trond.myklebust@fys.uio.no, jaharkes@cs.cmu.edu,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: 2.4 iget5_locked port attempt to 2.4
Date: Mon, 3 Mar 2003 19:04:17 +0300	[thread overview]
Message-ID: <20030303190417.C4513@namesys.com> (raw)
In-Reply-To: <Pine.SOL.3.96.1030303155239.1630A-100000@virgo.cus.cam.ac.uk>

Hello!

On Mon, Mar 03, 2003 at 03:57:19PM +0000, Anton Altaparmakov wrote:
> > > >    It's me again, I basically got no reply for this iget5_locked patch
> > > >    I have now. Would there be any objections if I try push it to Marcelo
> > > >    tomorrow? ;)
> > > I just binned it. Certainly its not the kind of stuff I want to test in -ac, 
> > > too many VFS changes outside reiserfs
> > Andrew Morton said "iget5_locked() looks simple enough, and as far as I can
> > tell does not change any existing code - it just adds new stuff.",
> > also this code (in its 2.5 incarnation) was tested in 2.5 for long time already.
> > Also it fixes real bug (and while I have another reiserfs-only fix for the bug,
> > it is fairly inelegant).
> I agree it should go into 2.4 but I don't think the patches you are
> submitting should go in. From what I can see they are an older version
> compared to what actually went into 2.5. (I am basing this on the comments

What I am submitting is just changesets 1.373.172.1..1.373.172.6 from
2.5 bk tree. So these patches are what went into 2.5 (plus all the bugs I
have made during adapting these to 2.4, of course).

> to the functions rather than thorough code comparisons but I don't have
> time to look at it more in depth atm.) Why don't you use the actual 2.5
> code and make new patches or at least make use of the patches that finally
> went into 2.5?

Looking at the changelog, it seems much later on there were ifind_fast() and ifind()
additions to this code, but I was not sure I should take these too.
I can though, if people think that would be useful.

Bye,
    Oleg

  reply	other threads:[~2003-03-03 15:53 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20030220175309.A23616@namesys.com>
     [not found] ` <20030220154924.7171cbd7.akpm@digeo.com>
2003-02-21 19:03   ` 2.4 iget5_locked port attempt to 2.4 Oleg Drokin
2003-02-21 20:04     ` Jan Harkes
2003-02-22  9:29       ` Oleg Drokin
2003-02-24 10:21       ` 2.4 iget5_locked port attempt to 2.4 (supposedly fixed NFS version this time) Oleg Drokin
2003-02-24 16:50         ` Trond Myklebust
2003-02-24 17:02           ` Trond Myklebust
2003-02-24 17:03           ` Oleg Drokin
2003-02-24 17:17             ` Trond Myklebust
2003-02-24 17:26               ` Oleg Drokin
2003-02-24 17:33                 ` Trond Myklebust
2003-02-24 17:38                   ` Oleg Drokin
2003-03-03 14:09       ` 2.4 iget5_locked port attempt to 2.4 Oleg Drokin
2003-03-03 16:25         ` Alan Cox
2003-03-03 15:35           ` Nikita Danilov
2003-03-03 15:38           ` Oleg Drokin
2003-03-03 15:57             ` Anton Altaparmakov
2003-03-03 16:04               ` Oleg Drokin [this message]
2003-03-03 16:27                 ` Anton Altaparmakov
2003-03-03 16:45                   ` Oleg Drokin
2003-03-03 16:50             ` Jan Harkes
2003-03-03 16:57               ` Oleg Drokin
2003-03-03 17:23               ` Jan Harkes

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=20030303190417.C4513@namesys.com \
    --to=green@namesys.com \
    --cc=aia21@cantab.net \
    --cc=akpm@digeo.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=jaharkes@cs.cmu.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mason@suse.com \
    --cc=trond.myklebust@fys.uio.no \
    /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