Re: [PATCH 1/5] 1.be_le.patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 20, 2005 at 01:59:23PM +0300, Denis Vlasenko wrote:
>
> @@ -265,10 +263,10 @@ aes_set_key(void *ctx_arg, const u8 *in_
>  
>  	ctx->key_length = key_len;
>  
> -	E_KEY[0] = u32_in (in_key);
> -	E_KEY[1] = u32_in (in_key + 4);
> -	E_KEY[2] = u32_in (in_key + 8);
> -	E_KEY[3] = u32_in (in_key + 12);
> +	E_KEY[0] = load_le32(in_key,0);
> +	E_KEY[1] = load_le32(in_key,1);
> +	E_KEY[2] = load_le32(in_key,2);
> +	E_KEY[3] = load_le32(in_key,3);

Please insert a space after the comma.
  
> @@ -448,4 +446,3 @@ module_exit(aes_fini);
>  
>  MODULE_DESCRIPTION("Rijndael (AES) Cipher Algorithm");
>  MODULE_LICENSE("Dual BSD/GPL");
> -

Please drop unrelated hunks like this.

> @@ -1159,9 +1114,7 @@ not_weak:
>  		w  |= (b1[k[18+24]] | b0[k[19+24]]) << 4;
>  		w  |= (b1[k[20+24]] | b0[k[21+24]]) << 2;
>  		w  |=  b1[k[22+24]] | b0[k[23+24]];
> -		
> -		ROR(w, 4, 28);      /* could be eliminated */
> -		expkey[1] = w;
> +		expkey[1] = ror32(w, 4);	/* could be eliminated */

Please split the ROR => ror32 change out.

> diff -urpN linux-2.6.12.0.orig/crypto/helper.h linux-2.6.12.1.n/crypto/helper.h
> --- linux-2.6.12.0.orig/crypto/helper.h	Thu Jan  1 03:00:00 1970
> +++ linux-2.6.12.1.n/crypto/helper.h	Sun Jun 19 18:51:23 2005

This should probably be moved to include/linux.  There's crypto
stuff living under arch/ and drivers/crypto.

This patch is really good idea overall.

Thanks,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux