[Xen-devel] [xenstat] Use backend path which is compatible with pvops -- status request

[Daniel Schroeder <sec@xxxxxxxxxxxxxxx>]

Ian Campbell wrote:
> On Thu, 2009-05-07 at 14:47 -0400, Valtteri Kiviniemi wrote:
>> Hi,
>>
>> Yeah, so it seems to be:
> 
> The switch to/from curses has hidden most of the interesting stuff
> (including a useful error message :-(), if you redirect to a file you
> get:
> open("/sys/devices/xen-backend/", 
> O_RDONLY|O_NONBLOCK|O_LARGEFILE|O_DIRECTORY) = -1 ENOENT (No such file or 
> directory)
> dup(2)                                  = 6
> fcntl64(6, F_GETFL)                     = 0x8001 (flags O_WRONLY|O_LARGEFILE)
> close(6)                                = 0
> write(2, "Error opening /sys/devices/xen-b"..., 67Error opening 
> /sys/devices/xen-backend/: No such file or directory
> ) = 67
> 
> This path doesn't exist in the pvops kernel. Looks
> like /sys/bus/xen-backend/devices is just as good and exists in both
> pvops and 2.6.18 kernels.
> 
> Subject: xenstat: Use backend path which is compatible with pvops and 
> 2.6.18-xen kernels.
> 
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> 
> diff -r 07812e857e67 tools/xenstat/libxenstat/src/xenstat_linux.c
> --- a/tools/xenstat/libxenstat/src/xenstat_linux.c    Thu Apr 09 12:09:14 
> 2009 +0100
> +++ b/tools/xenstat/libxenstat/src/xenstat_linux.c    Fri May 08 09:40:45 
> 2009 +0100
> @@ -31,7 +31,7 @@
>  
>  #include "xenstat_priv.h"
>  
> -#define SYSFS_VBD_PATH "/sys/devices/xen-backend/"
> +#define SYSFS_VBD_PATH "/sys/bus/xen-backend/devices"
>  
>  struct priv_data {
>       FILE *procnetdev;
> 
> 

ping
whats the status of this patch? Will it hit xen-unstable repo anytime soon?

daniel

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