Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935350AbcLOGiM (ORCPT ); Thu, 15 Dec 2016 01:38:12 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:33633 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933125AbcLOGiK (ORCPT ); Thu, 15 Dec 2016 01:38:10 -0500 MIME-Version: 1.0 In-Reply-To: <1480545633-2675-1-git-send-email-linux@rasmusvillemoes.dk> References: <1480545633-2675-1-git-send-email-linux@rasmusvillemoes.dk> From: Steve French Date: Thu, 15 Dec 2016 00:37:48 -0600 Message-ID: Subject: Re: [PATCH] cifs: use %16phN for formatting md5 sum To: Rasmus Villemoes Cc: Steve French , "linux-cifs@vger.kernel.org" , samba-technical , LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1534 Lines: 47 merged into cifs-2.6.git for-next On Wed, Nov 30, 2016 at 4:40 PM, Rasmus Villemoes wrote: > Passing a gazillion arguments takes a lot of code: > > add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-253 (-253) > > Signed-off-by: Rasmus Villemoes > --- > fs/cifs/link.c | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-) > > diff --git a/fs/cifs/link.c b/fs/cifs/link.c > index d031af8d3d4d..c4d996f78e1c 100644 > --- a/fs/cifs/link.c > +++ b/fs/cifs/link.c > @@ -45,13 +45,8 @@ > (CIFS_MF_SYMLINK_LINK_OFFSET + CIFS_MF_SYMLINK_LINK_MAXLEN) > > #define CIFS_MF_SYMLINK_LEN_FORMAT "XSym\n%04u\n" > -#define CIFS_MF_SYMLINK_MD5_FORMAT \ > - "%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x\n" > -#define CIFS_MF_SYMLINK_MD5_ARGS(md5_hash) \ > - md5_hash[0], md5_hash[1], md5_hash[2], md5_hash[3], \ > - md5_hash[4], md5_hash[5], md5_hash[6], md5_hash[7], \ > - md5_hash[8], md5_hash[9], md5_hash[10], md5_hash[11],\ > - md5_hash[12], md5_hash[13], md5_hash[14], md5_hash[15] > +#define CIFS_MF_SYMLINK_MD5_FORMAT "%16phN\n" > +#define CIFS_MF_SYMLINK_MD5_ARGS(md5_hash) md5_hash > > static int > symlink_hash(unsigned int link_len, const char *link_str, u8 *md5_hash) > -- > 2.1.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-cifs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks, Steve