Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-18 Thread Waiman Long
On 08/17/2014 07:41 PM, Dave Chinner wrote: On Fri, Aug 15, 2014 at 01:58:09PM -0400, Waiman Long wrote: On 08/14/2014 11:34 PM, Dave Chinner wrote: xfs_io -f -c "truncate 500t" -c "extsize 1m" /path/to/vm/image/file Thank for the testing recipe. I am afraid that I can't find a 500TB

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-18 Thread Waiman Long
On 08/17/2014 07:41 PM, Dave Chinner wrote: On Fri, Aug 15, 2014 at 01:58:09PM -0400, Waiman Long wrote: On 08/14/2014 11:34 PM, Dave Chinner wrote: create sparse vm image file of 500TB on ssd with XFS on it xfs_io -f -c truncate 500t -c extsize 1m /path/to/vm/image/file Thank for the

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-17 Thread Dave Chinner
On Fri, Aug 15, 2014 at 01:58:09PM -0400, Waiman Long wrote: > On 08/14/2014 11:34 PM, Dave Chinner wrote: > > > > > >xfs_io -f -c "truncate 500t" -c "extsize 1m" /path/to/vm/image/file > > Thank for the testing recipe. I am afraid that I can't find a 500TB > SSD for testing purpose. Which

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-17 Thread Dave Chinner
On Fri, Aug 15, 2014 at 01:58:09PM -0400, Waiman Long wrote: On 08/14/2014 11:34 PM, Dave Chinner wrote: create sparse vm image file of 500TB on ssd with XFS on it xfs_io -f -c truncate 500t -c extsize 1m /path/to/vm/image/file Thank for the testing recipe. I am afraid that I can't

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-16 Thread Mike Galbraith
On Fri, 2014-08-15 at 13:58 -0400, Waiman Long wrote: > Thank for the testing recipe. I am afraid that I can't find a 500TB SSD > for testing purpose. Do you think the test will still be valid for > exercising rwsem if I use a smaller SSD or maybe mechanical hard disk? I suspect fs_mark will

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-16 Thread Mike Galbraith
On Fri, 2014-08-15 at 13:58 -0400, Waiman Long wrote: Thank for the testing recipe. I am afraid that I can't find a 500TB SSD for testing purpose. Do you think the test will still be valid for exercising rwsem if I use a smaller SSD or maybe mechanical hard disk? I suspect fs_mark will fit

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-15 Thread Waiman Long
On 08/14/2014 11:34 PM, Dave Chinner wrote: xfs_io -f -c "truncate 500t" -c "extsize 1m" /path/to/vm/image/file In vm: download and build fsmark from here: git://oss.sgi.com/dgc/fs_mark download and install xfsprogs v3.2.1 from here: git://oss.sgi.com/xfs/cmds/xfsprogs.git tags/v3.2.1

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-15 Thread Waiman Long
On 08/14/2014 11:34 PM, Dave Chinner wrote: create sparse vm image file of 500TB on ssd with XFS on it xfs_io -f -c truncate 500t -c extsize 1m /path/to/vm/image/file start 16p/16GB RAM vm with image file configured as: -drive file=/path/to/vm/image/file,if=virtio,cache=none In vm: download

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-14 Thread Dave Chinner
On Wed, Aug 13, 2014 at 12:41:06PM -0400, Waiman Long wrote: > On 08/13/2014 01:51 AM, Dave Chinner wrote: > >On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: > >>On 08/04/2014 12:10 AM, Jason Low wrote: > >>>On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: > The

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-14 Thread Dave Chinner
On Wed, Aug 13, 2014 at 12:41:06PM -0400, Waiman Long wrote: On 08/13/2014 01:51 AM, Dave Chinner wrote: On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner()

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-13 Thread Waiman Long
On 08/13/2014 01:51 AM, Dave Chinner wrote: On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-13 Thread Waiman Long
On 08/13/2014 01:51 AM, Dave Chinner wrote: On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-12 Thread Dave Chinner
On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: > On 08/04/2014 12:10 AM, Jason Low wrote: > >On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: > >>The rwsem_can_spin_on_owner() function currently allows optimistic > >>spinning only if the owner field is defined and is running.

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-12 Thread Dave Chinner
On Mon, Aug 04, 2014 at 11:44:19AM -0400, Waiman Long wrote: On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-04 Thread Waiman Long
On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-04 Thread Waiman Long
On 08/04/2014 12:10 AM, Jason Low wrote: On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Jason Low
On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: > The rwsem_can_spin_on_owner() function currently allows optimistic > spinning only if the owner field is defined and is running. That is > too conservative as it will cause some tasks to miss the opportunity > of doing spinning in case the

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Davidlohr Bueso
On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: > The rwsem_can_spin_on_owner() function currently allows optimistic > spinning only if the owner field is defined and is running. That is > too conservative as it will cause some tasks to miss the opportunity > of doing spinning in case the

[PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Waiman Long
The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the opportunity of doing spinning in case the owner hasn't been able to set the owner field in time or the

[PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Waiman Long
The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the opportunity of doing spinning in case the owner hasn't been able to set the owner field in time or the

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Davidlohr Bueso
On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the opportunity of doing spinning in case the owner

Re: [PATCH 2/7] locking/rwsem: more aggressive use of optimistic spinning

2014-08-03 Thread Jason Low
On Sun, 2014-08-03 at 22:36 -0400, Waiman Long wrote: The rwsem_can_spin_on_owner() function currently allows optimistic spinning only if the owner field is defined and is running. That is too conservative as it will cause some tasks to miss the opportunity of doing spinning in case the owner