2011/10/25 Josef Bacik <josef@xxxxxxxxxx>: > On Tue, Oct 25, 2011 at 04:15:45PM -0400, Chris Mason wrote: >> On Tue, Oct 25, 2011 at 11:05:12AM -0400, Josef Bacik wrote: >> > On Tue, Oct 25, 2011 at 04:25:02PM +0200, Christian Brunner wrote: >> > > >> > > Attached is a perf-report. I have included the whole report, so that >> > > you can see the difference between the good and the bad >> > > btrfs-endio-wri. >> > > >> > >> > We also shouldn't be running run_ordered_operations, man this is screwed up, >> > thanks so much for this, I should be able to nail this down pretty easily. >> > Thanks, >> >> Looks like we're getting there from reserve_metadata_bytes when we join >> the transaction? >> > > We don't do reservations in the endio stuff, we assume you've reserved all the > space you need in delalloc, plus we would have seen reserve_metadata_bytes in > the trace. Though it does look like perf is lying to us in at least one case > sicne btrfs_alloc_logged_file_extent is only called from log replay and not > during normal runtime, so it definitely shouldn't be showing up. Thanks, Strange! - I'll check if symbols got messed up in the report tomorrow. Christian -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
