Messages in this thread | | | Date | Fri, 29 Apr 2011 16:31:45 +0200 | From | Ingo Molnar <> | Subject | Re: [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
| |