Bug 107341 - /sbin/modprobe ghash_clmulni_intel - invalid argument
Summary: /sbin/modprobe ghash_clmulni_intel - invalid argument
Status: NEW
Alias: None
Product: Other
Classification: Unclassified
Component: Modules (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: other_modules
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-06 06:00 UTC by KernelBug
Modified: 2016-07-01 04:03 UTC (History)
1 user (show)

See Also:
Kernel Version: 4.1.12
Subsystem:
Regression: No
Bisected commit-id:


Attachments
config-4.1.10 (94.26 KB, application/octet-stream)
2015-11-06 06:00 UTC, KernelBug
Details
patch (10.00 KB, application/octet-stream)
2015-12-07 03:11 UTC, Rui Wang
Details

Description KernelBug 2015-11-06 06:00:20 UTC
Created attachment 192221 [details]
config-4.1.10

When trying to load /sbin/modprobe ghash_clmulni_intel it gives back at the terminal; invalid argument

This does not happen for me using 4.1.10

There seems to be a bug that has been introduced here?

I'm attaching my 4.1.10 config, it's the same that I used for 4.1.12
Comment 1 KernelBug 2015-11-10 08:28:20 UTC
This is also happening in 4.1.13
Comment 2 Rui Wang 2015-12-01 05:03:54 UTC
Please help test the fix at: https://lkml.org/lkml/2015/11/29/85
This could be a dup of Bug 108151

Thanks
Comment 3 KernelBug 2015-12-06 07:35:24 UTC
When applying the patches I'm getting this output;

patch unexpectedly ends in middle of line


patching file crypto/cryptd.c
Hunk #1 succeeded at 628 (offset -9 lines).
patch unexpectedly ends in middle of line


patching file arch/x86/crypto/ghash-clmulni-intel_glue.c
patch unexpectedly ends in middle of line


Can we get these cleaned up?

thanks
Comment 4 Rui Wang 2015-12-07 03:11:36 UTC
Created attachment 196501 [details]
patch

The patch to fix the bug.
Comment 5 Rui Wang 2015-12-07 03:13:48 UTC
(In reply to KernelBug from comment #3)
> When applying the patches I'm getting this output;
> 
> patch unexpectedly ends in middle of line
> 
> 
> patching file crypto/cryptd.c
> Hunk #1 succeeded at 628 (offset -9 lines).
> patch unexpectedly ends in middle of line
> 
> 
> patching file arch/x86/crypto/ghash-clmulni-intel_glue.c
> patch unexpectedly ends in middle of line
> 
> 
> Can we get these cleaned up?
> 
> thanks

It's probably because you've cut/paste the code in Windows and it contains DOS CRLF. I've attached the patch. (The code is now already merged in the crypto dev git tree)
Comment 6 KernelBug 2015-12-07 07:08:49 UTC
I clicked on your name listed twice, then I clicked; 'get diff 1' then it loaded the patch and I then did 'ctrl s' to download it, and I renamed it blahblah.diff 


Hmmm
Comment 7 KernelBug 2015-12-07 09:14:46 UTC
The patch appears to be working!

No smoke seen yet! LOL :)
Comment 8 KernelBug 2015-12-07 09:15:06 UTC
by the way THANKS!
Comment 9 KernelBug 2015-12-12 10:37:46 UTC
This is still a problem in 4.1.14.

I hope this is going to get committed soon to LTS?
Comment 10 KernelBug 2015-12-18 05:28:37 UTC
Actually this just needs to get fixed in the latest kernels.

Seen this in 4.3.2 also...
Comment 11 Rui Wang 2015-12-18 06:08:01 UTC
It's already merged in the crypto maintainer Herbert Xu's tree: http://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git

It'll take some time to get into Linus's tree.
Comment 12 KernelBug 2016-01-27 03:40:16 UTC
It also just appeared now in 4.1.16
Comment 13 KernelBug 2016-06-05 02:18:53 UTC
Can someone please tell me what kernel version this is being merged into?

The patch also needs applying in 4.4x

thanks
Comment 14 Rui Wang 2016-06-06 02:22:57 UTC
Linux 4.4-rc4

commit 1a07834024dfca5c4bed5de8f8714306e0a11836
Author: Wang, Rui Y <rui.y.wang@intel.com>
Date:   Sun Nov 29 22:45:34 2015 +0800

    crypto: cryptd - Assign statesize properly


commit 3a020a723c65eb8ffa7c237faca26521a024e582
Author: Wang, Rui Y <rui.y.wang@intel.com>
Date:   Sun Nov 29 22:45:33 2015 +0800

    crypto: ghash-clmulni - Fix load failure
Comment 15 KernelBug 2016-06-29 00:33:36 UTC
@ Rui Wang, I don't understand your reply?

All I know is you committed this, but what kernel version(s) did it take effect in?

Because I still needed to apply the patches in 4.4.12...
Comment 16 Rui Wang 2016-06-29 00:52:00 UTC
It's in upstream kernel from git.kernel.org (v4.4-rc4). But it may not be in a stable kernel or a distribution like Ubuntu.
Comment 17 KernelBug 2016-07-01 04:03:48 UTC
Will it eventually be merged to a stable kernel?

Note You need to log in before you can comment on or make changes to this bug.