On Tue, Mar 05, 2013 at 05:37:34PM -0700, Liu Bo wrote: > On Tue, Mar 05, 2013 at 10:40:17AM -0500, Chris Mason wrote: > > diff --git a/fs/btrfs/delayed-inode.c b/fs/btrfs/delayed-inode.c > > index 0b278b1..460d1a8 100644 > > --- a/fs/btrfs/delayed-inode.c > > +++ b/fs/btrfs/delayed-inode.c > > + if (!signal_pending(current)) > > + schedule(); > > Do we need a 'break' here? > > Like > > if (!signal_pending(current)) > schedule(); > else > break; Seems like a good idea ;) Thanks -chris -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
