From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Dreier Subject: Re: [ofa-general] [PATCH 2.6.30] RDMA/cxgb3: Remove modulo math. Date: Wed, 11 Feb 2009 10:12:09 -0800 Message-ID: References: <20090210184448.22891.31130.stgit@dell3.ogc.int> <4992F26A.4030800@opengridcomputing.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: <4992F26A.4030800@opengridcomputing.com> (Steve Wise's message of "Wed, 11 Feb 2009 09:44:42 -0600") List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: general-bounces@lists.openfabrics.org Errors-To: general-bounces@lists.openfabrics.org To: Steve Wise Cc: randy.dunlap@oracle.com, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, general@lists.openfabrics.org List-Id: linux-next.vger.kernel.org > Note that wr->sg_list[i].addr was being cast to a u32. That was wrong. Is it possible for the page to be bigger than 4GB? If so then yes you might be chopping off high-order bits or something. Anyway please send me this change as a separate patch with a changelog explaining that you're avoiding the div etc.... I don't want to roll it in with the other unrelated fix (which changes code that was never upstream anyway).