Ping.

• Removed redundand lines from Makefile and pkg-descr, according to review feedback

A few small changes and it is ready for merge. The report also needs to have one sentence per line (I can do it at merge time if you prefer).
Thanks!

switch to "400Gb, 200Gb, 50Gb, 10Gb", expand list to include suffixes
taken from chelsio unified wire adapter webpage.

A few small changes and it is ready for merge. The report also needs to have one sentence per line (I can do it at merge time if you prefer).
Thanks!

Move setting of -DLLVM_BUILD_LLVM_DYLIB to the right place.

Sorry I forgot about this! This is a great improvement.

From the PR submitted on Bugzilla:

Thank you for the fast fix! I can confirm that your patch works great.

but don't provide an example

suggest using INSTKERNNAME for installkernel

In D54164#1239215, @christos wrote:

I actually think the latter is a good idea, and probably more robust.

Approved, thanks!

Approved, thanks!

In D54289#1242003, @imp wrote:

I'd still be tempted to put it in files and just spike 32bit builds. Sure, you can put it in the config file and yhe error is delayed until you build, but that's ok imho. Lots of things in files that don't work evertwhere. No need for a files.64.

I'd still be tempted to put it in files and just spike 32bit builds. Sure, you can put it in the config file and yhe error is delayed until you build, but that's ok imho. Lots of things in files that don't work evertwhere. No need for a files.64.

In D54289#1241834, @kgalazka wrote:

The module will build only on 64-bit arch (amd64 and powerpc64 was already there, this patch adds arm64). I don't think it was ever tested on 32-bits, and I don't have too many spare cycles to look into that.