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) > + 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. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html