On Mon, Jun 20, 2011 at 03:21:44PM +0200, Frantisek Hrbata wrote: > > +static inline int deny_write_access(struct file *file) > > +{ > > + struct inode *inode = file->f_path.dentry->d_inode; > > + int v, v1; > > + for (v = atomic_read(&inode->i_writecount); v <= 0; v = v1) { > > + v1 = atomic_cmpxchg(&inode->i_writecount, v, v + 1); > ^^^^^ > Shouldn't i_writecount be decreased here. Looks like cut & paste problem to > me. Yes, it should and yes, it is. Thanks for spotting... -- 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