[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110429143145.GB10590@elte.hu>
Date: Fri, 29 Apr 2011 16:31:45 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Borislav Petkov <bp@...64.org>
Cc: Frederic Weisbecker <fweisbec@...il.com>,
Arnaldo Carvalho de Melo <acme@...radead.org>,
Peter Zijlstra <peterz@...radead.org>,
Steven Rostedt <rostedt@...dmis.org>,
Tony Luck <tony.luck@...el.com>,
Mauro Carvalho Chehab <mchehab@...hat.com>,
David Ahern <dsahern@...il.com>,
EDAC devel <linux-edac@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 00/18] RAS daemon: Easter Eggs Edition
* Borislav Petkov <bp@...64.org> wrote:
> On Fri, Apr 29, 2011 at 10:08:48AM -0400, Frederic Weisbecker wrote:
> >
> > If you want to accelerate that, I suggest you cut your patchset into
> > smaller sets and focus on merging them piece by piece. Few people are going
> > to review 18 patches at once. They may plan to scatter their review over
> > days but that doesn't scale well, and it naturally reduces the number of
> > reviewers.
> >
> > You could start with exporting trace things into tools/trace, post the few
> > concerned patches and iterate them until everyone is happy. Then switch to
> > another subset of the big thing.
>
> Hi Frederic,
>
> I can definitely do that. I just wanted to know before I do it though,
> whether the general direction is ok and whether people are fine with the
> changes, i.e. Arnaldo, Ingo, Peter, Steven, any reactions? If yes, I can get
> to work and split the big pile into more digestible pieces.
I definitely like the general approach, it's a big step forward from mcelog.
Might make sense to progress to the small digestible patches stage to get out
the detailed reviews?
Thanks,
Ingo
--
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