Re: [PATCH 00/18] RAS daemon: Easter Eggs Edition

From: Arnaldo Carvalho de Melo
Date: Fri Apr 29 2011 - 13:35:40 EST


Em Fri, Apr 29, 2011 at 04:31:45PM +0200, Ingo Molnar escreveu:
>
> * Borislav Petkov <bp@xxxxxxxxx> 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?

+1

> Thanks,
>
> Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/