[dpdk-dev] [PATCH 1/2] ci: remove redundant configs disabling kmods

Thomas Monjalon thomas at monjalon.net
Mon Feb 17 19:00:33 CET 2020


17/02/2020 17:46, Bruce Richardson:
> On Mon, Feb 17, 2020 at 11:42:37AM -0500, Aaron Conole wrote:
> > Thomas Monjalon <thomas at monjalon.net> writes:
> > 
> > > The kernel modules are not built by default since below commit.
> > > The Travis CI matrix can be simplified
> > > by removing the redundant option enable_kmods=false,
> > > and by removing some jobs which become redundant.
> > >
> > > Note: there is no benefit in checking kmods in Travis Ubuntu.
> > > Anyway different kernel versions and distributions are needed,
> > > and the Technical Board is looking at moving the Linux kmods
> > > out of this repository.
> > >
> > > Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
> > > ---
> > 
> > Looks good to me, but do you think there's no benefit to even compiling
> > the kernel modules that get shipped?
> > 
> > Otherwise,
> > Acked-by: Aaron Conole <aconole at redhat.com>
> > 
> I think we need one compile that turns them on to test.

I don't think so.
Travis is compiling only on Ubuntu.
Kernel modules compilation must be tested on old kernels (RHEL)
and latest ones (Fedora).




More information about the dev mailing list