public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Re: PATCH: cdrecord: avoiding scsi device numbering for ide devic es
@ 2004-08-05 16:00 Kern Alexander
  2004-08-09 18:38 ` Jens Axboe
  0 siblings, 1 reply; 2+ messages in thread
From: Kern Alexander @ 2004-08-05 16:00 UTC (permalink / raw)
  To: 'linux-kernel@vger.kernel.org'; +Cc: 'alex.kern@gmx.de'

Jens Axboe wrote:

> On Thu, Aug 05 2004, Joerg Schilling wrote:
> 
>>>From: Jens Axboe <axboe@suse.de>
>>
>>>ATA method is misnamed, it's really SG_IO that is used. And you want to
>>>use that regardless of the device type, SCSI or ATAPI. There's no such
>>>thing as an ATA burner, and there's no need to differentiate between
>>>SCSI or ATAPI CD-ROM's when burning - SG_IO is the method to use. So
>>>forget browsing /proc/ide and other hacks.
>>
>>I am sorry but as Linux already has 6 different interfaces for sending 
>>Generic SCSI commands and thus, we are running out of names.
>>
>>Let me give you an advise: consolidate Linux so that is does only need
>>/dev/sg and fix the bugs in ide-scsi instead of constantly inventing new
>>unneeded interfaces.
> 
> 
> That's been the general direction for quite some time, just that SG_IO
> is the preferred method since that works all around. You were the one
> that merged support for the CDROM_SEND_PACKET interface, which has
> _never_ been advertised as a way to burn CDs in Linux. I'd suggest you
> remove that.
> 
Silly, as I suggested a patch to Joerg, it was the uniquely ability to burn
without
ide-scsi. And known you, it simply works, it let me to scan for burners(what
SG_IO cannot), it works
in 2.4.X and 2.6.X.

Make SG_IO better and CDROM_SEND_PACKET will die, without your suggestions.

Regards
Alex

P.S. I'm know that CDROM_SEND_PACKET has a overhead, by me it's 4%. I can
live with it.

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

* Re: PATCH: cdrecord: avoiding scsi device numbering for ide devic es
  2004-08-05 16:00 PATCH: cdrecord: avoiding scsi device numbering for ide devic es Kern Alexander
@ 2004-08-09 18:38 ` Jens Axboe
  0 siblings, 0 replies; 2+ messages in thread
From: Jens Axboe @ 2004-08-09 18:38 UTC (permalink / raw)
  To: Kern Alexander
  Cc: 'linux-kernel@vger.kernel.org',
	'alex.kern@gmx.de'


(if you don't cc me, I don't see your mail. it's been 4 days now. always
cc on linux-kernel, it's the etiquette!)

On Thu, Aug 05 2004, Kern Alexander wrote:
> Jens Axboe wrote:
> 
> > On Thu, Aug 05 2004, Joerg Schilling wrote:
> > 
> >>>From: Jens Axboe <axboe@suse.de>
> >>
> >>>ATA method is misnamed, it's really SG_IO that is used. And you want to
> >>>use that regardless of the device type, SCSI or ATAPI. There's no such
> >>>thing as an ATA burner, and there's no need to differentiate between
> >>>SCSI or ATAPI CD-ROM's when burning - SG_IO is the method to use. So
> >>>forget browsing /proc/ide and other hacks.
> >>
> >>I am sorry but as Linux already has 6 different interfaces for sending 
> >>Generic SCSI commands and thus, we are running out of names.
> >>
> >>Let me give you an advise: consolidate Linux so that is does only need
> >>/dev/sg and fix the bugs in ide-scsi instead of constantly inventing new
> >>unneeded interfaces.
> > 
> > 
> > That's been the general direction for quite some time, just that SG_IO
> > is the preferred method since that works all around. You were the one
> > that merged support for the CDROM_SEND_PACKET interface, which has
> > _never_ been advertised as a way to burn CDs in Linux. I'd suggest you
> > remove that.
> > 
> Silly, as I suggested a patch to Joerg, it was the uniquely ability to
> burn without ide-scsi. And known you, it simply works, it let me to
> scan for burners(what SG_IO cannot), it works in 2.4.X and 2.6.X.

Yes it simply works, as long as it works...

> Make SG_IO better and CDROM_SEND_PACKET will die, without your
> suggestions.

SG_IO is perfect as is, what problems are you referring to?

> P.S. I'm know that CDROM_SEND_PACKET has a overhead, by me it's 4%. I
> can live with it.

In 2.6, there is basically zero extra overhead with CDROM_SEND_PACKET,
since it's just a small wrapper on top of SG_IO (see scsi_ioctl.c if you
are curious), so you cannot measure a performance difference between the
two there.

In 2.4 it's orders of magnitude slower, since it will not use
CDROM_SEND_PACKET and it puts unnecessary strain on the memory allocator
to larger chunks of physically contigous memory. Note that you will not
see interrupt handler load if you are only doing casual performance
monitoring.

-- 
Jens Axboe


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

end of thread, other threads:[~2004-08-09 18:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-08-05 16:00 PATCH: cdrecord: avoiding scsi device numbering for ide devic es Kern Alexander
2004-08-09 18:38 ` Jens Axboe

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