linux-kernel - Re: [PATCH v2 2/2] Crypto: Talitos: Support for Async

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <2868C8CF-584B-4FA7-9C3B-2FACEF77527E@kernel.crashing.org>
Date:	Wed, 16 Dec 2009 16:38:50 -0600
From:	Kumar Gala <galak@...nel.crashing.org>
To:	Kim Phillips <Kim.Phillips@...escale.com>
Cc:	Vishnu Suresh <Vishnu@...escale.com>, herbert@...dor.apana.org.au,
	B04825@...escale.com, linux-kernel@...r.kernel.org,
	linux-raid@...r.kernel.org, linuxppc-dev@...abs.org,
	linux-crypto@...r.kernel.org,
	Dipen Dudhat <Dipen.Dudhat@...escale.com>,
	dan.j.williams@...el.com,
	Maneesh Gupta <Maneesh.Gupta@...escale.com>,
	R58472@...escale.com
Subject: Re: [PATCH v2 2/2] Crypto: Talitos: Support for Async_tx XOR offload


On Dec 16, 2009, at 4:41 PM, Kim Phillips wrote:

> On Wed, 16 Dec 2009 21:04:58 +0530
> Vishnu Suresh <Vishnu@...escale.com> wrote:
> 
>> Expose Talitos's XOR functionality to be used for
>> RAID Parity calculation via the Async_tx layer.
>> 
>> Known Issue:
>> When used with fsldma, random crashes are observed
>> on some platforms. Hence, inter-operability with fsldma
>> is currently disabled
>> 
>> Thanks to Surender Kumar and Lee Nipper for their help in
>> realising this driver
>> 
>> Signed-off-by: Kim Phillips <kim.phillips@...escale.com>
>> Signed-off-by: Dipen Dudhat <Dipen.Dudhat@...escale.com>
>> Signed-off-by: Maneesh Gupta <Maneesh.Gupta@...escale.com>
>> Signed-off-by: Vishnu Suresh <Vishnu@...escale.com>
>> ---
>> Changes with respect to v1 as per comments received
>> o. Rebased to linux-next as of 20091216
>> o. The selection is based exclusive of fsldma
>> o. Intoduced a new Kernel Configuration variable
>>   *. This enables selecting the Cryptographic functionality
>>      of Talitos along with fsldma.
>>   *. Disables the XOR parity calculation offload, if fsldma enabled
>>      either as kernel in-built or as a module
>>   *. Once the inter-operability with fsldma is resolved, this option
>>      can be removed
> 
> wait, why can't the interoperability bug be fixed in the first place?

I agree w/Kim.  We need to better understand what the bug is and how to reproduce it so we can get to the root cause.

Paper taping over it by disabling fsldma is not the right solution.

- k--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/