From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933215Ab2HPTrR (ORCPT ); Thu, 16 Aug 2012 15:47:17 -0400 Received: from e4.ny.us.ibm.com ([32.97.182.144]:52357 "EHLO e4.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932285Ab2HPTrP (ORCPT ); Thu, 16 Aug 2012 15:47:15 -0400 Date: Thu, 16 Aug 2012 12:46:51 -0700 From: "Paul E. McKenney" To: Stephen Rothwell Cc: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Peter Zijlstra , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Rusty Russell , Namhyung Kim , "Srivatsa S. Bhat" Subject: Re: linux-next: build failure after merge of the tip tree Message-ID: <20120816194651.GK2445@linux.vnet.ibm.com> Reply-To: paulmck@linux.vnet.ibm.com References: <20120816131236.13013e1ec18b1f1074f6dc92@canb.auug.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120816131236.13013e1ec18b1f1074f6dc92@canb.auug.org.au> User-Agent: Mutt/1.5.21 (2010-09-15) X-Content-Scanned: Fidelis XPS MAILER x-cbid: 12081619-3534-0000-0000-00000B8E2378 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 16, 2012 at 01:12:36PM +1000, Stephen Rothwell wrote: > Hi all, > > After merging the tip tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > drivers/infiniband/hw/ehca/ehca_irq.c: In function 'find_next_online_cpu': > drivers/infiniband/hw/ehca/ehca_irq.c:672:2: error: expected ';' before 'spin_unlock_irqrestore' > > Caused by commit 81942621bd6b ("infiniband: Ehca: Use hotplug thread > infrastructure"). > > I have used the tip tree from next-20120814 for today. My first reaction was "we tested that!!!", but I see the conversion from while(1) to do-while. Does the following cure it? Thanx, Paul ------------------------------------------------------------------------ infiniband: ehca: Fix while->do-while conversion typo This commit just adds a needed semicolon. Signed-off-by: Paul E. McKenney Signed-off-by: Paul E. McKenney diff --git a/drivers/infiniband/hw/ehca/ehca_irq.c b/drivers/infiniband/hw/ehca/ehca_irq.c index 4eeac40..83a0095 100644 --- a/drivers/infiniband/hw/ehca/ehca_irq.c +++ b/drivers/infiniband/hw/ehca/ehca_irq.c @@ -668,7 +668,7 @@ static int find_next_online_cpu(struct ehca_comp_pool *pool) if (cpu >= nr_cpu_ids) cpu = cpumask_first(cpu_online_mask); pool->last_cpu = cpu; - } while (!per_cpu_ptr(pool->cpu_comp_tasks, cpu)->active) + } while (!per_cpu_ptr(pool->cpu_comp_tasks, cpu)->active); spin_unlock_irqrestore(&pool->last_cpu_lock, flags); return cpu;