lkml.org 
[lkml]   [2007]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 2/2] Replace pid_t in autofs with struct pid reference
    From
    Date
    On Thu, 2007-03-22 at 08:31 -0500, Serge E. Hallyn wrote:
    > > >
    > > > From: "Serge E. Hallyn" <serue@us.ibm.com>
    > > > Subject: [PATCH] autofs: prevent pid wraparound in waitqs
    > > >
    > > > Instead of storing pid numbers for waitqs, store references
    > > > to struct pids. Also store a reference to the mounter's pid
    > > > namespace in the autofs4 sb info so that pid numbers for
    > > > mount miss and expiry msgs can send the pid# in the mounter's
    > > > pidns.
    > >
    > > I think this amounts to what I suggested in my previous replies.
    > > Hopefully my comments are enough to clear up any questions on
    > > correctness of this approach.
    > >
    > > Sorry to be a pain but I'm having a little trouble reviewing the patch
    > > because I'm not clear on where the code to handle the automount process
    > > group (so called oz_pgrp), from the first patch, fits in with this.
    >
    > It also has pidspace infrastructure code in it, so I think we will just
    > hold off on this until we have that infrastructure merged into the
    > pidspace code and into -mm. Then we can send you a single, more concise
    > patch.

    OK great.

    I'll have a close look through the code and it's related calls when I
    get hold of a complete patch. That will be much easier and mistakes will
    be less likely.

    Ian


    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-03-22 16:13    [W:4.153 / U:1.464 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site