Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Martin Husemann
On Sun, May 10, 2020 at 07:52:42PM -, Michael van Elst wrote: > And I'd like to talk to Martin, since 100% of the task is done if > we just add the missing dependency without the option. Lets start with just the dependency and see if everything fits in the next build run. We can add the

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Michael van Elst wrote: p...@whooppee.com (Paul Goyette) writes: Well, 90%+ of the effort has already been expended, so why not? 90%+ is probably the time needed to test optional builds in the future. FWIW, I have just successfully completed my build of my custom

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Michael van Elst
p...@whooppee.com (Paul Goyette) writes: >> Well, 90%+ of the effort has already been expended, so why not? 90%+ is probably the time needed to test optional builds in the future. >FWIW, I have just successfully completed my build of my custom kernel >(and an entire ``build.sh release'') and it

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Paul Goyette wrote: On Sun, 10 May 2020, Michael van Elst wrote: p...@whooppee.com (Paul Goyette) writes: FWIW, here's an additional patch to update the options(4) man page: But is it worth the effort to make 16kB optional ? Well, 90%+ of the effort has already been

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Michael van Elst wrote: p...@whooppee.com (Paul Goyette) writes: FWIW, here's an additional patch to update the options(4) man page: But is it worth the effort to make 16kB optional ? Well, 90%+ of the effort has already been expended, so why not?

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Michael van Elst
p...@whooppee.com (Paul Goyette) writes: >FWIW, here's an additional patch to update the options(4) man page: But is it worth the effort to make 16kB optional ? -- -- Michael van Elst Internet: mlel...@serpens.de "A potential

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Michael van Elst wrote: On Sun, May 10, 2020 at 08:46:30AM -0700, Paul Goyette wrote: Here is a patch that makes encrypted swap (but not rijndael) optional: http://ftp.netbsd.org/pub/NetBSD/misc/mlelstv/uvm_swap.diff That looks good to me, too. I guess you should also

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Paul Goyette wrote: Here is a patch that makes encrypted swap (but not rijndael) optional: http://ftp.netbsd.org/pub/NetBSD/misc/mlelstv/uvm_swap.diff That looks good to me, too. I guess you should also add VM_SWAPCRYPT option to various kernels (GENERIC*, XEN3*, ...)?

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
Here is a patch that makes encrypted swap (but not rijndael) optional: http://ftp.netbsd.org/pub/NetBSD/misc/mlelstv/uvm_swap.diff That looks good to me, too. I guess you should also add VM_SWAPCRYPT option to various kernels (GENERIC*, XEN3*, ...)? Please commit.

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Michael van Elst wrote: On Sun, May 10, 2020 at 07:54:06AM -0700, Paul Goyette wrote: Prior to the encrypted-swap commit, a kernel configured without the ``pseudo-device crypto'' was able to link and run successfully. (Any attempt to use rijndael results in an auto-load

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Michael van Elst
On Sun, May 10, 2020 at 07:54:06AM -0700, Paul Goyette wrote: > Prior to the encrypted-swap commit, a kernel configured without the > ``pseudo-device crypto'' was able to link and run successfully. (Any > attempt to use rijndael results in an auto-load of the crypto module.) Even without

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Taylor R Campbell
> Date: Sun, 10 May 2020 13:44:49 - > From: mlel...@serpens.de (Michael van Elst) > > Here is a patch that makes encrypted swap (but not rijndael) optional: > > http://ftp.netbsd.org/pub/NetBSD/misc/mlelstv/uvm_swap.diff > > which might be useful to create small kernels (it's just about

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
On Sun, 10 May 2020, Michael van Elst wrote: p...@whooppee.com (Paul Goyette) writes: the kernel. Without this, the kernel fails to link, with undefined references to several symbols: rijndael_cipherInit rijndael_blockEncrypt rijndael_blockDecrypt

Re: Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Michael van Elst
p...@whooppee.com (Paul Goyette) writes: >the kernel. Without this, the kernel fails to link, with undefined >references to several symbols: > rijndael_cipherInit > rijndael_blockEncrypt > rijndael_blockDecrypt > rijndael_makeKey >Since the two crypto devices are

Build error when crypto and swcrypto are omitted from kernel

2020-05-10 Thread Paul Goyette
With the recent introduction of swap encryption, it is now mandatory to include "pseudo-device crypto" and/or "pseudo-device swcrypto" in the kernel. Without this, the kernel fails to link, with undefined references to several symbols: rijndael_cipherInit rijndael_blockEncrypt