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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
