From: Julia Lawall Subject: Re: [PATCH] crypto: Use zeroing memory allocator instead of allocator/memset Date: Sun, 31 Dec 2017 18:10:05 +0100 (CET) Message-ID: References: <1514723063-5614-1-git-send-email-himanshujha199640@gmail.com> <1514735169.1967.13.camel@perches.com> <20171231164623.GA2709@himanshu-Vostro-3559> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: Joe Perches , herbert@gondor.apana.org.au, davem@davemloft.net, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org To: Himanshu Jha Return-path: Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:7186 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750936AbdLaRKI (ORCPT ); Sun, 31 Dec 2017 12:10:08 -0500 In-Reply-To: <20171231164623.GA2709@himanshu-Vostro-3559> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Sun, 31 Dec 2017, Himanshu Jha wrote: > On Sun, Dec 31, 2017 at 07:46:09AM -0800, Joe Perches wrote: > > On Sun, 2017-12-31 at 17:54 +0530, Himanshu Jha wrote: > > > Use dma_zalloc_coherent for allocating zeroed > > > memory and remove unnecessary memset function. > > > > > > Done using Coccinelle. > > > Generated-by: scripts/coccinelle/api/alloc/kzalloc-simple.cocci > > > > I thought you were going to change this tag to > > not use the "-by:" form and use something else > > like "generated-using:" or "created-with:". > > Thanks for pointing that out! > > I was going to ask this to Julia for the appropriate tag, but it slipped > my mind. In future I will definitely choose more appropriate tag > on the Coccinelle patches. > > Just to give an insight of the problem to everyone : > > While running checkpatch on patches involving the above Generated-by > tag, which is generally used to specify the cocci script used, > checkpatch prompts the below warning and error : > > WARNING: Non-standard signature: Generated-by: > #10: > Generated-by: scripts/coccinelle/api/alloc/kzalloc-simple.cocci > > ERROR: Unrecognized email address: > 'scripts/coccinelle/api/alloc/kzalloc-simple.cocci' > > Joe suggested to not use the "-by:" form since it should be reserved > for actual people's names and email addresses. > And if you use other tag than the "-by:" tag, then there is no > error/warning produced. kbuild uses eg Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci julia > > Thanks > Himanshu Jha >