Re: crypto API and IBM z990 hardware support

From: Arnd Bergmann (
Date: Wed Jul 02 2003 - 17:06:21 EST

James Morris wrote:

> On Wed, 2 Jul 2003, Thomas Spatzier wrote:
> I'd like to avoid these kind of macros, and make it a general case
> solution (e.g. which can be used for various hardware implementations).

> One possibility would be to allow registration with an alias list in
> crypto API with attributes indicating whether the module is hardware,
> arch-specific etc.

Still too complicated imho. The built-in code should not need to know
about which modules exist. For example that would prevent autoloading
of third-party hardware crypto drivers. In most cases, a
first-come-first-serve approach is sufficient and works without
changes to the current code. Modules implementing the same algorithm
can just set .cra_name to the same value.

For built-in drivers, link order decides which implementation is
preferred. Consequently, hardware crypto drivers need to come before
software implementations and must not register themselves if the
hardware is not found at initcall time.

For the module case, the aes-z990.o module could declare
'MODULE_ALIAS(aes-hw);', the simple patch below makes sure
that any aes-hw module is preferred to the software aes
module. If there is more than one hardware implementation
available for an architecture, either the autoloader can be
extended further, or modprobe has to be configured

        Arnd <><

===== crypto/autoload.c 1.7 vs edited =====
--- 1.7/crypto/autoload.c Sat May 17 21:39:13 2003
+++ edited/crypto/autoload.c Wed Jul 2 23:48:10 2003
@@ -21,16 +21,15 @@
  * A far more intelligent version of this is planned. For now, just
  * try an exact match on the name of the algorithm.
-void crypto_alg_autoload(const char *name)
- request_module("%s", name);
 struct crypto_alg *crypto_alg_mod_lookup(const char *name)
        struct crypto_alg *alg = crypto_alg_lookup(name);
        if (alg == NULL) {
- crypto_alg_autoload(name);
+ request_module("%s-hw", name);
+ alg = crypto_alg_lookup(name);
+ }
+ if (alg == NULL) {
+ request_module("%s", name);
                alg = crypto_alg_lookup(name);
        return alg;

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

This archive was generated by hypermail 2b29 : Mon Jul 07 2003 - 22:00:17 EST