Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761293AbZCSUlv (ORCPT ); Thu, 19 Mar 2009 16:41:51 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1759865AbZCSUlN (ORCPT ); Thu, 19 Mar 2009 16:41:13 -0400 Received: from systemlinux.org ([83.151.29.59]:59394 "EHLO m18s25.vlinux.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757018AbZCSUlM (ORCPT ); Thu, 19 Mar 2009 16:41:12 -0400 Date: Thu, 19 Mar 2009 21:10:08 +0100 From: Andre Noll To: Dan Williams Cc: linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org, neilb@suse.de, maciej.sosnowski@intel.com, Yuri Tikhonov , Ilya Yanok Subject: Re: [PATCH 02/13] async_tx: don't use src_list argument of async_xor() for dma addresses Message-ID: <20090319201008.GC10491@skl-net.de> References: <20090318191248.20375.40560.stgit@dwillia2-linux.ch.intel.com> <20090318192026.20375.3053.stgit@dwillia2-linux.ch.intel.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="B4IIlcmfBL/1gGOG" Content-Disposition: inline In-Reply-To: <20090318192026.20375.3053.stgit@dwillia2-linux.ch.intel.com> User-Agent: Mutt/1.5.9i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1270 Lines: 40 --B4IIlcmfBL/1gGOG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 12:20, Dan Williams wrote: > Using src_list argument of async_xor() as a storage for dma addresses > implies sizeof(dma_addr_t) <=3D sizeof(struct page *) restriction which is > not always true (e.g. ppc440spe). This message does not tell what the patch is going to do about that. It would also be nice to mention (either in the log message or as a comment) why allocating the dma_src arrays on the stack is not going to be a problem. Andre --=20 The only person who always got his work done by Friday was Robinson Crusoe --B4IIlcmfBL/1gGOG Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.1 (GNU/Linux) iD8DBQFJwqagWto1QDEAkw8RAol7AJ9jG26hiovmH4WW54bhvV1PctW/LQCeOxo7 a4ILQjqS5mMDQoLby5qlPvs= =mIQl -----END PGP SIGNATURE----- --B4IIlcmfBL/1gGOG-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/