On Mon, Jul 13, 2020 at 01:52:24PM +0100, Trahe, Fiona wrote: > > > > -----Original Message----- > > From: Bruce Richardson <[email protected]> > > Sent: Monday, July 13, 2020 1:45 PM > > To: Nicolau, Radu <[email protected]> > > Cc: [email protected]; Xing, Beilei <[email protected]>; Guo, Jia > > <[email protected]>; Ananyev, Konstantin > > <[email protected]>; [email protected]; > > [email protected]; Trahe, Fiona > > <[email protected]>; Zhao1, Wei <[email protected]> > > 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 <[email protected]> > > > --- > > 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... :-)
