Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1756262AbcCUOOS (ORCPT <rfc822;w@1wt.eu>);
	Mon, 21 Mar 2016 10:14:18 -0400
Received: from smtp02.citrix.com ([66.165.176.63]:3948 "EHLO SMTP02.CITRIX.COM"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1755215AbcCUOON (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Mon, 21 Mar 2016 10:14:13 -0400
X-IronPort-AV: E=Sophos;i="5.24,372,1454976000"; 
   d="scan'208";a="347399896"
Subject: Re: [Xen-devel] [PATCH v2 5/6] virt, sched: add cpu pinning to
 smp_call_sync_on_phys_cpu()
To: Jan Beulich <JBeulich@suse.com>, Juergen Gross <JGross@suse.com>
References: <1458563075-30298-1-git-send-email-jgross@suse.com>
 <1458563075-30298-6-git-send-email-jgross@suse.com>
 <56F0087202000078000DED9F@prv-mh.provo.novell.com>
CC: <Douglas_Warzecha@dell.com>, <pali.rohar@gmail.com>, <jeremy@goop.org>,
        <peterz@infradead.org>, <x86@kernel.org>, <tglx@linutronix.de>,
        <virtualization@lists.linux-foundation.org>,
        <xen-devel@lists.xenproject.org>, <boris.ostrovsky@oracle.com>,
        <mingo@redhat.com>, <linux@roeck-us.net>, <rusty@rustcorp.com.au>,
        <chrisw@sous-sol.org>, Jean Delvare <jdelvare@suse.com>,
        <linux-kernel@vger.kernel.org>, <akataria@vmware.com>, <hpa@zytor.com>
From: David Vrabel <david.vrabel@citrix.com>
X-Enigmail-Draft-Status: N1110
Message-ID: <56F001AB.2000600@citrix.com>
Date: Mon, 21 Mar 2016 14:14:03 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Icedove/38.5.0
MIME-Version: 1.0
In-Reply-To: <56F0087202000078000DED9F@prv-mh.provo.novell.com>
Content-Type: text/plain; charset="windows-1252"
Content-Transfer-Encoding: 7bit
X-DLP: MIA2
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 164
Lines: 7

On 21/03/16 13:42, Jan Beulich wrote:
>
> Also don't you need to call smp_processor_id() after preempt_disable()?

I suggest using get_cpu()/put_cpu() here.

David