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

Re: [Xen-devel] [RFC][PATCH] walking the page lists needs the page_alloc lock



>>> On 13.08.10 at 08:46, Keir Fraser <keir.fraser@xxxxxxxxxxxxx> wrote:
> On 13/08/2010 07:40, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:
> 
>>> Sorry, my intention was to say that changes to the debug-key printouts
>>> are safe, not that they didn't require changes.
>>> 
>>> The debug-key printouts (in particular the NUMA one) are where I
>>> actually hit this bug on a running system.
>> 
>> But then, to avoid a hanging system, these should be trylock-s
>> rather than plain locks, shouldn't they?
> 
> Why? The handler is called in softirq context. It should be safe to spin.

Hmm, indeed. I was looking at others, and at least
domain_dump_evtchn_info() also uses a trylock - apparently for
no good reason.

Jan


_______________________________________________
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®.