Re: [PATCH 0/2] Add struct crypto_alg->cra_check_optimized forcrc32c_intel

From: Nicholas A. Bellinger
Date: Thu Mar 10 2011 - 04:01:16 EST


On Thu, 2011-03-10 at 16:43 +0800, Herbert Xu wrote:
> On Thu, Mar 10, 2011 at 12:21:10AM -0800, Nicholas A. Bellinger wrote:
> >
> > Nicholas Bellinger (2):
> > crypto: Add struct crypto_alg->cra_check_optimized
> > crypto/crc32c: Add crc32c_cra_check_optimized for crc32c_intel
>
> I think you misunderstood my suggestion. The reason I asked
> for this to be done in the crypto API is because it is not specific
> to crc32c_intel.
>
> So sending me a patch that modifies crc32c clearly misses the
> point :)
>
> This should be done in the core API, i.e., api.c/algapi.c.
>

OK, so you mean each struct crypto_alg should define something like a
'cra_optimized_name' for which request_module(alg->cra_optimized_name)
is called somewhere in libcrypto code..?

Would you mind being a bit more specific where this should be happening
in api.c/algapi.c..?

Thanks,

--nab

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/