2011-07-28 12:03:17

by Ralf Jung

[permalink] [raw]
Subject: cryptsetup: "FATAL" padlock_{aes,sha} error during boot

Hi,

as per request of Jonathan at http://bugs.debian.org/cgi-
bin/bugreport.cgi?bug=485070, I am reporting the mentioned bug to this list as
well:

With current Debian testing (Kernel 2.6.39), I am getting this error on each
boot:
FATAL: Error inserting padlock_sha (/lib/modules/2.6.39-2-
amd64/kernel/drivers/crypto/padlock-sha.ko): No such device
I understand it is harmless, but it should still be fixed since it irritates
users suggesting something is seriously going wrong.

No patch though, sorry, compiling and botting a custom kernel still sounds
like black magic for me ;-)

Kind regards,
Ralf


2011-07-30 06:13:12

by Herbert Xu

[permalink] [raw]
Subject: Re: cryptsetup: "FATAL" padlock_{aes,sha} error during boot

Ralf Jung <[email protected]> wrote:
> Hi,
>
> as per request of Jonathan at http://bugs.debian.org/cgi-
> bin/bugreport.cgi?bug=485070, I am reporting the mentioned bug to this list as
> well:
>
> With current Debian testing (Kernel 2.6.39), I am getting this error on each
> boot:
> FATAL: Error inserting padlock_sha (/lib/modules/2.6.39-2-
> amd64/kernel/drivers/crypto/padlock-sha.ko): No such device
> I understand it is harmless, but it should still be fixed since it irritates
> users suggesting something is seriously going wrong.

That message comes from user-space and needs to be fixed there.

Cheers,
--
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

2011-07-30 09:15:15

by Ben Hutchings

[permalink] [raw]
Subject: Re: Bug#485070: cryptsetup: "FATAL" padlock_{aes,sha} error during boot

Version: 3.7-pre2-1

On Sat, 2011-07-30 at 14:13 +0800, Herbert Xu wrote:
> Ralf Jung <[email protected]> wrote:
> > Hi,
> >
> > as per request of Jonathan at http://bugs.debian.org/cgi-
> > bin/bugreport.cgi?bug=485070, I am reporting the mentioned bug to this list as
> > well:
> >
> > With current Debian testing (Kernel 2.6.39), I am getting this error on each
> > boot:
> > FATAL: Error inserting padlock_sha (/lib/modules/2.6.39-2-
> > amd64/kernel/drivers/crypto/padlock-sha.ko): No such device
> > I understand it is harmless, but it should still be fixed since it irritates
> > users suggesting something is seriously going wrong.
>
> That message comes from user-space and needs to be fixed there.

Right, it comes from modprobe.

Fixed in module-init-tools version 3.5, commit
cdafffbadc5831ad56fe00ecd420ca9c36e9474f ("don't warn noisely if loading
multiple module aliases").

Ben.


Attachments:
signature.asc (828.00 B)
This is a digitally signed message part

2011-07-30 11:50:33

by Jonathan Nieder

[permalink] [raw]
Subject: [PATCH] crypto: padlock-aes - Make module loading even quieter when hardware is missing

When loading aes via the module alias, a padlock module failing to
load due to missing hardware is not particularly notable. With
v2.6.27-rc1~1107^2~14 (crypto: padlock - Make module loading quieter
when hardware isn't available, 2008-07-03), the padlock-aes module
suppresses the relevant messages when the "quiet" flag is in use; but
better to suppress this particular message completely, since the
administrator can already distinguish such errors by the absence of a
message indicating initialization failing or succeeding.

This avoids occasional messages in syslog of the form

padlock_aes: VIA PadLock not detected.

Signed-off-by: Jonathan Nieder <[email protected]>
---
Herbert Xu wrote:
> Ralf Jung <[email protected]> wrote:

>> With current Debian testing (Kernel 2.6.39), I am getting this error on each
>> boot:
>> FATAL: Error inserting padlock_sha (/lib/modules/2.6.39-2-
>> amd64/kernel/drivers/crypto/padlock-sha.ko): No such device
[...]
> That message comes from user-space and needs to be fixed there.

Thanks. Indeed, I was sloppy when reading the original report and
thought he was talking about a different message. Sorry for the
noise.

drivers/crypto/padlock-aes.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/drivers/crypto/padlock-aes.c b/drivers/crypto/padlock-aes.c
index db33d300..29b9469f 100644
--- a/drivers/crypto/padlock-aes.c
+++ b/drivers/crypto/padlock-aes.c
@@ -508,10 +508,8 @@ static int __init padlock_init(void)
int ret;
struct cpuinfo_x86 *c = &cpu_data(0);

- if (!cpu_has_xcrypt) {
- printk(KERN_NOTICE PFX "VIA PadLock not detected.\n");
+ if (!cpu_has_xcrypt)
return -ENODEV;
- }

if (!cpu_has_xcrypt_enabled) {
printk(KERN_NOTICE PFX "VIA PadLock detected, but not enabled. Hmm, strange...\n");
--
1.7.6

2011-08-03 06:08:13

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH] crypto: padlock-aes - Make module loading even quieter when hardware is missing

On Sat, Jul 30, 2011 at 01:50:12PM +0200, Jonathan Nieder wrote:
> When loading aes via the module alias, a padlock module failing to
> load due to missing hardware is not particularly notable. With
> v2.6.27-rc1~1107^2~14 (crypto: padlock - Make module loading quieter
> when hardware isn't available, 2008-07-03), the padlock-aes module
> suppresses the relevant messages when the "quiet" flag is in use; but
> better to suppress this particular message completely, since the
> administrator can already distinguish such errors by the absence of a
> message indicating initialization failing or succeeding.
>
> This avoids occasional messages in syslog of the form
>
> padlock_aes: VIA PadLock not detected.
>
> Signed-off-by: Jonathan Nieder <[email protected]>


Patch applied. Thanks Jonathan!
--
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt