On Mon, Jul 13, 2020 at 01:52:24PM +0100, Trahe, Fiona wrote: > > > > -----Original Message----- > > From: Bruce Richardson <bruce.richard...@intel.com> > > Sent: Monday, July 13, 2020 1:45 PM > > To: Nicolau, Radu <radu.nico...@intel.com> > > Cc: dev@dpdk.org; Xing, Beilei <beilei.x...@intel.com>; Guo, Jia > > <jia....@intel.com>; Ananyev, Konstantin > > <konstantin.anan...@intel.com>; jerinjac...@gmail.com; > > david.march...@redhat.com; Trahe, Fiona > > <fiona.tr...@intel.com>; Zhao1, Wei <wei.zh...@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.nico...@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... :-)