ATAng support for reading from register on ATAREQUEST command?

[Matt Kucenski]

Thanks, I was able to get those changes into my source and they work fine.  Unfortunately, I overlooked the fact that Read Native Max Address returns part of the address in the Device register as well as the three lba registers.  Evidently bits 27:24 of the native address come back in the bottom 4 bits of the device register.

Is that register easily accessible somewhere?

-Matt

> 
> From: Søren Schmidt <sos at DeepCore.dk>
> Date: 2004/11/21 Sun AM 07:00:27 EST
> To: m.kucenski at computer.org
> CC: freebsd-current at freebsd.org
> Subject: Re: ATAng support for reading from register on ATAREQUEST command?
> 
> Matt Kucenski wrote:
> > I am trying to develop a program that will allow modifications to the Host Protected Area settings and one of the commands (READ NATIVE MAX ADDRESS) returns the native max address back in the register (LBA high, low, mid).
> > 
> > I have been looking at the smartmontools project for pointers on how to write this code and according to that source, this is not possible yet with ATAng.  There is a comment in their code that another command (ATA_CMD_READ_REG) patch has been submitted to ATAng, but it does not appear to have made it into any of the latest sources.
> > 
> > Can anyone offer any information on this?
> 
> The following patch returns the register values in the request you sent 
> through ioctl call. That should do the trick without any new calls..
> 
> 
> -- 
> 
> -Søren
> 
> 
> 
-------------- next part --------------
Index: ata-all.c
===================================================================
RCS file: /home/ncvs/src/sys/dev/ata/ata-all.c,v
retrieving revision 1.233
diff -u -r1.233 ata-all.c
--- ata-all.c	19 Oct 2004 20:13:38 -0000	1.233
+++ ata-all.c	19 Nov 2004 18:59:55 -0000
@@ -546,10 +546,10 @@
 	    bcopy(iocmd->u.request.u.atapi.ccb, request->u.atapi.ccb, 16);
 	}
 	else {
-	     request->u.ata.command = iocmd->u.request.u.ata.command;
-	     request->u.ata.feature = iocmd->u.request.u.ata.feature;
-	     request->u.ata.lba = iocmd->u.request.u.ata.lba;
-	     request->u.ata.count = iocmd->u.request.u.ata.count;
+	    request->u.ata.command = iocmd->u.request.u.ata.command;
+	    request->u.ata.feature = iocmd->u.request.u.ata.feature;
+	    request->u.ata.lba = iocmd->u.request.u.ata.lba;
+	    request->u.ata.count = iocmd->u.request.u.ata.count;
 	}
 
 	request->timeout = iocmd->u.request.timeout;
@@ -566,6 +566,10 @@
 
 	ata_queue_request(request);
 
+	iocmd->u.request.u.ata.command = request->u.ata.command;
+	iocmd->u.request.u.ata.feature = request->u.ata.feature;
+	iocmd->u.request.u.ata.lba = request->u.ata.lba;
+	iocmd->u.request.u.ata.count = request->u.ata.count;
 	if (request->result)
 	    iocmd->u.request.error = request->result;
 	else {