Re: [Xen-devel] [PATCH ARM v5 15/20] mini-os: enable test_xenbus again

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2014-06-27 at 13:48 +0100, Thomas Leonard wrote:
> On 27 June 2014 12:05, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> > On Thu, 2014-06-26 at 12:28 +0100, Thomas Leonard wrote:
> >> The old code said "Xenbus tests disabled, because of a Xend bug".
> >> Since xend doesn't exist any longer, we can enable these again.
> >
> > Out of curiosity, do you know if they work now? (I don't think the
> > answer should block this patch either way).
> 
> On ARM, with all the patches I sent applied, the output is:

Which I suppose looks like success despite the messages towards the end.
Anywya, thanks.

> 
> [...]
> Doing xenbus test.
> Periodic thread started.
> T(s=0 us=1253)
> Got a reply, type 16, id 1, len 7.
> Doing ls test.
> ls device...
> ls device[0] -> suspend
> ls device[1] -> vbd
> ls device[2] -> vif
> ls device/vif...
> ls device/vif[0] -> 0
> ls device/vif/0...
> ls device/vif/0[0] -> backend
> ls device/vif/0[1] -> backend-id
> ls device/vif/0[2] -> state
> ls device/vif/0[3] -> handle
> ls device/vif/0[4] -> mac
> Doing read test.
> Read device/vif/0/mac...
> Read device/vif/0/mac -> 00:16:3e:7d:69:0d.
> Read device/vif/0/backend...
> Read device/vif/0/backend -> /local/domain/0/backend/vif/1/0.
> Doing write test.
> Write flobble to device/vif/0/flibble...
> Success.
> Read device/vif/0/flibble...
> Read device/vif/0/flibble -> flobble.
> Write widget to device/vif/0/flibble...
> Success.
> Read device/vif/0/flibble...
> Read device/vif/0/flibble -> widget.
> Doing rm test.
> rm device/vif/0/flibble...
> Success.
> Read device/vif/0/flibble...
> Error in xenbus read: ENOENT
> (Should have said ENOENT)
> Thread "xenbus_tester" exited.
> T(s=1 us=1543)
> [...]
> 
> >> Signed-off-by: Thomas Leonard <talex5@xxxxxxxxx>
> >> ---
> >>  extras/mini-os/test.c | 3 +--
> >>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>
> >> diff --git a/extras/mini-os/test.c b/extras/mini-os/test.c
> >> index bbfc613..20d372b 100644
> >> --- a/extras/mini-os/test.c
> >> +++ b/extras/mini-os/test.c
> >> @@ -56,8 +56,7 @@ void test_xenbus(void);
> >>
> >>  static void xenbus_tester(void *p)
> >>  {
> >> -    printk("Xenbus tests disabled, because of a Xend bug.\n");
> >> -    /* test_xenbus(); */
> >> +    test_xenbus();
> >>  }
> >>  #endif
> >>
> >
> >
> 
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel