[lkml]   [2008]   [Mar]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch 6/6] vfs: mountinfo: only show mounts under tasks root
    > >  static int show_mountinfo(struct seq_file *m, void *v)
    > > {
    > > + struct proc_mounts *p = m->private;
    > > + size_t count_save = m->count;
    > *UGH*. Do you really need that? Frankly, in that case I'd rather
    > separate the check from __d_path(); unwinds like that are Not Nice(tm).

    Agreed. I did the separate check first, but then there's a window
    between the check and __d_path() where the mountpoint might move
    outside the root, and trying to prevent that would also add quite a
    bit of ugliness.


     \ /
      Last update: 2008-03-20 01:21    [W:0.019 / U:0.424 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site