On Mon, Jul 31, 2017 at 02:04:58PM +0200, Stephan Müller wrote: > Hi, > > with the update of the algif_aead and algif_skcipher memory management, > a lot of code duplication has been introduced deliberately. > > This patch set cleans up the low-hanging fruits. The cleanup of the > recvmsg RX SGL copy will come separately as this is not a simple copy > and paste operation. > > Each patch was tested individually with libkcapi's test suite. > > The patch set goes on top of patches "crypto: AF_ALG - return error code when > no data was processed" and "crypto: algif_aead - copy AAD from src to dst". > > Stephan Mueller (16): > crypto: AF_ALG - consolidation of common data structures > crypto: AF_ALG - consolidation of context data structure > crypto: AF_ALG - consolidate send buffer service functions > crypto: AF_ALG - consolidate RX buffer service functions > crypto: AF_ALG - consolidate TX SGL allocation > crypto: AF_ALG - consolidate counting TX SG entries > crypto: AF_ALG - consolidate counting TX SG entries > crypto: AF_ALG - consolidate freeing TX/RX SGLs > crypto: AF_ALG - consolidate waiting for wmem > crypto: AF_ALG - consolidate waking up on writable memory > crypto: AF_ALG - consolidate waiting for TX data > crypto: AF_ALG - consolidate waking up caller for TX data > crypto: AF_ALG - consolidate sendmsg implementation > crypto: AF_ALG - consolidate sendpage implementation > crypto: AF_ALG - consolidate AIO callback handler > crypto: AF_ALG - consolidate poll syscall handler This split makes no sense. I don't see why you can't merge them into a single patch if it's just rearranging the code. Cheers, -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt