[Devel] Re: [PATCH v7 00/10] IPC: checkpoint/restore in userspace enhancements

Eric W. Biederman ebiederm at xmission.com
Thu Oct 18 05:11:08 PDT 2012


Nacked-by: "Eric W. Biederman" <ebiederm at xmission.com>

You ignored all of my feedback that the first 7 messages of your
patchset are unnecessary.  In particular you did not attempt to focus
your patchset on those operations that are most important.

Upon examination it appears also that the 8th and 9th patches of the
patchset are also unnecessary.  And the 10th patch is just a test of the
previous patches, making the 10th patch unneceesary without the rest.

In net this entire patchset is unnecessary and a waste of your reviewers
time.

Once you have IPC checkpoint and restore working there may be a point to
come back and optimize things.  Please don't come back with any System V
Interprocess Communication patches until you can report how much time
is saved by each and every change.

Eric




More information about the Devel mailing list