Re: sense visible despite ide-floppy in 2.6 maybe

[Pat LaVarre <p.lavarre@ieee.org>]

Jens A:

> a) kernel.org 2.6.7-rc3 plus the patch of this thread,
> b) PATAPI drive connected as /dev/hdd,
> c) open of /dev/hdd with flags O_NONBLOCK|O_RDWR, ioctl SG_IO
> d) cdb = x 1B 00:00:00 00 00 // Stop
> e) cdb = x 1B 00:00:00 01 00 // Start

Yes now "Hn" (i.e. expecting no data) works here, every time I try.

Cool instant demo of how helpful linux-scsi folk are, thank you.

> For data transfer, it's more tricky. Well actually, if you restrict
> yourself to non-bio backed request, then it becomes pretty trivial!
> ide-floppy expects a contig buffer, a bio is scattered by nature.

Yes "Hi" (i.e. expecting data in) gives me zeroes.

> ...

Also something's broken (far far outside of the kernel) in plscsi.  I
found success only by patching the trivial sgio.c that I posted in this
thread, as shown below.

Pat LaVarre

--- sgio.c	2004-06-08 09:33:33.000000000 -0600
+++ sgio-up.c	2004-06-09 12:23:28.901649448 -0600
@@ -17,14 +17,14 @@ static void populate(sg_io_hdr_t * sih)
 	memset(&static_cdb, '\0', sizeof static_cdb);
 	sih->cmdp = static_cdb;
 	sih->cmd_len = 6;
-	memcpy(sih->cmdp, "\x12\0\0\0\x24\0", sih->cmd_len);
+	memcpy(sih->cmdp, "\x1B\0\0\0\x01\0", sih->cmd_len);
 
 	memset(&static_data, '\0', sizeof static_data);
 	sih->dxferp = static_data;
-	sih->dxfer_len = 0x24;
-	sih->dxfer_direction = SG_DXFER_NONE;
+	sih->dxfer_len = 0x00;
 	sih->dxfer_direction = SG_DXFER_TO_DEV;
 	sih->dxfer_direction = SG_DXFER_FROM_DEV;
+	sih->dxfer_direction = SG_DXFER_NONE;
 
 	memset(&static_sense, '\0', sizeof static_sense);
 	sih->sbp = static_sense;
--- sgio-up.c	2004-06-09 12:23:28.901649448 -0600
+++ sgio-dn.c	2004-06-09 12:23:58.069215304 -0600
@@ -17,7 +17,7 @@ static void populate(sg_io_hdr_t * sih)
 	memset(&static_cdb, '\0', sizeof static_cdb);
 	sih->cmdp = static_cdb;
 	sih->cmd_len = 6;
-	memcpy(sih->cmdp, "\x1B\0\0\0\x01\0", sih->cmd_len);
+	memcpy(sih->cmdp, "\x1B\0\0\0\x00\0", sih->cmd_len);
 
 	memset(&static_data, '\0', sizeof static_data);
 	sih->dxferp = static_data;