Messages in this thread | | | Date | Fri, 29 Apr 2011 14:35:21 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 00/18] RAS daemon: Easter Eggs Edition |
| |
Em Fri, Apr 29, 2011 at 04:31:45PM +0200, Ingo Molnar escreveu: > > * 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?
+1
> Thanks, > > Ingo
| |