lkml.org 
[lkml]   [2014]   [Dec]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [CFT] Can I get some Tested-By's on this series?
    Am 16.12.2014 um 03:05 schrieb Andy Lutomirski:
    > On Wed, Dec 10, 2014 at 8:39 AM, Eric W. Biederman
    > <ebiederm@xmission.com> wrote:
    >>
    >> 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 <luto@amacapital.net> # 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


    \
     
     \ /
      Last update: 2014-12-16 11:01    [W:4.985 / U:0.216 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site