[rumpuserxen test] 34129: regressions - FAIL

[rumpuserxen test] 34129: regressions - FAIL

[xen.org]

flight 34129 rumpuserxen real [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/34129/

Regressions :-(

Tests which did not succeed and are blocking,
including tests which could not be run:
 build-amd64-rumpuserxen       6 xen-build                 fail REGR. vs. 33866
 build-i386-rumpuserxen        6 xen-build                 fail REGR. vs. 33866

Tests which did not succeed, but are not blocking:
 test-amd64-amd64-rumpuserxen-amd64  1 build-check(1)               blocked n/a
 test-amd64-i386-rumpuserxen-i386  1 build-check(1)               blocked  n/a

version targeted for testing:
 rumpuserxen          05e06b0fe52918d6575e33b7d7551d85c93f7aff
baseline version:
 rumpuserxen          30d72f3fc5e35cd53afd82c8179cc0e0b11146ad

------------------------------------------------------------
People who touched revisions under test:
  Martin Lucina <martin@lucina.net>
------------------------------------------------------------

jobs:
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 build-amd64-rumpuserxen                                      fail    
 build-i386-rumpuserxen                                       fail    
 test-amd64-amd64-rumpuserxen-amd64                           blocked 
 test-amd64-i386-rumpuserxen-i386                             blocked 


------------------------------------------------------------
sg-report-flight on osstest.cam.xci-test.com
logs: /home/xc_osstest/logs
images: /home/xc_osstest/images

Logs, config files, etc. are available at
    http://www.chiark.greenend.org.uk/~xensrcts/logs

Test harness code can be found at
    http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary


Not pushing.

------------------------------------------------------------
commit 05e06b0fe52918d6575e33b7d7551d85c93f7aff
Author: Martin Lucina <martin@lucina.net>
Date:   Mon Feb 2 18:01:52 2015 +0100

    Sync Travis CI configuration with app-tools rename
    
    Signed-off-by: Martin Lucina <martin@lucina.net>

commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c
Author: Martin Lucina <martin@lucina.net>
Date:   Mon Feb 2 18:00:36 2015 +0100

    Remove even older rumpxen-app-* variants of app-tools
    
    Signed-off-by: Martin Lucina <martin@lucina.net>

commit 91d56232d987f5df594723ed46b9000b4d43e21a
Author: Martin Lucina <martin@lucina.net>
Date:   Mon Feb 2 17:52:41 2015 +0100

    Renaming platform macros, app-tools and autoconf target string
    
    As discussed at: http://thread.gmane.org/gmane.comp.rumpkernel.user/739
    
    This commit renames the platform macros, app-tools and autoconf target
    string to be consistent with current naming of the entire stack:
    
    app-tools/rumpapp-xen-* -> app-tools/rumprun-xen-*
    $ARCH-rumpxen-netbsd -> $ARCH-rumprun-netbsd
    -D__RUMPUSER_XEN__ -D__RUMPAPP__ -> -D__RUMPRUN__
    
    Signed-off-by: Martin Lucina <martin@lucina.net>

Re: [rumpuserxen test] 34129: regressions - FAIL

[Martin Lucina]

Caused by:

> commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c
> Author: Martin Lucina <martin@lucina.net>
> Date:   Mon Feb 2 18:00:36 2015 +0100
> 
>     Remove even older rumpxen-app-* variants of app-tools
>     
>     Signed-off-by: Martin Lucina <martin@lucina.net>

bash: line 14: /home/osstest/build.34129.build-amd64-rumpuserxen/rumpuserxen/app-tools/rumpxen-app-configure: No such file or directory

Ian, can you update osstest to use rumprun-xen-configure as discussed on
rumpkernel-users? 

Thanks,

Martin

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

Re: [Xen-devel] [rumpuserxen test] 34129: regressions - FAIL

[Ian Campbell]

On Wed, 2015-02-04 at 12:21 +0100, Martin Lucina wrote:
> Caused by:
> 
> > commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c
> > Author: Martin Lucina <martin@lucina.net>
> > Date:   Mon Feb 2 18:00:36 2015 +0100
> > 
> >     Remove even older rumpxen-app-* variants of app-tools
> >     
> >     Signed-off-by: Martin Lucina <martin@lucina.net>
> 
> bash: line 14: /home/osstest/build.34129.build-amd64-rumpuserxen/rumpuserxen/app-tools/rumpxen-app-configure: No such file or directory
> 
> Ian, can you update osstest to use rumprun-xen-configure as discussed on
> rumpkernel-users? 

I think we also need Wei's patch from
<1422971228-4320-1-git-send-email-wei.liu2@citrix.com> / 
http://article.gmane.org/gmane.comp.emulators.xen.devel/230968

to deal with:

        commit 91d56232d987f5df594723ed46b9000b4d43e21a
        Author: Martin Lucina <martin@lucina.net>
        Date:   Mon Feb 2 17:52:41 2015 +0100
        
            Renaming platform macros, app-tools and autoconf target string
            
            As discussed at: http://thread.gmane.org/gmane.comp.rumpkernel.user/739
            
            This commit renames the platform macros, app-tools and autoconf target
            string to be consistent with current naming of the entire stack:
            
            app-tools/rumpapp-xen-* -> app-tools/rumprun-xen-*
            $ARCH-rumpxen-netbsd -> $ARCH-rumprun-netbsd
            -D__RUMPUSER_XEN__ -D__RUMPAPP__ -> -D__RUMPRUN__
            
            Signed-off-by: Martin Lucina <martin@lucina.net>

Ian.


------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

[PATCH] ts-rumpuserxen-build: Use new app-tools

[Martin Lucina]

Ian.Campbell@citrix.com said:
> On Wed, 2015-02-04 at 12:21 +0100, Martin Lucina wrote:
> > Caused by:
> > 
> > > commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c
> > > Author: Martin Lucina <martin@lucina.net>
> > > Date:   Mon Feb 2 18:00:36 2015 +0100
> > > 
> > >     Remove even older rumpxen-app-* variants of app-tools
> > >     
> > >     Signed-off-by: Martin Lucina <martin@lucina.net>
> > 
> > bash: line 14: /home/osstest/build.34129.build-amd64-rumpuserxen/rumpuserxen/app-tools/rumpxen-app-configure: No such file or directory
> > 
> > Ian, can you update osstest to use rumprun-xen-configure as discussed on
> > rumpkernel-users? 
> 
> I think we also need Wei's patch from
> <1422971228-4320-1-git-send-email-wei.liu2@citrix.com> / 
> http://article.gmane.org/gmane.comp.emulators.xen.devel/230968

Ack. Patch to update Osstest follows.

>From ff80800099956e429df65bdf7669083cf5c5a8df Mon Sep 17 00:00:00 2001
From: Martin Lucina <martin@lucina.net>
Date: Wed, 4 Feb 2015 12:31:33 +0100
Subject: [PATCH] ts-rumpuserxen-build: Use new app-tools

Osstest was using old names for rumprun-xen app-tools. These have been
removed in rumprun-xen commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c.

Update to use the new app-tools names.

Signed-off-by: Martin Lucina <martin@lucina.net>
---
 ts-rumpuserxen-build |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-rumpuserxen-build b/ts-rumpuserxen-build
index 47bcf76..916100b 100755
--- a/ts-rumpuserxen-build
+++ b/ts-rumpuserxen-build
@@ -98,7 +98,7 @@ END
 }
 
 sub recordtools() {
-    my $apptool = "$rux/app-tools/rumpxen-app";
+    my $apptool = "$rux/app-tools/rumprun-xen";
     store_runvar('cmdprefix_configure', "$apptool-configure");
     store_runvar('cmdprefix_make',      "$apptool-make");
 }
-- 
1.7.10.4


------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

Re: [PATCH] ts-rumpuserxen-build: Use new app-tools

[Ian Campbell]

On Wed, 2015-02-04 at 12:36 +0100, Martin Lucina wrote:
> Ian.Campbell@citrix.com said:
> > On Wed, 2015-02-04 at 12:21 +0100, Martin Lucina wrote:
> > > Caused by:
> > > 
> > > > commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c
> > > > Author: Martin Lucina <martin@lucina.net>
> > > > Date:   Mon Feb 2 18:00:36 2015 +0100
> > > > 
> > > >     Remove even older rumpxen-app-* variants of app-tools
> > > >     
> > > >     Signed-off-by: Martin Lucina <martin@lucina.net>
> > > 
> > > bash: line 14: /home/osstest/build.34129.build-amd64-rumpuserxen/rumpuserxen/app-tools/rumpxen-app-configure: No such file or directory
> > > 
> > > Ian, can you update osstest to use rumprun-xen-configure as discussed on
> > > rumpkernel-users? 
> > 
> > I think we also need Wei's patch from
> > <1422971228-4320-1-git-send-email-wei.liu2@citrix.com> / 
> > http://article.gmane.org/gmane.comp.emulators.xen.devel/230968
> 
> Ack. Patch to update Osstest follows.
> 
> From ff80800099956e429df65bdf7669083cf5c5a8df Mon Sep 17 00:00:00 2001
> From: Martin Lucina <martin@lucina.net>
> Date: Wed, 4 Feb 2015 12:31:33 +0100
> Subject: [PATCH] ts-rumpuserxen-build: Use new app-tools
> 
> Osstest was using old names for rumprun-xen app-tools. These have been
> removed in rumprun-xen commit 3b36d1f55a08e1849ccd5424afb0fbe29647bd6c.
> 
> Update to use the new app-tools names.

I'm not sure if we need to support bisection over this change -- but I
think we probably do (Ian?) in which case we would need to probe for
which one actually exists, preferring the newer name.

Ian?

> 
> Signed-off-by: Martin Lucina <martin@lucina.net>
> ---
>  ts-rumpuserxen-build |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ts-rumpuserxen-build b/ts-rumpuserxen-build
> index 47bcf76..916100b 100755
> --- a/ts-rumpuserxen-build
> +++ b/ts-rumpuserxen-build
> @@ -98,7 +98,7 @@ END
>  }
>  
>  sub recordtools() {
> -    my $apptool = "$rux/app-tools/rumpxen-app";
> +    my $apptool = "$rux/app-tools/rumprun-xen";
>      store_runvar('cmdprefix_configure', "$apptool-configure");
>      store_runvar('cmdprefix_make',      "$apptool-make");
>  }



