WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Fix non-debug build after c/s 23767:80e9fcdaef36

To: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Fix non-debug build after c/s 23767:80e9fcdaef36
From: Tim Deegan <tim@xxxxxxx>
Date: Mon, 22 Aug 2011 15:07:33 +0100
Cc: Keir Fraser <keir.xen@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 22 Aug 2011 07:08:38 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CAFLBxZYgmWu-w9PXRFL5NY3gb1ahS5gbz4mNcBENsoV6KxamSQ@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <ac4de6b4c457d27c49c7.1314019381@elijah> <CA781999.1F93E%keir.xen@xxxxxxxxx> <CAFLBxZYgmWu-w9PXRFL5NY3gb1ahS5gbz4mNcBENsoV6KxamSQ@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
At 15:02 +0100 on 22 Aug (1314025327), George Dunlap wrote:
> That seems to compile just fine, and obviously makes the code a lot
> cleaner.  The only thing is that this will still cause functions
> inside ASSERTS (spin_is_locked() in this case) to be called; I thought
> part of the reason for having ASSERTs in the debug build only was to
> reduce the cost of all the checks (the other reason to avoid
> unnecessary crashes on production builds)?

How about #define ASSERT(p) do { if (0 && (p)); } while (0) ?

Tim.

> On Mon, Aug 22, 2011 at 2:35 PM, Keir Fraser <keir.xen@xxxxxxxxx> wrote:
> > George,
> >
> > Would something like this work more generically for the non-debug case?
> >
> > #define ASSERT(p) do { if (p); } while (0)
> >
> >  -- Keir
> >

-- 
Tim Deegan <tim@xxxxxxx>
Principal Software Engineer, Xen Platform Team
Citrix Systems UK Ltd.  (Company #02937203, SL9 0BG)

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