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

[Xen-devel] [PATCH] Make xentrace hypercall a bit more sane


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: "George Dunlap " <dunlapg@xxxxxxxxx>
  • Date: Thu, 1 Jun 2006 16:17:03 -0400
  • Delivery-date: Thu, 01 Jun 2006 13:17:31 -0700
  • Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:sender:to:subject:mime-version:content-type:x-google-sender-auth; b=nWMiX++RskbOvea3bETMkWcOX5i/VJnYBj6Z/cqQsCF8hFD9aCLIVF+bDifz7pxzW7DBYHOVO3kgZYQPhEKUryeuOsUYc0TLKIymQ46mAKUj447p2eeCppw4YBlkz3UI3pmapK2x8Q3hCqNQ8sWjgwcak5ObEfjLkzh7dZ4+pXg=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

xentrace, setsize and the hypervisor don't seem to be on the same
page.  If tracing is not enabled, the hypercall will only allow
TBUF_SET_SIZE and TBUF_ENABLE.  However, set_size calls TBUF_GET_INFO,
and xentrace wants to call TBUF_SET_EVT_MASK and TBUF_SET_CPU_MASK
before calling TBUF_ENABLE.

I don't see any reason not to call SET_*_MASK and GET_INFO if tracing
is not enabled.    If the buffer hasn't been allocated yet, GET_INFO
should just return something invalid in the mfn field.  Anyone calling
should check this value.

Comments?
-George

Attachment: xentrace.patch
Description: Text Data

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