| To: | Waiman Long <Waiman.Long@xxxxxxx> |
|---|---|
| Subject: | Re: [RFC PATCH-tip 2/6] locking/rwsem: Enable optional count-based spinning on reader |
| From: | Davidlohr Bueso <dave@xxxxxxxxxxxx> |
| Date: | Tue, 14 Jun 2016 11:27:57 -0700 |
| Cc: | Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, x86@xxxxxxxxxx, linux-alpha@xxxxxxxxxxxxxxx, linux-ia64@xxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, linux-arch@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, Jason Low <jason.low2@xxxxxx>, Dave Chinner <david@xxxxxxxxxxxxx>, Scott J Norton <scott.norton@xxxxxxx>, Douglas Hatch <doug.hatch@xxxxxxx> |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1465927959-39719-3-git-send-email-Waiman.Long@xxxxxxx> |
| References: | <1465927959-39719-1-git-send-email-Waiman.Long@xxxxxxx> <1465927959-39719-3-git-send-email-Waiman.Long@xxxxxxx> |
| User-agent: | Mutt/1.5.24 (2015-08-30) |
On Tue, 14 Jun 2016, Waiman Long wrote: This patch provides a way for the kernel code to designate specific rwsems to be more aggressive in term of optimistic spinning that the writers will continue to spin for some additional count-based time to see if it can get the lock before sleeping. This aggressive spinning mode should only be used on rwsems where the readers are unlikely to go to sleep. Yikes, exposing this sort of thing makes me _very_ uneasy, not to mention the ad-hoc nature and its easiness to mess up. I'm not really for this, even if it shows extraordinary performance boosts on benchmarks. Thanks, Davidlohr |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [RFC PATCH-tip 6/6] xfs: Enable reader optimistic spinning for DAX inodes, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [RFC PATCH-tip 6/6] xfs: Enable reader optimistic spinning for DAX inodes, Waiman Long |
| Previous by Thread: | [RFC PATCH-tip 2/6] locking/rwsem: Enable optional count-based spinning on reader, Waiman Long |
| Next by Thread: | Re: [RFC PATCH-tip 2/6] locking/rwsem: Enable optional count-based spinning on reader, Waiman Long |
| Indexes: | [Date] [Thread] [Top] [All Lists] |