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

[Xen-devel] RE: [PATCH] Fix xentop on pv-ops domain0


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: Steven Maresca <steve.maresca@xxxxxxxxx>
  • Date: Mon, 6 Jul 2009 10:50:40 -0400
  • Cc: dongxiao.xu@xxxxxxxxx, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, echo@xxxxxxxxxxxx
  • Delivery-date: Mon, 06 Jul 2009 08:17:26 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:cc:content-type:content-transfer-encoding; b=RoCa+tHrzkuj5f8M/wY2/sXeDkIf+9QknUw3Pu+XRuamQM/um5alHQUrgRet82E1/0 CF5tBj175R08Se05vpzzUerJvcrAdZr6ts0lyX3tyiCcQOUPGFl39lB2WA/FVAPnSnEP 7XhXXRZJZXO/1qcDLsvHBio1taHqVvZTcmhqo=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

Hello,

>Xentop encounters error in pv-ops domain0, because the VBD path in sysfs 
>changes to "/sys/devices"
>Also delete the macro in xenstat_netbsd.c, because it is not used.
>
>Signed-off-by: Dongxiao Xu <dongxiao.xu@xxxxxxxxx>
>
>Best Regards,
>-- Dongxiao

The patch you've provided explicitly checks for a .18-xen kernel;
doing so runs the risk of breaking behavior on every forward port
after .18.   This is quite fragile.

It is my suggestion that, instead, we might utilize the patch Ian
Campbell suggested some time ago. See
http://lists.xensource.com/archives/html/xen-devel/2009-05/msg00299.html

That patch is a small tweak: #define SYSFS_VBD_PATH
"/sys/bus/xen-backend/devices"
Luckily, this change satisfies legacy .18-xen, pvops, (and even older
.16-xen kernels),

Xenstat - while useful - is brittle enough as it is, so keeping things
generic would be best.

-Steve Maresca

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