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

Re: [Xen-devel] [OSSTEST PATCH 2/2] RFC: enable CONFIG_LIVEPATCH in all relevant arm64 builds [and 1 more messages]



Julien Grall writes ("Re: [OSSTEST PATCH 2/2] RFC: enable CONFIG_LIVEPATCH in 
all relevant arm64 builds"):
> On 05/06/2019 12:13, Ian Jackson wrote:
> > Switching on CONFIG_LIVEPATCH for the affected tests will, hopefully,
> > make this problem "go away" again.  This is clearly a bodge.  But it
> > is better than simply force pushing: if we can get it to boot, we will
> > be able to run the other tests.
> 
> How about disabling the test on rochester?

I could invent a new hostflag
  xen-4.11-arm64-seems-to-boot-ok
but that seems ridiculous.

And, we presumably want to actually test other things on rochester!

Jan Beulich writes ("Re: [OSSTEST PATCH 2/2] RFC: enable CONFIG_LIVEPATCH in 
all relevant arm64 builds"):
> On 05.06.19 at 13:13, <ian.jackson@xxxxxxxxxxxxx> wrote:
> > --- a/mfi-common
> > +++ b/mfi-common
> > @@ -373,7 +373,8 @@ create_build_jobs () {
> >          fi
> >      fi
> >  
> > -    if branch_wants_livepatch; then
> > +    if branch_wants_livepatch ||
> > +          [ $arch = arm64 -a "$xenbranch" = xen-4.11-testing ]; then
> >         livepatch_runvars='enable_livepatch=true'
> >      fi
> 
> Isn't this overly restrictive, i.e. wouldn't this better be done uniformly
> for all branches?

No.  Because the bug is a random failure, I don't want to permute all
the other branches and maybe cause them to experience it.

I still think it would be better to fix this in the Xen code.

Would it be possible to make whether to do "wrong thing A" (which does
not boot on rochster) or "wrong thing B" (which boots on rochester but
not on some other unknown machine(s)) a config or boot-time option ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.