From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.web.de ([212.227.15.14]:52665 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726768AbfKKIL2 (ORCPT ); Mon, 11 Nov 2019 03:11:28 -0500 Subject: Re: [v3] s390/pkey: Use memdup_user() rather than duplicating its implementation References: <08422b7e-2071-ee52-049e-c3ac55bc67a9@web.de> <6137855bb4170c438c7436cbdb7dfd21639a8855.camel@perches.com> <833d7d5e-6ede-6bdd-a2cc-2da7f0b03908@de.ibm.com> <1b65bc81-f47a-eefa-f1f4-d5af6a1809c0@web.de> <733b29df-207e-a165-ee80-46be8720c0c4@de.ibm.com> <8f98f9fc-57df-5993-44b5-5ea4c0de7ef9@web.de> <61244676-8ac1-20af-ed94-99e19c1f95d5@web.de> <040f3e18-d97a-fc32-b237-20e7553e1733@de.ibm.com> From: Markus Elfring Message-ID: Date: Mon, 11 Nov 2019 09:11:04 +0100 MIME-Version: 1.0 In-Reply-To: <040f3e18-d97a-fc32-b237-20e7553e1733@de.ibm.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable Sender: linux-s390-owner@vger.kernel.org List-ID: To: =?UTF-8?Q?Christian_Borntr=c3=a4ger?= , linux-s390@vger.kernel.org, Harald Freudenberger , Heiko Carstens , Ingo Franzki , Vasily Gorbik , Joe Perches Cc: LKML , kernel-janitors@vger.kernel.org, Kangjie Lu , Navid Emamdoost , Stephen McCamant >> From: Markus Elfring >> Date: Fri, 8 Nov 2019 17:50:22 +0100 =E2=80=A6 >> Fixes: f2bbc96e7cfad3891b7bf9bd3e566b9b7ab4553d ("s390/pkey: add CCA AE= S cipher key support") >> Signed-off-by: Markus Elfring > > You were sending this from a different email address. Yes. > Can you use the same for the sender and the signoff? I would prefer to use the other email address (for a while). > Can you also change the subject to indicate the "fix". e.g. something li= ke > > s390/pkey: fix memory leak in error case by using memdup_user() rather t= han open coding Does this change request indicate also a need to split the software update between the discussed two function implementations? Regards, Markus