Re: [PATCH OSSTEST v5 9/9] mfi-common, make-flight: create XSM test jobs

[Ian Campbell <Ian.Campbell@citrix.com>]

On Tue, 2015-02-03 at 10:36 +0000, Wei Liu wrote:
> Duplicate Debian PV and HVM test jobs for XSM testing.

If ~/.xen-osstest/config contains e.g. "PlatformsArmhf midway cubietruck
arndale" (correspodning to the platform-{...} flag being set on some
host) then I think this will generate a per platform XSM test, which
doesn't seem necessary since XSM isn't platform specific.

So:

> +  for xsm in $test_xsm ; do
> +
> +    # Basic PV Linux test with xl
> +    for platform in '' `getplatforms $xenarch` ; do
> +      suffix=${platform:+-$platform}
> +      hostflags=${most_hostflags}${platform:+,platform-$platform}
> +
> +      job_create_test test-$xenarch$kern-$dom0arch-xl$suffix test-debian xl \
> +                      $xenarch $dom0arch                                    \
> +                      enable_xsm=$xsm                                       \
> +                      $debian_runvars all_hostflags=$hostflags
> +    done

I think you could pull the for platform loop outside of the for xsm one
and remove '' from it, and readd the generic one standalone inside the
xsm loop. (You can also simplify the suffix stuff in the platform loop
since that is just to handle '' correctly).

Or add "if xsm = true and platform != '' -> continue"? 

Ian.