------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

Re: [PATCH] ts-rumpuserxen-build: Use new app-tools

[Ian Jackson]

Ian Campbell writes ("Re: [PATCH] ts-rumpuserxen-build: Use new app-tools"):
> On Wed, 2015-02-04 at 12:36 +0100, Martin Lucina wrote:
> > Update to use the new app-tools names.
> 
> I'm not sure if we need to support bisection over this change -- but I
> think we probably do (Ian?) in which case we would need to probe for
> which one actually exists, preferring the newer name.

It's not just bisection; our push gate will fail because of the lack
of compatibility.

So we need to probe.  I will prepare a patch.

Ian.

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

Re: [PATCH] ts-rumpuserxen-build: Use new app-tools

[Antti Kantee]

On 04/02/15 14:33, Ian Jackson wrote:
> Ian Campbell writes ("Re: [PATCH] ts-rumpuserxen-build: Use new app-tools"):
>> On Wed, 2015-02-04 at 12:36 +0100, Martin Lucina wrote:
>>> Update to use the new app-tools names.
>>
>> I'm not sure if we need to support bisection over this change -- but I
>> think we probably do (Ian?) in which case we would need to probe for
>> which one actually exists, preferring the newer name.
>
> It's not just bisection; our push gate will fail because of the lack
> of compatibility.
>
> So we need to probe.  I will prepare a patch.

