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

Re: [Xen-devel] [PATCH RFC 1/7] xen: Relocate mem_access and mem_event into common.

>>> On 26.08.14 at 12:52, <tamas.lengyel@xxxxxxxxxxxx> wrote:
> On Mon, Aug 25, 2014 at 7:19 PM, Andres Lagar Cavilla <
> andres@xxxxxxxxxxxxxxxx> wrote:
>> On Fri, Aug 22, 2014 at 2:30 AM, Tamas K Lengyel <
>>> -        req->flags |= MEM_EVENT_FLAG_FOREIGN;
>>> -        ASSERT( !(req->flags & MEM_EVENT_FLAG_VCPU_PAUSED) );
>> Take the opportunity to downgrade the aggressiveness of this at some point
>> in this series. (I'd prefer to keep code motion patches as purely code
>> motion).
>> A faulty tool stack can brick a debug hypervisor. Unpleasant while
>> dev/test.
> I'm a little bit hazy in what situations this could arise and what it is
> trying to protect against. I could wrap the condition into an unlikely()
> and have a debug message printed rather than bricking the VMM with ASSERT()
> and/or enable the ASSERT only when we are building with debug=y.

ASSERT()s are enabled only when debug=y (i.e. NDEBUG not defined
at the C level).


Xen-devel mailing list



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