lkml.org 
[lkml]   [2011]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 00/18] RAS daemon: Easter Eggs Edition

* Borislav Petkov <bp@amd64.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


\
 
 \ /
  Last update: 2011-04-29 16:35    [W:0.439 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site