Can the grievous-bodge grievances now be removed, or are prehistoric 
revisions still required to build?

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

[OSSTEST PATCH] ts-rumpuserxen-build: Cope with rumprun-xen rename

[Ian Jackson]

rumpxen-app-* has been renamed to rumprun-xen-*.  We need to cope with
either, at least for the transition.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Martin Lucina <martin@lucina.net>
---
 ts-rumpuserxen-build |   11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/ts-rumpuserxen-build b/ts-rumpuserxen-build
index 47bcf76..9e6580a 100755
--- a/ts-rumpuserxen-build
+++ b/ts-rumpuserxen-build
@@ -98,9 +98,14 @@ END
 }
 
 sub recordtools() {
-    my $apptool = "$rux/app-tools/rumpxen-app";
-    store_runvar('cmdprefix_configure', "$apptool-configure");
-    store_runvar('cmdprefix_make',      "$apptool-make");
+    foreach my $stem (qw(rumprun-xen rumpxen-app)) {
+	my $apptool = "$rux/app-tools/$stem";
+	next unless target_file_exists($ho, "$apptool-configure");
+	store_runvar('cmdprefix_configure', "$apptool-configure");
+	store_runvar('cmdprefix_make',      "$apptool-make");
+	return;
+    }
+    die "app-tools not found ($rux)";
 }
 
 sub install() {
-- 
1.7.10.4

Re: [PATCH] ts-rumpuserxen-build: Use new app-tools

[Ian Jackson]

Antti Kantee writes ("Re: [PATCH] ts-rumpuserxen-build: Use new app-tools"):
> On 04/02/15 14:33, Ian Jackson wrote:
> > So we need to probe.  I will prepare a patch.
> 
> Can the grievous-bodge grievances now be removed, or are prehistoric 
> revisions still required to build?

I think we can probably delete those now but we should do that as a
separate step to reduce the risk of messing up.  We want the fix to
cope with rumprun-xen to pass the osstest self-push-gate.

Ian.

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/

Re: [OSSTEST PATCH] ts-rumpuserxen-build: Cope with rumprun-xen rename

[Ian Campbell]

On Wed, 2015-02-04 at 14:41 +0000, Ian Jackson wrote:
> rumpxen-app-* has been renamed to rumprun-xen-*.  We need to cope with
> either, at least for the transition.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Martin Lucina <martin@lucina.net>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

> ---
>  ts-rumpuserxen-build |   11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/ts-rumpuserxen-build b/ts-rumpuserxen-build
> index 47bcf76..9e6580a 100755
> --- a/ts-rumpuserxen-build
> +++ b/ts-rumpuserxen-build
> @@ -98,9 +98,14 @@ END
>  }
>  
>  sub recordtools() {
> -    my $apptool = "$rux/app-tools/rumpxen-app";
> -    store_runvar('cmdprefix_configure', "$apptool-configure");
> -    store_runvar('cmdprefix_make',      "$apptool-make");
> +    foreach my $stem (qw(rumprun-xen rumpxen-app)) {
> +	my $apptool = "$rux/app-tools/$stem";
> +	next unless target_file_exists($ho, "$apptool-configure");
> +	store_runvar('cmdprefix_configure', "$apptool-configure");
> +	store_runvar('cmdprefix_make',      "$apptool-make");
> +	return;
> +    }
> +    die "app-tools not found ($rux)";
>  }
>  
>  sub install() {