Well I found this, so I think it's likely: root@gwboss2:~# dmesg |grep bitten [ 3196.193238] this would have bitten us in the ass [ 3196.193784] this would have bitten us in the ass
On Jan 29, 2013, at 9:54 AM, Josef Bacik <jba...@fusionio.com> wrote: > On Mon, Jan 28, 2013 at 05:12:12PM -0700, Sage Weil wrote: >> A ceph user observed a incorrect i_size on btrfs. The pattern looks like >> this: >> >> - some writes at low file offsets >> - a write to 4185600 len 8704 (i_size should be 4MB) >> - more writes to low offsets >> - a write to 4181504 len 4096 (abutts the write above) >> - a bit of time goes by... >> - stat returns 4186112 (4MB - 8192) >> - that's a fwe bytes to the right of the top write above. >> >> There are some logs showing the full read/write activity to the file at >> >> http://tracker.newdream.net/attachments/658/object_log.txt >> >> on issue >> >> http://tracker.newdream.net/issues/3810 >> >> The kernel was 3.7.0-030700-generic (and probably also observed on 3.7.1). >> >> Is this a known bug? > > Not known but I took a long hard look at our ordered i size updating and I > think > I spotted the bug. Could you run this patch and see if you get the printk? > If > you do then that was the problem and you should be good to go. It definitely > needs to be fixed, hopefully it's also your bug. Thanks, > > Josef > > > diff --git a/fs/btrfs/ordered-data.c b/fs/btrfs/ordered-data.c > index cbd4838..dbd4905 100644 > --- a/fs/btrfs/ordered-data.c > +++ b/fs/btrfs/ordered-data.c > @@ -895,8 +895,14 @@ int btrfs_ordered_update_i_size(struct inode *inode, u64 > offset, > * if the disk i_size is already at the inode->i_size, or > * this ordered extent is inside the disk i_size, we're done > */ > - if (disk_i_size == i_size || offset <= disk_i_size) { > + if (disk_i_size == i_size) > goto out; > + > + if (offset <= disk_i_size) { > + if (ordered && ordered->outstanding_isize > disk_i_size) > + printk(KERN_ERR "this would have bitten us in the > ass\n"); > + else > + goto out; > } > > /* -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
