Thread (8 messages) 8 messages, 4 authors, 2023-03-24

Re: [PATCH] crypto: p10-aes-gcm - remove duplicate include header

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2023-03-16 03:58:13
Also in: linux-crypto, lkml

Herbert Xu [off-list ref] writes:
On Tue, Mar 14, 2023 at 09:44:52PM +1100, Michael Ellerman wrote:
quoted
Hmm. Seems none of them were ever Cc'ed to linuxppc-dev. So this is the
first I've seen of them.
Sorry, I didn't know that you weren't aware of this change.  I
will be more careful with these ppc patches in future.
No worries, not your fault. My comments were mostly intended for Danny.

Although one question I do have for you is what rules, if any, do we
have for deciding whether crypto code goes in drivers/crypto vs
arch/*/crypto?

On powerpc we have some in arch/powerpc/crypto and some in
drivers/crypto/vmx, and I don't really know why it's split that way.

It seems like drivers/crypto is where non-CPU crypto accelerator drivers
go, but then it also has lots of in-CPU crypto code as well AFAICS.

I wonder if we should move drivers/crypto/vmx into arch/powerpc/crypto,
so that all the powerpc CRYPTOGAMS code is in one place. That would help
to clean up some of the duplication of perl scripts we now have.

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help