From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756558AbaFYM3B (ORCPT ); Wed, 25 Jun 2014 08:29:01 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:37930 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752609AbaFYM27 (ORCPT ); Wed, 25 Jun 2014 08:28:59 -0400 Date: Wed, 25 Jun 2014 05:28:56 -0700 From: Christoph Hellwig To: Hannes Reinecke Cc: James Bottomley , Christoph Hellwig , Ewan Milne , linux-scsi@vger.kernel.org, Rusty Russell , linux-kernel@vger.kernel.org Subject: Re: [PATCH 5/6] scsi: use 64-bit value for 'max_luns' Message-ID: <20140625122856.GA3221@infradead.org> References: <1401785937-43581-1-git-send-email-hare@suse.de> <1401785937-43581-6-git-send-email-hare@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1401785937-43581-6-git-send-email-hare@suse.de> User-Agent: Mutt/1.5.23 (2014-03-12) X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 03, 2014 at 10:58:56AM +0200, Hannes Reinecke wrote: > Now that we're using 64-bit LUNs internally we need to increase > the size of max_luns to 64 bits, too. > > Signed-off-by: Hannes Reinecke > Reviewed-by: Christoph Hellwig > Reviewed-by: Ewan Milne I just noticed that this has changes to the module param code. These should be split into a separate patch and be ACKed by the modules maintainer. I'd still love to take the change through the SCSI tree to be able to get this into 3.17 easily. moduleparam changes below: > diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h > index 204a677..21e2ba6 100644 > --- a/include/linux/moduleparam.h > +++ b/include/linux/moduleparam.h > @@ -381,6 +381,11 @@ extern int param_set_ulong(const char *val, const struct kernel_param *kp); > extern int param_get_ulong(char *buffer, const struct kernel_param *kp); > #define param_check_ulong(name, p) __param_check(name, p, unsigned long) > > +extern struct kernel_param_ops param_ops_ullong; > +extern int param_set_ullong(const char *val, const struct kernel_param *kp); > +extern int param_get_ullong(char *buffer, const struct kernel_param *kp); > +#define param_check_ullong(name, p) __param_check(name, p, unsigned long long) > + > extern struct kernel_param_ops param_ops_charp; > extern int param_set_charp(const char *val, const struct kernel_param *kp); > extern int param_get_charp(char *buffer, const struct kernel_param *kp); > diff --git a/kernel/params.c b/kernel/params.c > index b00142e..2b2a9dd 100644 > --- a/kernel/params.c > +++ b/kernel/params.c > @@ -253,6 +253,7 @@ STANDARD_PARAM_DEF(int, int, "%i", kstrtoint); > STANDARD_PARAM_DEF(uint, unsigned int, "%u", kstrtouint); > STANDARD_PARAM_DEF(long, long, "%li", kstrtol); > STANDARD_PARAM_DEF(ulong, unsigned long, "%lu", kstrtoul); > +STANDARD_PARAM_DEF(ullong, unsigned long long, "%llu", kstrtoull); > > int param_set_charp(const char *val, const struct kernel_param *kp) > { > -- > 1.7.12.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ---end quoted text---