[lkml]   [2011]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 0/2] Add struct crypto_alg->cra_check_optimized for crc32c_intel
    On Thu, Mar 10, 2011 at 02:05:17AM -0800, Nicholas A. Bellinger wrote:
    > We are still expecting the libcrypto consumer (iscsi_target_mod.ko) to
    > call the arch independent crypto_alloc_hash("crc32c", ...) in order to
    > have libcrypto backend logic perform a request_module() upon
    > architecture dependent offload modules (like crc32c_intel.ko) that
    > libcrypto consumers are not (and should not) be calling directly via
    > crypto_alloc_host("crc32c_intel", ...), correct..?


    > Where I am getting confused is wrt to a new crypto_alg_mod_lookup() ->
    > request_module() call for a struct shash_alg that has not yet be loaded
    > via arch/x86/crypto/crc32c-intel.c:crc32c_intel_mod_init() ->
    > crypto_register_shash().

    If you look at crypto_alg_mod_lookup, basically there are two paths.
    Either we already have a registered algorithm of the requested name,
    or we don't.

    In the first case, we won't invoke request_module and in the second
    case we will.

    So what I'm suggesting is that in the first case we also invoke
    request_module conditionally. Now exactly what that condition is
    is the tricky bit.

    The easiest is to flip a bit in the algorithm we just found. This
    isn't optimal as it'll mean that for each unregistered algorithm
    we'll end up modprobing twice, but that shouldn't be too bad I

    Email: Herbert Xu <>
    Home Page:
    PGP Key:

     \ /
      Last update: 2011-03-13 10:05    [W:0.021 / U:3.340 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site