[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86/IRQ: fix create_irq() after c/s 24068:6928172f7ded


  • To: Jan Beulich <JBeulich@xxxxxxxx>
  • From: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
  • Date: Fri, 04 Nov 2011 14:01:44 +0100
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 04 Nov 2011 06:07:34 -0700
  • Domainkey-signature: s=s1536a; d=ts.fujitsu.com; c=nofws; q=dns; h=X-SBRSScore:X-IronPort-AV:Received:X-IronPort-AV: Received:Received:Message-ID:Date:From:Organization: User-Agent:MIME-Version:To:CC:Subject:References: In-Reply-To:Content-Type:Content-Transfer-Encoding; b=kZrJAElWTZ1HpFZCFiCfzRjBuu4JdloUfOQEK+azcLsizFfowaDZ0YR6 qK9Nwjae4S72nSv2rRzCG0HBnZ/CJuK7y0CaSzTcxQPdOUakmMs5Yc3pk o6tMheFbRyUOxS7A32cmVHU7MwOSlhg+P0zGELOImTEdDoKclubieeltf EoxshT8sOIWE8RuSi3ep+5YnXCglXUKwJB4HGLpxKI+AI/NCHLwaoYHQH YS9RgG+mwwb3ZsvIgI0/5jx2UQlIN;
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

On 11/04/2011 12:52 PM, Jan Beulich wrote:
init_one_irq_desc() must be called with interrupts enabled (as it may
call functions from the xmalloc() group). Rather than mis-using
vector_lock to also protect the finding of an unused IRQ, make this
lockless through using cmpxchg(), and obtain the lock only around the
actual assignment of the vector.

Works for me (machine coming up again).

Juergen

--
Juergen Gross                 Principal Developer Operating Systems
PDG ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@xxxxxxxxxxxxxx
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.