NAK: [PULL] [B] dkms: build out-of-tree modules from dkms source

[Andrea Righi]

On Fri, Jun 28, 2019 at 08:35:25AM -0500, Seth Forshee wrote:
> On Thu, Jun 13, 2019 at 09:31:24AM +0200, Andrea Righi wrote:
> > BugLink: https://bugs.launchpad.net/bugs/1764792
> > BugLink: https://bugs.launchpad.net/bugs/1807378
> > 
> > This is a backport of the dkms packaging from disco to bionic that
> > allows to build out-of-tree modules (nvidia drivers and zfs/spl for now)
> > from DKMS source as part of the regular kernel build process.
> > 
> > In this way we don't have to maintain a complete copy of zfs/spl source
> > code in the kernel (source code is downloaded and compiled as part of
> > the dkms build process).
> > 
> > About nvidia, modules are built and signed and only the signatures are
> > shipped.
> > 
> > The following changes since commit edaa4ede0ca39f3578255fbe06a9db53df1d87f1:
> > 
> >   drm/i915/gvt: Fix aperture read/write emulation when enable x-no-mmap=on (2019-06-11 17:58:47 +0200)
> > 
> > are available in the Git repository at:
> > 
> >   git://git.launchpad.net/~arighi/+git/bionic-linux dkms-build
> > 
> > for you to fetch changes up to 9aa2076c1603ee94a20d762756d2377dc7e3206f:
> > 
> >   UBUNTU: [Packaging] dkms -- drop zfs/spl source code from kernel (2019-06-12 17:39:27 +0200)
> 
> These patches don't have anything to do with dkms builds:
> 
>   UBUNTU: [Packaging] remove handoff check for uefi signing
>   UBUNTU: [Packaging] decompress gzipped efi images in signing tarball

OK, I'll drop these two.

> 
> They are for supporting signed arm64 kernels for UEFI secure boot. It
> looks like I must have used the wrong bug number when applying those
> patches, sorry about that. They should be dropped from the series.
> 
> Additionally I sent a patch for disco yesterday that I think should be
> included:
> 
> https://lists.ubuntu.com/archives/kernel-team/2019-June/101662.html

And I'll add this in.

> 
> Otherwise this looks good.

I have to send a v2 anyway, I'm testing a linux-lrm repo right now, so
I may have also other changes to apply.

Thanks for the review!

-Andrea