From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from kirsty.vergenet.net ([202.4.237.240]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UBEpL-0001lU-RD for kexec@lists.infradead.org; Fri, 01 Mar 2013 01:32:01 +0000 Date: Fri, 1 Mar 2013 10:31:56 +0900 From: Simon Horman Subject: Re: [PATCH v3 1/2] kexec: fix wrong types of some local variables Message-ID: <20130301013156.GA11434@verge.net.au> References: <512C44E4.70907@cn.fujitsu.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <512C44E4.70907@cn.fujitsu.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: Zhang Yanfei Cc: Andrew Morton , "kexec@lists.infradead.org" , "Eric W. Biederman" , "linux-kernel@vger.kernel.org" On Tue, Feb 26, 2013 at 01:15:16PM +0800, Zhang Yanfei wrote: > The types of the following local variables: > - ubytes/mbytes in kimage_load_crash_segment()/kimage_load_normal_segment() > - r in vmcoreinfo_append_str() > are wrong, so fix them. > > Cc: "Eric W. Biederman" > Cc: Andrew Morton > Cc: Simon Horman > Signed-off-by: Zhang Yanfei No complaints here. Reviewed-by: Simon Horman > --- > kernel/kexec.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/kernel/kexec.c b/kernel/kexec.c > index 2436ffc..3cbfcc7 100644 > --- a/kernel/kexec.c > +++ b/kernel/kexec.c > @@ -789,7 +789,7 @@ static int kimage_load_normal_segment(struct kimage *image, > struct kexec_segment *segment) > { > unsigned long maddr; > - unsigned long ubytes, mbytes; > + size_t ubytes, mbytes; > int result; > unsigned char __user *buf; > > @@ -853,7 +853,7 @@ static int kimage_load_crash_segment(struct kimage *image, > * We do things a page at a time for the sake of kmap. > */ > unsigned long maddr; > - unsigned long ubytes, mbytes; > + size_t ubytes, mbytes; > int result; > unsigned char __user *buf; > > @@ -1455,7 +1455,7 @@ void vmcoreinfo_append_str(const char *fmt, ...) > { > va_list args; > char buf[0x50]; > - int r; > + size_t r; > > va_start(args, fmt); > r = vsnprintf(buf, sizeof(buf), fmt, args); > -- > 1.7.1 > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec