2003-09-05 14:39:23

by Erlend Aasland

[permalink] [raw]
Subject: [CRYPTO] add alg. type to /proc/crypto output

Hi

Here is a patch that add alg. type output to /proc/crypto. Booted and
tested.


Erlend Aasland

diff -urN linux-2.6.0-test4-bk7/crypto/proc.c linux-2.6.0-test4-bk7-dirty/crypto/proc.c
--- linux-2.6.0-test4-bk7/crypto/proc.c 2003-08-23 01:53:43.000000000 +0200
+++ linux-2.6.0-test4-bk7-dirty/crypto/proc.c 2003-09-05 18:17:35.000000000 +0200
@@ -57,6 +57,7 @@

switch (alg->cra_flags & CRYPTO_ALG_TYPE_MASK) {
case CRYPTO_ALG_TYPE_CIPHER:
+ seq_printf(m, "type : cipher\n");
seq_printf(m, "blocksize : %u\n", alg->cra_blocksize);
seq_printf(m, "min keysize : %u\n",
alg->cra_cipher.cia_min_keysize);
@@ -65,10 +66,17 @@
break;

case CRYPTO_ALG_TYPE_DIGEST:
+ seq_printf(m, "type : digest\n");
seq_printf(m, "blocksize : %u\n", alg->cra_blocksize);
seq_printf(m, "digestsize : %u\n",
alg->cra_digest.dia_digestsize);
break;
+ case CRYPTO_ALG_TYPE_COMPRESS:
+ seq_printf(m, "type : compression\n");
+ break;
+ default:
+ seq_printf(m, "type : unknown\n");
+ break;
}

seq_putc(m, '\n');


2003-09-05 14:44:08

by David Miller

[permalink] [raw]
Subject: Re: [CRYPTO] add alg. type to /proc/crypto output

On Fri, 5 Sep 2003 16:38:59 +0200
Erlend Aasland <[email protected]> wrote:

> Here is a patch that add alg. type output to /proc/crypto. Booted and
> tested.

Is it even useful?

When you see names like "md5" and parameters such as "digestsize"
listed, do you really have no clue that it is a "digest"? :-)

The information seems completely redundant to me.

2003-09-06 10:08:25

by Erlend Aasland

[permalink] [raw]
Subject: Re: [CRYPTO] add alg. type to /proc/crypto output

On 09/05/03 07:34, David S. Miller wrote:
> Is it even useful?
Check http://samba.org/~jamesm/crypto/TODO
It's listed under medium priority, so I thought I was doing a good
thing implementing it.

>
> When you see names like "md5" and parameters such as "digestsize"
> listed, do you really have no clue that it is a "digest"? :-)
Good point.

2003-09-06 16:44:33

by James Morris

[permalink] [raw]
Subject: Re: [CRYPTO] add alg. type to /proc/crypto output

On Sat, 6 Sep 2003, Erlend Aasland wrote:

> On 09/05/03 07:34, David S. Miller wrote:
> > Is it even useful?
> Check http://samba.org/~jamesm/crypto/TODO
> It's listed under medium priority, so I thought I was doing a good
> thing implementing it.

I think someone asked for it once, not sure exactly why, but I guess it
makes parsing simpler and theoretically less likely to break if e.g. a
field is added or removed for an algorithm type.


- James
--
James Morris
<[email protected]>

2003-09-07 00:56:45

by Valdis Klētnieks

[permalink] [raw]
Subject: Re: [CRYPTO] add alg. type to /proc/crypto output

On Sat, 06 Sep 2003 12:08:18 +0200, Erlend Aasland said:
> On 09/05/03 07:34, David S. Miller wrote:

> > When you see names like "md5" and parameters such as "digestsize"
> > listed, do you really have no clue that it is a "digest"? :-)
> Good point.

On the flip side, it makes it possible for an *AUTOMATED* process to search for
appropriate types of entries. So for instance, a GUI can say "The following
3 digest functions are available, please choose one" or "The following 4 symmetric
encryptions are available".....


Attachments:
(No filename) (226.00 B)

2003-09-11 23:46:29

by David Miller

[permalink] [raw]
Subject: Re: [CRYPTO] add alg. type to /proc/crypto output

On Fri, 5 Sep 2003 16:38:59 +0200
Erlend Aasland <[email protected]> wrote:

> Here is a patch that add alg. type output to /proc/crypto. Booted and
> tested.

I've applied this, thank you.