On Thu, Apr 08, 2021 at 08:11:34AM -0700, Eric Biggers wrote: > On Thu, Apr 08, 2021 at 07:58:08PM +0800, Hangbin Liu wrote: > > On Thu, Apr 08, 2021 at 09:06:52AM +0800, Hangbin Liu wrote: > > > > Also, couldn't you just consider WireGuard to be outside your FIPS module > > > > boundary, which would remove it from the scope of the certification? > > > > > > > > And how do you handle all the other places in the kernel that use ChaCha20 and > > > > SipHash? For example, drivers/char/random.c? > > > > > > Good question, I will check it and reply to you later. > > > > I just read the code. The drivers/char/random.c do has some fips specific > > parts(seems not related to crypto). After commit e192be9d9a30 ("random: replace > > non-blocking pool with a Chacha20-based CRNG") we moved part of chacha code to > > lib/chacha20.c and make that code out of control. > > > So you are saying that you removed drivers/char/random.c and lib/chacha20.c from > your FIPS module boundary? Why not do the same for WireGuard? No, I mean this looks like a bug (using not allowed crypto in FIPS mode) and we should fix it. Thanks Hangbin