[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1403448187.1949.9.camel@jarvis.lan>
Date: Sun, 22 Jun 2014 07:43:07 -0700
From: James Bottomley <James.Bottomley@...senPartnership.com>
To: Daniel Phillips <daniel@...nq.net>
Cc: Lukáš Czerner <lczerner@...hat.com>,
Pavel Machek <pavel@....cz>,
Dave Chinner <david@...morbit.com>,
linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [RFC] Tux3 for review
On Sat, 2014-06-21 at 20:32 -0700, Daniel Phillips wrote:
> On Saturday, June 21, 2014 12:29:01 PM PDT, James Bottomley wrote:
> > On Thu, 2014-06-19 at 14:58 -0700, Daniel Phillips wrote:
> >> We already removed 450 lines of core kernel workarounds from Tux3 with
> an
> >> approach that was literally cut and pasted from one of Dave's
> >> emails. Then
> >> Dave changed his mind. Now the Tux3 team has been assigned a research
> >> project to improve core kernel writeback instead of simply adapting the
> >> approach that is already proven to work well enough. That is a rather
> >> blatant example of "perfect is the enemy of good enough". Please read
> the
> >> thread.
> >
> > That's a bit disingenuous: the concern has always been how page forking
> > interacted with writeback. It's not new, it was one of the major things
> > brought up at LSF 14 months ago, so you weren't just assigned this.
>
> [citation needed]
Really? I was there; I remember and it's in my notes of the discussion.
However, it's also in Jon's at paragraph 6 if you need to refer to
something to refresh your memory.
However, when it was spotted isn't the issue; how we add tux3 without a
large maintenance burden on writeback is, as I carefully explained in
the rest of the email you cut.
James
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists