public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [bk patches] move scsi headers
@ 2003-05-27  3:42 Jeff Garzik
  2003-05-27  5:14 ` Christoph Hellwig
  0 siblings, 1 reply; 5+ messages in thread
From: Jeff Garzik @ 2003-05-27  3:42 UTC (permalink / raw)
  To: linux-scsi


James, please do a

	bk pull bk://kernel.bkbits.net/jgarzik/scsi-misc-2.5

This will update the following files:

 drivers/scsi/hosts.h          |  603 ------------------------------------
 drivers/scsi/scsi.h           |  690 -----------------------------------------
 drivers/scsi/scsi_obsolete.h  |  106 ------
 drivers/scsi/hosts.h          |    6 
 drivers/scsi/scsi.h           |    6 
 drivers/scsi/scsi_obsolete.h  |    6 
 drivers/scsi/st.h             |    4 
 include/linux/scsi_defs.h     |  692 +++++++++++++++++++++++++++++++++++++++++-
 include/linux/scsi_hosts.h    |  603 ++++++++++++++++++++++++++++++++++++
 include/linux/scsi_obsolete.h |  106 ++++++
 10 files changed, 1419 insertions(+), 1403 deletions(-)

through these ChangeSets:

<jgarzik@redhat.com> (03/05/26 1.1195)
   [scsi] add drivers/scsi/{scsi,hosts,scsi_obsolete}.h wrappers
   
   Also, remove "#ifndef _SCSI_H" from st.h, and
   include linux/scsi_obsolete.h rather than "scsi_obsolete.h"
   in linux/scsi_defs.h.

<jgarzik@redhat.com> (03/05/26 1.1194)
   [scsi] mv drv/scsi/{scsi,hosts,scsi_obsolete}.h to include/linux
   
   New names:
   include/linux/scsi_hosts.h
   include/linux/scsi_defs.h
   include/linux/scsi_obsolete.h


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [bk patches] move scsi headers
  2003-05-27  3:42 [bk patches] move scsi headers Jeff Garzik
@ 2003-05-27  5:14 ` Christoph Hellwig
  2003-05-27  5:19   ` Jeff Garzik
  0 siblings, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2003-05-27  5:14 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: linux-scsi

On Mon, May 26, 2003 at 11:42:39PM -0400, Jeff Garzik wrote:
> 
> James, please do a
> 
> 	bk pull bk://kernel.bkbits.net/jgarzik/scsi-misc-2.5

Also I already told you that the header organisation is crap.  include/scsi
exists for a reason, the scsi.h/host.h split is broken and scsi_obsolete.h
should go away not moved around.

If you want to do something useful you could start removing scsi_obsolete.h,
make the scsi core and scsi.h/hosts.h clean of the obsolete typedefs first.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [bk patches] move scsi headers
  2003-05-27  5:14 ` Christoph Hellwig
@ 2003-05-27  5:19   ` Jeff Garzik
  2003-05-27  5:35     ` Christoph Hellwig
  0 siblings, 1 reply; 5+ messages in thread
From: Jeff Garzik @ 2003-05-27  5:19 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: linux-scsi

Christoph Hellwig wrote:
> On Mon, May 26, 2003 at 11:42:39PM -0400, Jeff Garzik wrote:
> 
>>James, please do a
>>
>>	bk pull bk://kernel.bkbits.net/jgarzik/scsi-misc-2.5
> 
> 
> Also I already told you that the header organisation is crap.  include/scsi
> exists for a reason, the scsi.h/host.h split is broken and scsi_obsolete.h
> should go away not moved around.
> 
> If you want to do something useful you could start removing scsi_obsolete.h,
> make the scsi core and scsi.h/hosts.h clean of the obsolete typedefs first.


This is too ambitious this close to 2.6-pre, and there is also 
disagreement about use of include/scsi.  I took the minimal patch.

As we are very close to 2.6-pre, your suggestions may even want waiting 
until 2.7.  I'll leave it up to James...

Consider this a first step.  The world does not change in a day :)

	Jeff




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [bk patches] move scsi headers
  2003-05-27  5:19   ` Jeff Garzik
@ 2003-05-27  5:35     ` Christoph Hellwig
  2003-05-27  5:40       ` Jeff Garzik
  0 siblings, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2003-05-27  5:35 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: linux-scsi

On Tue, May 27, 2003 at 01:19:36AM -0400, Jeff Garzik wrote:
>This is too ambitious this close to 2.6-pre,

No, it's not.  Eihet we do it properly or not at all.  Having to use
different headers for 2.4,2.6and 2.7/2.8 is just extremly stupid.

Also we have more intrusive stuff in scsi pending, 2.6.pre or not.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [bk patches] move scsi headers
  2003-05-27  5:35     ` Christoph Hellwig
@ 2003-05-27  5:40       ` Jeff Garzik
  0 siblings, 0 replies; 5+ messages in thread
From: Jeff Garzik @ 2003-05-27  5:40 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: linux-scsi

Christoph Hellwig wrote:
> On Tue, May 27, 2003 at 01:19:36AM -0400, Jeff Garzik wrote:
> 
>>This is too ambitious this close to 2.6-pre,
> 
> 
> No, it's not.  Eihet we do it properly or not at all.  Having to use
> different headers for 2.4,2.6and 2.7/2.8 is just extremly stupid.

I'm going to send my patch to Marcelo too, since it doesn't break 
anything, and greatly reduces include annoyance for in-tree drivers 
outside drivers/scsi.


> Also we have more intrusive stuff in scsi pending, 2.6.pre or not.

We'll see what akpm says...

	Jeff





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2003-05-27  5:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-05-27  3:42 [bk patches] move scsi headers Jeff Garzik
2003-05-27  5:14 ` Christoph Hellwig
2003-05-27  5:19   ` Jeff Garzik
2003-05-27  5:35     ` Christoph Hellwig
2003-05-27  5:40       ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox