From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH] xen: clamp bitmaps to correct number of bits Date: Thu, 06 Sep 2012 16:51:43 +0100 Message-ID: References: <5048D39D0200007800099684@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5048D39D0200007800099684@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich , Ian Campbell Cc: "Tim (Xen.org)" , Dario Faggioli , "Keir (Xen.org)" , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On 06/09/2012 15:47, "Jan Beulich" wrote: >> I'll fold them in, it was convenient to have variables while I was >> printk'ing what I was doing but not any more. > > I won't ask you for another round because of this, but you > still left the parameter and remaining local variable as plain > int, nor did you insert whitespace into the expressions. If I > were the one to commit this, I would do the adjustment while > committing... > > Anyway, as long as there's no easily visible tools side bug > addressed by this, I would think we should rather leave this > for after branching - Keir? Yes, we'll leave it for post 4.2.0.