Comment 29 for bug 521523

Revision history for this message
Tyler Hicks (tyhicks) wrote :

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.