On Sat, Feb 03, 2007 at 08:51:36AM +0000, Christoph Hellwig wrote: > On Sat, Feb 03, 2007 at 12:33:59AM -0500, J. Bruce Fields wrote: > > + */ > > +int vfs_lock_file(struct file *filp, unsigned int cmd, struct file_lock > > *fl) > > +{ > > + if (filp->f_op && filp->f_op->lock) > > + return filp->f_op->lock(filp, cmd, fl); > > + else > > + return posix_lock_file(filp, fl); > > +} > > +EXPORT_SYMBOL(vfs_lock_file); > > _GPL please (and same for the last patch)
Oops, sorry--I think you asked for that before. OK, done. > > > + for (;;) { > > + error = vfs_lock_file(filp, cmd, file_lock); > > + if ((error != -EAGAIN) || (cmd == F_SETLK)) > > No need for the inside braces here. > > > + error = vfs_lock_file(filp, cmd, file_lock); > > + if ((error != -EAGAIN) || (cmd == F_SETLK64)) > > again. Got it, thanks.--b. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html