From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 3/4] scsi_data_buffer Date: Mon, 12 Nov 2007 23:07:03 -0800 Message-ID: <20071112230703.f41309cc.akpm@linux-foundation.org> References: <47333E2A.7020307@panasas.com> <47334072.7030908@panasas.com> <20071112220652.dd9562c5.akpm@linux-foundation.org> <20071113154309K.tomof@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from smtp2.linux-foundation.org ([207.189.120.14]:56150 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750765AbXKMHHu (ORCPT ); Tue, 13 Nov 2007 02:07:50 -0500 In-Reply-To: <20071113154309K.tomof@acm.org> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: FUJITA Tomonori Cc: bharrosh@panasas.com, fujita.tomonori@lab.ntt.co.jp, James.Bottomley@SteelEye.com, linux-scsi@vger.kernel.org, pw@osc.edu, bhalevy@panasas.com, jes@sgi.com On Tue, 13 Nov 2007 15:40:42 +0900 FUJITA Tomonori wrote: > On Mon, 12 Nov 2007 22:06:52 -0800 > Andrew Morton wrote: > > > On Thu, 08 Nov 2007 18:59:30 +0200 Boaz Harrosh wrote: > > > > > In preparation for bidi we abstract all IO members of scsi_cmnd, > > > that will need to duplicate, into a substructure. > > > > > > - Group all IO members of scsi_cmnd into a scsi_data_buffer > > > structure. > > > > drivers/scsi/qla1280.c: In function 'qla1280_done': > > drivers/scsi/qla1280.c:1313: error: 'struct scsi_cmnd' has no member named 'use_sg' > > drivers/scsi/qla1280.c:1314: error: 'struct scsi_cmnd' has no member named 'request_buffer' > > drivers/scsi/qla1280.c:1315: error: 'struct scsi_cmnd' has no member named 'use_sg' > > drivers/scsi/qla1280.c:1316: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > drivers/scsi/qla1280.c:1318: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > drivers/scsi/qla1280.c: In function 'qla1280_return_status': > > drivers/scsi/qla1280.c:1409: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > drivers/scsi/qla1280.c:1416: error: 'struct scsi_cmnd' has no member named 'resid' > > drivers/scsi/qla1280.c: In function 'qla1280_64bit_start_scsi': > > drivers/scsi/qla1280.c:2791: error: 'struct scsi_cmnd' has no member named 'use_sg' > > drivers/scsi/qla1280.c:2792: error: 'struct scsi_cmnd' has no member named 'request_buffer' > > drivers/scsi/qla1280.c:2793: error: 'struct scsi_cmnd' has no member named 'use_sg' > > drivers/scsi/qla1280.c:2801: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > drivers/scsi/qla1280.c:2896: error: 'struct scsi_cmnd' has no member named 'use_sg' > > drivers/scsi/qla1280.c:2991: error: 'struct scsi_cmnd' has no member named 'request_buffer' > > drivers/scsi/qla1280.c:2992: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > drivers/scsi/qla1280.c:3004: error: 'struct scsi_cmnd' has no member named 'request_bufflen' > > make[2]: *** [drivers/scsi/qla1280.o] Error 1 > > > > It mystfies me how a patch like this can have been floating about in N > > submissions across M months and nobody has done an allmodconfig build or > > even a grep to find out what broke. > > > > ho hum. I shall mark qla1280 BROKEN and shall plod onwards. > > A patch to fix this is in James' scsi-pending tree. Jes tested and > fixed it (thanks !) so it will go to -mm via scsi-misc soon. oh gawd. So we have git-scsi-misc, git-scsi-rc-fixes and now git-scsi-pending? I hope you fixed imm, ppa and any other broken drivers? > Boaz, it's better to send major scsi patches to -mm via scsi-misc to > avoid problems like this. > > > By the way, Andrew, can you add the following patchset to -mm? > > http://lkml.org/lkml/2007/10/24/138 > > It fixes the IOMMUs' problem to merge scatter/gather segments without > considering LLDs' restrictions. hmm, OK, I saved them away to look at after next -mm.