From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 19 Nov 2002 01:22:31 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 19 Nov 2002 01:22:31 -0500 Received: from parcelfarce.linux.theplanet.co.uk ([195.92.249.252]:64262 "EHLO www.linux.org.uk") by vger.kernel.org with ESMTP id ; Tue, 19 Nov 2002 01:22:24 -0500 Message-ID: <3DD9DA23.1070102@pobox.com> Date: Tue, 19 Nov 2002 01:28:51 -0500 From: Jeff Garzik User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021018 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Andre Hedrick CC: Douglas Gilbert , "J. E. J. Bottomley" , Linus Torvalds , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: linux-2.4.18-modified-scsi-h.patch References: In-Reply-To: Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Andre Hedrick wrote: > Greetings Doug et al. > > Please consider the addition of this simple void ptr to the scsi_request > struct. The addition of this simple void pointer allows one to map any > and all request execution caller the facility to search for a specific > operation without having to run in circles. Hunting for these details > over the global device list of all HBA's is silly and one of the key > reasons why there error recovery path is so painful. > > > Scsi_Request *req = sc_cmd->sc_request; > blah_blah_t *trace = NULL; > > trace = (blah_blah_t *)req->trace_ptr; > > > Therefore the specific transport invoking operations via the midlayer will > have the ablity to track and trace any operation. > > It will save everyone headaches. > > Cheers, > > > Andre Hedrick, CTO & Founder > iSCSI Software Solutions Provider > http://www.PyXTechnologies.com/ > > > ------------------------------------------------------------------------ > > --- linux/drivers/scsi/scsi.h.orig 2002-10-31 01:45:39.000000000 -0800 > +++ linux/drivers/scsi/scsi.h 2002-10-31 01:46:31.000000000 -0800 > @@ -667,8 +667,11 @@ > unsigned short sr_sglist_len; /* size of malloc'd scatter-gather list */ > unsigned sr_underflow; /* Return error if less than > this amount is transferred */ > + void *trace_ptr; /* capable of cmd-cmnd-error tracing */ ok > }; > > +#define MODIFIED_SCSI_H This falls into C style :) Instead of this I would do #define HAVE_TRACE_PTR 1 just like we already do HAVE_xxx in include/linux/netdevice.h and other places. Jeff