Re: [PATCH v3] sched/fair: Add advisory flag for borrowing a timeslice

From: Khalid Aziz
Date: Tue Nov 25 2014 - 14:25:27 EST


On 11/25/2014 11:27 AM, Davidlohr Bueso wrote:
On Tue, 2014-11-25 at 07:45 -0700, Khalid Aziz wrote:
This solution has been used by both database and java on other OSs and
has shown performance improvement. Andrew had asked for performance
numbers on Linux with this patch last time I sent this out and it took
me a while to get performance folks to run a full TPC-C workload. They
did see a 3% improvement in tpcc as I noted in commit log and that is a
significant improvement.

3% for such a change seems pretty worthless... I would have expected
this having to impact performance much more.


Performance impact will depend upon how big a bottleneck the spinlock was creating and how severe the resulting convoy problem was. Database guys try to squeeze every bit out of the system and 3% is considered to be very good gain. 10% gain would have been nicer :)

--
Khalid
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/