Re: Can crypto API provide information about hw acceleration?

From: David Sterba
Date: Wed May 15 2019 - 08:55:55 EST


On Tue, May 14, 2019 at 02:34:10PM -0700, Eric Biggers wrote:
> On Tue, May 14, 2019 at 06:33:48PM +0200, David Sterba wrote:
> > Hi,
> >
> > Q: is there a way to query the crypto layer whether a given algorithm
> > (digest, crypto) is accelerated by the driver?
> >
> > This information can be used to decide if eg. a checksum should can be
> > calculated right away or offloaded to a thread. This is done in btrfs,
> > (fs/btrfs/disk-io.c:check_async_write).
> >
> > At this moment it contains a static check for a cpu feature, and only
> > for x86. I briefly searched the arch/ directory for implementations of
> > crc32c that possibly use hw aid and there are several of them. Adding a
> > static check a-la x86 for the other architectures (arm, ppc, mips,
> > sparc, s390) is wrong, so I'm looking for a clean solution.
> >
> > The struct shash_alg definition of the algorithms does not say anything
> > about the acceleration. The closest thing is the cra_priority, but I
> > don't know if this is reliable information. The default implementations
> > seem to have 100, and acceleated 200 or 300.
> >
> > This would be probably sufficient, but I'd like a confirmation from
> > crypto people.
> >
>
> There's only one default implementation of crc32c, not multiple, and it has
> priority 100. All other crc32c implementations have priority > 100. So yes,
> you can check the priority (which would require adding a function to
> lib/libcrc32c.c to get it). Alternatively you could check whether the driver
> name is "crc32c-generic" or not.

Thanks, the driver name check seems to be ok for my needs. At mount time
the struct crypto_shash is initialized and this provides the driver
name, then a bit is set whether it's generic or not and later used to
decide whether to offload.