On Okt 21 10:37, Jinhao Fan wrote:
> at 7:35 PM, Klaus Jensen <i...@irrelevant.dk> wrote:
> 
> > Commit 2e53b0b45024 ("hw/nvme: Use ioeventfd to handle doorbell
> > updates") had the unintended effect of disabling batching of CQEs.
> > 
> > This patch changes the sq/cq timers to bottom halfs and instead of
> > calling nvme_post_cqes() immediately (causing an interrupt per cqe), we
> > defer the call.
> 
> This change is definitely desired.
> 
> > 
> >                   | iops
> >  -----------------+------
> >    baseline       | 138k
> >    +cqe batching  | 233k
> 
> What is the queue depth config for this test?
> 

That's 64. My box isnt nearly as beefy as yours ;)

Attachment: signature.asc
Description: PGP signature

Reply via email to