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

[Xen-devel] [PATCH 0 of 4] xentrace [v2]: non-contiguous allocation of p

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH 0 of 4] xentrace [v2]: non-contiguous allocation of per-cpu buffer
From: Olaf Hering <olaf@xxxxxxxxx>
Date: Tue, 10 May 2011 16:32:35 +0200
Cc: George Dunlap <george.dunlap@xxxxxxxxxx>
Delivery-date: Tue, 10 May 2011 07:34:31 -0700
Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; t=1305037956; l=731; s=domk; d=aepfle.de; h=Cc:To:From:Date:Subject:Content-Transfer-Encoding:MIME-Version: Content-Type:X-RZG-CLASS-ID:X-RZG-AUTH; bh=+AKrzWeGYMgPJasd2Zu5iGBQ1KU=; b=m0iQjcHmqO5GngkRRW+He4I+yAjRBrulRpSM2LWG+yJ7VyOFsk3dg35wvcQp1iJmRpr iGdRHZsOqoMpb2HQs4d0VnGqPN7RmL/jqSTrhoAA9IgEI2QLGf2VJg5m84Ve6a6BGLyqK rMdy8VJgIF83kMc+fN3udmg0t/fwWJU3gIU=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mercurial-patchbomb/1.8.1
This series implements non-contiguous trace buffers.

There was an overflow in the offset calculation because it was stored in
an int. Also the calculation of the per-cpu buffer did not take the type
of the offset value into account. This was the reason why I ran into the
checks in bogus(). The crash in out_dealloc: was caused by the lack of
check for the offset value.

Regarding the math added to next_record() another array of pointers could be
added to remove the mfn_to_virt() calls. In my testing 9343 trace pages on 8
cpus are used. Such an array would required additional (9343*8*8)/1024=583Kb.
Which is not too much, given the amount of 291MB tracebuffers.
Such a change is not part of this series.

Olaf

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