Re: [PATCH 2/2] vhost: lockless enqueuing

2016-04-26 Thread Pankaj Gupta
> > > > On 04/26/2016 02:24 PM, Pankaj Gupta wrote: > > Hi Jason, > > > > Overall patches look good. Just one doubt I have is below: > >> We use spinlock to synchronize the work list now which may cause > >> unnecessary contentions. So this patch switch to use llist to remove > >> this contenti

Re: [PATCH 2/2] vhost: lockless enqueuing

2016-04-26 Thread Jason Wang
On 04/26/2016 02:24 PM, Pankaj Gupta wrote: > Hi Jason, > > Overall patches look good. Just one doubt I have is below: >> We use spinlock to synchronize the work list now which may cause >> unnecessary contentions. So this patch switch to use llist to remove >> this contention. Pktgen tests shows

Re: [PATCH 2/2] vhost: lockless enqueuing

2016-04-25 Thread Pankaj Gupta
Hi Jason, Overall patches look good. Just one doubt I have is below: > > We use spinlock to synchronize the work list now which may cause > unnecessary contentions. So this patch switch to use llist to remove > this contention. Pktgen tests shows about 5% improvement: > > Before: > ~130 pps

[PATCH 2/2] vhost: lockless enqueuing

2016-04-25 Thread Jason Wang
We use spinlock to synchronize the work list now which may cause unnecessary contentions. So this patch switch to use llist to remove this contention. Pktgen tests shows about 5% improvement: Before: ~130 pps After: ~137 pps Signed-off-by: Jason Wang --- drivers/vhost/vhost.c | 52 +