On Thu Feb 24, 2011 at 02:40:03PM -0000, Serge Hallyn <email address hidden> wrote:
> Quoting Tyler Hicks (<email address hidden>):
> > I'm not sure I like loops to handle -EINTR in kernel code. That's
> > normally for user space to handle and I don't see why this is any
>
> But you don't let it, bc ecryptfs_lookup_and_interpose_lower
> doesn't expose that it got -EINTR. So either loop, or pass
> the return value to userspace. Current code is broken.
It doesn't sound like you looked at the patch I posted in bug #509180.
Sorry for jumping around between bug reports, but 509180 is the
original report for the interrupted metadata read issue and this one
should likely be marked as a duplicate.
On Thu Feb 24, 2011 at 02:40:03PM -0000, Serge Hallyn <email address hidden> wrote: lookup_ and_interpose_ lower
> Quoting Tyler Hicks (<email address hidden>):
> > I'm not sure I like loops to handle -EINTR in kernel code. That's
> > normally for user space to handle and I don't see why this is any
>
> But you don't let it, bc ecryptfs_
> doesn't expose that it got -EINTR. So either loop, or pass
> the return value to userspace. Current code is broken.
It doesn't sound like you looked at the patch I posted in bug #509180.
Sorry for jumping around between bug reports, but 509180 is the
original report for the interrupted metadata read issue and this one
should likely be marked as a duplicate.