* [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs @ 2019-11-21 5:47 Heiko Schocher 2019-11-21 5:51 ` Bin Meng 0 siblings, 1 reply; 4+ messages in thread From: Heiko Schocher @ 2019-11-21 5:47 UTC (permalink / raw) To: u-boot remove from NXP arm32 all layerscape boards and build them instead in already existing layerscape jobs (which now not only build aarch64 boards) Signed-off-by: Heiko Schocher <hs@denx.de> --- travis build: https://travis-ci.org/hsdenx/u-boot-test/builds/614396275?utm_medium=notification&utm_source=email Changes in v2: - collect in the layerscape jobs not only aarch64 based boards as Tom suggested. - change subject, old was "travis: split NXP ARM32 job" .travis.yml | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/.travis.yml b/.travis.yml index f6aec96770..d5f34f96cc 100644 --- a/.travis.yml +++ b/.travis.yml @@ -183,27 +183,27 @@ matrix: - name: "buildman ARM bcm" env: - BUILDMAN="bcm -x mips" - - name: "buildman NXP ARM32" + - name: "buildman NXP ARM32 (catch-all)" env: - - BUILDMAN="freescale -x powerpc,m68k,aarch64" - - name: "buildman NXP AArch64 LS101x" + - BUILDMAN="freescale -x powerpc,m68k,aarch64,ls101,ls102,ls104,ls108,ls20,lx216" + - name: "buildman NXP LS101x" env: - - BUILDMAN="freescale&aarch64&ls101" - - name: "buildman NXP AArch64 LS102x" + - BUILDMAN="freescale&ls101" + - name: "buildman NXP LS102x" env: - - BUILDMAN="freescale&aarch64&ls102" - - name: "buildman NXP AArch64 LS104x" + - BUILDMAN="freescale&ls102" + - name: "buildman NXP LS104x" env: - - BUILDMAN="freescale&aarch64&ls104" - - name: "buildman NXP AArch64 LS108x" + - BUILDMAN="freescale&ls104" + - name: "buildman NXP LS108x" env: - - BUILDMAN="freescale&aarch64&ls108" - - name: "buildman NXP AArch64 LS20xx" + - BUILDMAN="freescale&ls108" + - name: "buildman NXP LS20xx" env: - - BUILDMAN="freescale&aarch64&ls20" - - name: "buildman NXP AArch64 LX216x" + - BUILDMAN="freescale&ls20" + - name: "buildman NXP LX216x" env: - - BUILDMAN="freescale&aarch64&lx216" + - BUILDMAN="freescale&lx216" - name: "buildman i.MX6 tqc" env: - BUILDMAN="mx6&tqc" -- 2.21.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs 2019-11-21 5:47 [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs Heiko Schocher @ 2019-11-21 5:51 ` Bin Meng 2019-11-21 6:24 ` Heiko Schocher 0 siblings, 1 reply; 4+ messages in thread From: Bin Meng @ 2019-11-21 5:51 UTC (permalink / raw) To: u-boot Hi Heiko, On Thu, Nov 21, 2019 at 1:48 PM Heiko Schocher <hs@denx.de> wrote: > > remove from NXP arm32 all layerscape boards and > build them instead in already existing layerscape > jobs (which now not only build aarch64 boards) > > Signed-off-by: Heiko Schocher <hs@denx.de> > --- > travis build: > https://travis-ci.org/hsdenx/u-boot-test/builds/614396275?utm_medium=notification&utm_source=email > > Changes in v2: > - collect in the layerscape jobs not only aarch64 based > boards as Tom suggested. > - change subject, old was > "travis: split NXP ARM32 job" > > .travis.yml | 28 ++++++++++++++-------------- > 1 file changed, 14 insertions(+), 14 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index f6aec96770..d5f34f96cc 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -183,27 +183,27 @@ matrix: > - name: "buildman ARM bcm" > env: > - BUILDMAN="bcm -x mips" > - - name: "buildman NXP ARM32" > + - name: "buildman NXP ARM32 (catch-all)" > env: > - - BUILDMAN="freescale -x powerpc,m68k,aarch64" > - - name: "buildman NXP AArch64 LS101x" > + - BUILDMAN="freescale -x powerpc,m68k,aarch64,ls101,ls102,ls104,ls108,ls20,lx216" > + - name: "buildman NXP LS101x" > env: > - - BUILDMAN="freescale&aarch64&ls101" > - - name: "buildman NXP AArch64 LS102x" > + - BUILDMAN="freescale&ls101" > + - name: "buildman NXP LS102x" > env: > - - BUILDMAN="freescale&aarch64&ls102" > - - name: "buildman NXP AArch64 LS104x" > + - BUILDMAN="freescale&ls102" > + - name: "buildman NXP LS104x" > env: > - - BUILDMAN="freescale&aarch64&ls104" > - - name: "buildman NXP AArch64 LS108x" > + - BUILDMAN="freescale&ls104" > + - name: "buildman NXP LS108x" > env: > - - BUILDMAN="freescale&aarch64&ls108" > - - name: "buildman NXP AArch64 LS20xx" > + - BUILDMAN="freescale&ls108" > + - name: "buildman NXP LS20xx" > env: > - - BUILDMAN="freescale&aarch64&ls20" > - - name: "buildman NXP AArch64 LX216x" > + - BUILDMAN="freescale&ls20" > + - name: "buildman NXP LX216x" > env: > - - BUILDMAN="freescale&aarch64&lx216" > + - BUILDMAN="freescale&lx216" > - name: "buildman i.MX6 tqc" > env: > - BUILDMAN="mx6&tqc" Will you do the same for Azure? Or does this only apply to Travis? Regards, Bin ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs 2019-11-21 5:51 ` Bin Meng @ 2019-11-21 6:24 ` Heiko Schocher 2019-11-21 12:58 ` Tom Rini 0 siblings, 1 reply; 4+ messages in thread From: Heiko Schocher @ 2019-11-21 6:24 UTC (permalink / raw) To: u-boot Hello Bin, Am 21.11.2019 um 06:51 schrieb Bin Meng: > Hi Heiko, > > On Thu, Nov 21, 2019 at 1:48 PM Heiko Schocher <hs@denx.de> wrote: >> >> remove from NXP arm32 all layerscape boards and >> build them instead in already existing layerscape >> jobs (which now not only build aarch64 boards) >> >> Signed-off-by: Heiko Schocher <hs@denx.de> >> --- >> travis build: >> https://travis-ci.org/hsdenx/u-boot-test/builds/614396275?utm_medium=notification&utm_source=email >> >> Changes in v2: >> - collect in the layerscape jobs not only aarch64 based >> boards as Tom suggested. >> - change subject, old was >> "travis: split NXP ARM32 job" >> >> .travis.yml | 28 ++++++++++++++-------------- >> 1 file changed, 14 insertions(+), 14 deletions(-) >> >> diff --git a/.travis.yml b/.travis.yml >> index f6aec96770..d5f34f96cc 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -183,27 +183,27 @@ matrix: >> - name: "buildman ARM bcm" >> env: >> - BUILDMAN="bcm -x mips" >> - - name: "buildman NXP ARM32" >> + - name: "buildman NXP ARM32 (catch-all)" >> env: >> - - BUILDMAN="freescale -x powerpc,m68k,aarch64" >> - - name: "buildman NXP AArch64 LS101x" >> + - BUILDMAN="freescale -x powerpc,m68k,aarch64,ls101,ls102,ls104,ls108,ls20,lx216" >> + - name: "buildman NXP LS101x" >> env: >> - - BUILDMAN="freescale&aarch64&ls101" >> - - name: "buildman NXP AArch64 LS102x" >> + - BUILDMAN="freescale&ls101" >> + - name: "buildman NXP LS102x" >> env: >> - - BUILDMAN="freescale&aarch64&ls102" >> - - name: "buildman NXP AArch64 LS104x" >> + - BUILDMAN="freescale&ls102" >> + - name: "buildman NXP LS104x" >> env: >> - - BUILDMAN="freescale&aarch64&ls104" >> - - name: "buildman NXP AArch64 LS108x" >> + - BUILDMAN="freescale&ls104" >> + - name: "buildman NXP LS108x" >> env: >> - - BUILDMAN="freescale&aarch64&ls108" >> - - name: "buildman NXP AArch64 LS20xx" >> + - BUILDMAN="freescale&ls108" >> + - name: "buildman NXP LS20xx" >> env: >> - - BUILDMAN="freescale&aarch64&ls20" >> - - name: "buildman NXP AArch64 LX216x" >> + - BUILDMAN="freescale&ls20" >> + - name: "buildman NXP LX216x" >> env: >> - - BUILDMAN="freescale&aarch64&lx216" >> + - BUILDMAN="freescale&lx216" >> - name: "buildman i.MX6 tqc" >> env: >> - BUILDMAN="mx6&tqc" > > Will you do the same for Azure? Or does this only apply to Travis? The patch is only for travis, as travis has a job timeout of 50 minutes, and this job took to long for me... The question is, does have Azure also such a job timeout? Do we want to have the Azure jobs in sync with travis jobs? bye, Heiko -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: hs at denx.de ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs 2019-11-21 6:24 ` Heiko Schocher @ 2019-11-21 12:58 ` Tom Rini 0 siblings, 0 replies; 4+ messages in thread From: Tom Rini @ 2019-11-21 12:58 UTC (permalink / raw) To: u-boot On Thu, Nov 21, 2019 at 07:24:26AM +0100, Heiko Schocher wrote: > Hello Bin, > > Am 21.11.2019 um 06:51 schrieb Bin Meng: > > Hi Heiko, > > > > On Thu, Nov 21, 2019 at 1:48 PM Heiko Schocher <hs@denx.de> wrote: > > > > > > remove from NXP arm32 all layerscape boards and > > > build them instead in already existing layerscape > > > jobs (which now not only build aarch64 boards) > > > > > > Signed-off-by: Heiko Schocher <hs@denx.de> > > > --- > > > travis build: > > > https://travis-ci.org/hsdenx/u-boot-test/builds/614396275?utm_medium=notification&utm_source=email > > > > > > Changes in v2: > > > - collect in the layerscape jobs not only aarch64 based > > > boards as Tom suggested. > > > - change subject, old was > > > "travis: split NXP ARM32 job" > > > > > > .travis.yml | 28 ++++++++++++++-------------- > > > 1 file changed, 14 insertions(+), 14 deletions(-) > > > > > > diff --git a/.travis.yml b/.travis.yml > > > index f6aec96770..d5f34f96cc 100644 > > > --- a/.travis.yml > > > +++ b/.travis.yml > > > @@ -183,27 +183,27 @@ matrix: > > > - name: "buildman ARM bcm" > > > env: > > > - BUILDMAN="bcm -x mips" > > > - - name: "buildman NXP ARM32" > > > + - name: "buildman NXP ARM32 (catch-all)" > > > env: > > > - - BUILDMAN="freescale -x powerpc,m68k,aarch64" > > > - - name: "buildman NXP AArch64 LS101x" > > > + - BUILDMAN="freescale -x powerpc,m68k,aarch64,ls101,ls102,ls104,ls108,ls20,lx216" > > > + - name: "buildman NXP LS101x" > > > env: > > > - - BUILDMAN="freescale&aarch64&ls101" > > > - - name: "buildman NXP AArch64 LS102x" > > > + - BUILDMAN="freescale&ls101" > > > + - name: "buildman NXP LS102x" > > > env: > > > - - BUILDMAN="freescale&aarch64&ls102" > > > - - name: "buildman NXP AArch64 LS104x" > > > + - BUILDMAN="freescale&ls102" > > > + - name: "buildman NXP LS104x" > > > env: > > > - - BUILDMAN="freescale&aarch64&ls104" > > > - - name: "buildman NXP AArch64 LS108x" > > > + - BUILDMAN="freescale&ls104" > > > + - name: "buildman NXP LS108x" > > > env: > > > - - BUILDMAN="freescale&aarch64&ls108" > > > - - name: "buildman NXP AArch64 LS20xx" > > > + - BUILDMAN="freescale&ls108" > > > + - name: "buildman NXP LS20xx" > > > env: > > > - - BUILDMAN="freescale&aarch64&ls20" > > > - - name: "buildman NXP AArch64 LX216x" > > > + - BUILDMAN="freescale&ls20" > > > + - name: "buildman NXP LX216x" > > > env: > > > - - BUILDMAN="freescale&aarch64&lx216" > > > + - BUILDMAN="freescale&lx216" > > > - name: "buildman i.MX6 tqc" > > > env: > > > - BUILDMAN="mx6&tqc" > > > > Will you do the same for Azure? Or does this only apply to Travis? > > The patch is only for travis, as travis has a job timeout of 50 minutes, > and this job took to long for me... > > The question is, does have Azure also such a job timeout? > > Do we want to have the Azure jobs in sync with travis jobs? So, we have a few things to consider here (and it goes for this patch and the sun8i/oranagepi one too). First, Travis times can be heavily variable and I think that's what happened here. I don't see these jobs timing out usually but when they get put on a slow VM, they can. That said, working around that can result in a little cleaner separation in the file, so it's worth dealing with. Second, there's pros and cons to trying to keep Travis and Azure splits in-sync. Since it's not a 1:1 match in the split, as we can get more done per Azure job, it's more of a guess work on when to keep things in sync (for more parallelization and faster Azure jobs) or not. So in this case, I think we should try and keep them in sync as it's just changes to what we pass buildman (so easy enough to get right by inspection) and might speed the overall task up slightly. In the case of the orangepi split, at this time I don't think it's worth doing to Azure as well. Excluding orangepi in a few places, but then spinning up another job isn't going to be wall-clock win and might be a small loss, and the limit within Azure is parallel jobs not build run time. Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20191121/cd6fbc71/attachment.sig> ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-21 12:58 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-11-21 5:47 [U-Boot] [PATCH v2] travis: rework NXP layerscape jobs Heiko Schocher 2019-11-21 5:51 ` Bin Meng 2019-11-21 6:24 ` Heiko Schocher 2019-11-21 12:58 ` Tom Rini
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox