On 28 April 2017 at 16:00, Andres Gomez <ago...@igalia.com> wrote: > On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: >> From: Emil Velikov <emil.veli...@collabora.com> >> >> Split the target to allow faster builds for each run. >> >> The overall build time will be more, yet Travis runs multiple builds in >> parallel so we're limited by the slowest one. >> >> Things are split roughly as: >> - DRI loaders, classic DRI drivers, classic OSMesa, make check >> - All Gallium drivers (minus the SWR) alongside st/dri (mesa) >> - The Vulkan drivers - ANV and RADV, make check (anv) > > I think it would be better to split ANV and RADV on different builds > too. > Any particular reason why?
As-is the jobs are fairly well balanced - be that with cold [1] or warm ccache [2] [3]. In the latter case, big hunk of the time is dominated by the build/install of dependencies (even w/o wayland/va/vdpau). [1] https://travis-ci.org/evelikov/Mesa/builds/226462487 [2] https://travis-ci.org/evelikov/Mesa/builds/226874115 [3] https://travis-ci.org/evelikov/Mesa/builds/226879305 >> @@ -207,18 +255,33 @@ install: >> (cd $LIBTXC_DXTN_VERSION && ./configure --prefix=$HOME/prefix && make >> install); >> fi >> >> + # Generate the header since one is missing on the Travis instance >> + - mkdir -p linux >> + - echo "#ifndef _LINUX_MEMFD_H" > linux/memfd.h >> + - echo "#define _LINUX_MEMFD_H" >> linux/memfd.h >> + - echo "" >> linux/memfd.h >> + - echo "#define __NR_memfd_create 319" >> linux/memfd.h >> + - echo "#define SYS_memfd_create __NR_memfd_create" >> linux/memfd.h >> + - echo "" >> linux/memfd.h >> + - echo "#define MFD_CLOEXEC 0x0001U" >> linux/memfd.h >> + - echo "#define MFD_ALLOW_SEALING 0x0002U" >> linux/memfd.h >> + - echo "" >> linux/memfd.h >> + - echo "#endif /* _LINUX_MEMFD_H */" >> linux/memfd.h > > This is a bit ugly on the travis log output. What about replacing it > with something like?: > > - printf "%s\n" \ > "#ifndef _LINUX_MEMFD_H" \ > "#define _LINUX_MEMFD_H" \ > "" \ > "#define __NR_memfd_create 319" \ > "#define SYS_memfd_create __NR_memfd_create" \ > "" \ > "#define MFD_CLOEXEC 0x0001U" \ > "#define MFD_ALLOW_SEALING 0x0002U" \ > "" \ > "#endif /* _LINUX_MEMFD_H */" > linux/memfd.h > Looks nice and works like a charm. Much appreciated. -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
