[dpdk-dev] [PATCH v6 3/4] qat: use WC store to update queue tail registers

[Bruce Richardson]

On Mon, Jul 13, 2020 at 01:52:24PM +0100, Trahe, Fiona wrote:
> 
> 
> > -----Original Message-----
> > From: Bruce Richardson <bruce.richardson at intel.com>
> > Sent: Monday, July 13, 2020 1:45 PM
> > To: Nicolau, Radu <radu.nicolau at intel.com>
> > Cc: dev at dpdk.org; Xing, Beilei <beilei.xing at intel.com>; Guo, Jia <jia.guo at intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev at intel.com>; jerinjacobk at gmail.com; david.marchand at redhat.com; Trahe, Fiona
> > <fiona.trahe at intel.com>; Zhao1, Wei <wei.zhao1 at intel.com>
> > Subject: Re: [PATCH v6 3/4] qat: use WC store to update queue tail registers
> >
> > On Mon, Jul 13, 2020 at 01:27:26PM +0100, Radu Nicolau wrote:
> > > Performance improvement: use a write combining store
> > > instead of a regular mmio write to update queue tail
> > > registers.
> > >
> > > Signed-off-by: Radu Nicolau <radu.nicolau at intel.com>
> > > ---
> > minor nit, I think the subject prefix on the patch should be crypto/qat.
> when it affects all qat PMDs, i.e. crypto sym, crypto asym and compression, as
> this does we usually use drivers/qat

Ah, yes, sorry, I forgot that there was more than one qat directory in
drivers. In this case, the changes are in common only, so common/qat would
seem to be the right prefix. Not that it matters to the patch
functionality, mind... :-)