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

["Xu, Dongxiao" <dongxiao.xu@xxxxxxxxx>]

Using "/sys/bus/xen-backend/devices/" is fine, formerly I didn't notice there 
was a link in this directory. Thanks!

Best Regards, 
-- Dongxiao

-----Original Message-----
From: Tim Post [mailto:echo@xxxxxxxxxxxx] 
Sent: Tuesday, July 07, 2009 12:02 AM
To: Christoph Egger
Cc: Xu, Dongxiao; xen-devel@xxxxxxxxxxxxxxxxxxx; keir.fraser@xxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] Fix xentop on pv-ops domain0

Hi,

On Mon, 2009-07-06 at 17:49 +0200, Christoph Egger wrote:
> This patch is good.
> 

The following is going to break xenstat on almost every forward port of
2.6.18.8 (including Andy's 2.6.30 vanilla port):


+       if (strncmp(system_info.release, "2.6.18.8-xen", 
+               sizeof("2.6.18.8-xen")) == 0)
+               sysfs_vbd_path = "/sys/devices/xen-backend/";
+       else
+               sysfs_vbd_path = "/sys/devices/";
+

I would really recommend considering just
using /sys/bus/xen-backend/devices, as Stephen noted.

Cheers,
--Tim



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