Thread (6 messages) flat view 6 messages, 5 authors, 2014-12-16

Re: [CFT] Can I get some Tested-By's on this series?

From: Richard Weinberger <hidden>
Date: 2014-12-16 09:23:38
Also in: linux-man, lkml, stable

Am 16.12.2014 um 03:05 schrieb Andy Lutomirski:
On Wed, Dec 10, 2014 at 8:39 AM, Eric W. Biederman
[off-list ref] wrote:
quoted
Will people please test these patches with their container project?

These changes break container userspace (hopefully in a minimal way) if
I could have that confirmed by testing I would really appreciate it.  I
really don't want to send out a bug fix that accidentally breaks
userspace again.

The only issue sort of under discussion is if there is a better name for
/proc/<pid>/setgroups, and the name of the file will not affect the
functionality of the patchset.

With the code reviewed and written in simple obviously correct, easily
reviewable ways I am hoping/planning to send this to Linus ASAP.

I tested this with Sandstorm.  It breaks as is and it works if I add
the setgroups thing.

Tested-by: Andy Lutomirski <redacted> # breaks things as designed :(

I still don't like the name "setgroups".
I agree that the name is not optimal.
But I don't have a counterproposal as my naming skills are miserable.

Thanks,
//richard
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help