Re: [PATCH] crypto: ux500/hash: Add empty export and import

From: Kamil Konieczny
Date: Thu Jan 18 2018 - 07:53:01 EST




On 18.01.2018 11:06, Linus Walleij wrote:
> On Tue, Jan 16, 2018 at 5:32 PM, Kamil Konieczny
> <k.konieczny@xxxxxxxxxxxxxxxxxxx> wrote:
>
>> Crypto framework will require async hash export/import, so add empty
>> functions to prevent OOPS.
>>
>> Signed-off-by: Kamil Konieczny <k.konieczny@xxxxxxxxxxxxxxxxxxx>
>
> Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>
> But why isn't the framework code just checking the vtable for NULL?
>
> if (foo->fp)
> foo->fp(bar);

This will be inefficient,
it should be checked once at ahash alg register,
or with the old method by using wrapper

--
Best regards,
Kamil Konieczny
Samsung R&D Institute Poland