Re: [PATCH] Btrfs: track transid for delayed ref flushing

2016-06-15 Thread Liu Bo
On Wed, Apr 27, 2016 at 09:59:38AM -0400, Chris Mason wrote: > On Mon, Apr 11, 2016 at 05:37:40PM -0400, Josef Bacik wrote: > > Using the offwakecputime bpf script I noticed most of our time was spent > > waiting > > on the delayed ref throttling. This is what is supposed to happen, but > >

Re: [PATCH] Btrfs: track transid for delayed ref flushing

2016-04-27 Thread David Sterba
On Wed, Apr 27, 2016 at 09:59:38AM -0400, Chris Mason wrote: > > @@ -2854,9 +2855,16 @@ static void delayed_ref_async_start(struct > > btrfs_work *work) > > > > async = container_of(work, struct async_delayed_refs, work); > > > > - trans = btrfs_join_transaction(async->root); > > +

Re: [PATCH] Btrfs: track transid for delayed ref flushing

2016-04-27 Thread Chris Mason
On Mon, Apr 11, 2016 at 05:37:40PM -0400, Josef Bacik wrote: > Using the offwakecputime bpf script I noticed most of our time was spent > waiting > on the delayed ref throttling. This is what is supposed to happen, but > sometimes the transaction can commit and then we're waiting for throttling

Re: [PATCH] Btrfs: track transid for delayed ref flushing

2016-04-12 Thread Josef Bacik
On 04/12/2016 01:43 PM, Liu Bo wrote: On Mon, Apr 11, 2016 at 05:37:40PM -0400, Josef Bacik wrote: Using the offwakecputime bpf script I noticed most of our time was spent waiting on the delayed ref throttling. This is what is supposed to happen, but sometimes the transaction can commit and

Re: [PATCH] Btrfs: track transid for delayed ref flushing

2016-04-12 Thread Liu Bo
On Mon, Apr 11, 2016 at 05:37:40PM -0400, Josef Bacik wrote: > Using the offwakecputime bpf script I noticed most of our time was spent > waiting > on the delayed ref throttling. This is what is supposed to happen, but > sometimes the transaction can commit and then we're waiting for throttling

[PATCH] Btrfs: track transid for delayed ref flushing

2016-04-11 Thread Josef Bacik
Using the offwakecputime bpf script I noticed most of our time was spent waiting on the delayed ref throttling. This is what is supposed to happen, but sometimes the transaction can commit and then we're waiting for throttling that doesn't matter anymore. So change this stuff to be a little