From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH] use mmiowb in qla1280.c Date: Mon, 25 Oct 2004 09:18:01 -0700 Sender: linux-kernel-owner@vger.kernel.org Message-ID: <200410250918.01786.jbarnes@engr.sgi.com> References: <200410211613.19601.jbarnes@engr.sgi.com> <200410211617.14809.jbarnes@engr.sgi.com> <1098634812.10906.38.camel@mulgrave> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1098634812.10906.38.camel@mulgrave> Content-Disposition: inline To: James Bottomley Cc: Andrew Morton , Linux Kernel , SCSI Mailing List , jeremy@sgi.com, jes@sgi.com List-Id: linux-scsi@vger.kernel.org On Sunday, October 24, 2004 9:20 am, James Bottomley wrote: > On Thu, 2004-10-21 at 19:17, Jesse Barnes wrote: > > There are a few spots in qla1280.c that don't need a full PCI write flush > > to the device, but rather a simple write ordering guarantee. This patch > > changes some of the PIO reads that cause write flushes into mmiowb calls > > instead, which is a lighter weight way of ensuring ordering. > > > > Jes and James, can you ack this and/or push it in via the SCSI BK tree? > > This doesn't seem to work: > > CC [M] drivers/scsi/qla1280.o > drivers/scsi/qla1280.c: In function `qla1280_64bit_start_scsi': > drivers/scsi/qla1280.c:3404: warning: implicit declaration of function > `mmiowb' > > MODPOST > *** Warning: "mmiowb" [drivers/scsi/qla1280.ko] undefined! Only works in Andrew's tree until he pushes mmiowb to Linus. Jesse