[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] PoD: Check p2m assumption in debug builds
Keir Fraser wrote: Hmm... my only concern is that the assumption seems like something which might easily change, with no one realizing that they'd broken the assumption, and the codepath not being tested since most OSS users probably won't be using PoD.On 17/09/2009 02:26, "Kouya Shimura" <kouya@xxxxxxxxxxxxxx> wrote:real 0m2.343s user 0m0.040s sys 0m0.030s real 2m25.741s user 0m0.090s sys 0m0.020s I'm about to lose my patience with it. :-) Can you control it by a boot parameter? (ex. "pod_debug=nnn")Yeah, that's too slow for our normal debug build. I think we'll revert the patch -- noone would ever enable it, unless maybe if helping George track down a PoD bug. Hmm, perhaps we could wrap it with an if() statement checking if there are any outstanding PoD entries? The assumption only really matters when PoD is in use. Alternately, a separate #define for PoD debug might be useful, so we can easily turn on all these options when we need to. -George _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |