Ed,

Well on the surface if you are doing a client side AoE driver you would
look at the ATA spec and decide that all the N/A values can be just set to
zero. Which is what I did but immediately got a bad arguments error from
the blade server. When you look at the blade server code it does this test
for "1" for no apparent reason.

My initial suspicion was that the blade code was wanting to reuse the
packet that it received the request on and as such wanted to make sure it
was big enough by having the client set the sector count to 1 and further
adding some noop data to the request packet. But looking at the blade code
the way it carves up packet buffers it didn't need to do that so it is
unclear why it is even in there.

>From my perspective, it is unneeded and actually complicates things because
the AoE spec implies that you are just supposed to map the ATA commands and
registers on to the request without doing any special modifications...
which isn't true in this particular case. I've not seen what a normal SATA
request for ID would look like from a HD perspective but I suspect they
don't even set those register values and may even potentially just set them
all to zero.

I popped the question on this discussion board to see if I'm missing
something.

David


On Thu, Oct 3, 2013 at 7:29 PM, Ed Cashin <ecas...@coraid.com> wrote:

> On Oct 3, 2013, at 2:36 PM, David Leach wrote:
>
> > Can someone explain why the ATA command Identify Device (0xEC) has the
> following test in the switch statement:
> >
> >     case 0xec:        // identify device
> >         if (p->sectors != 1 || ndp < 512)
> >             return -1;
> >
> >
> > Given that the ATA spec says that the sector register is N/A (as are
> pretty much all the rest) when making this command I don't know why the
> blade server requires that submission of this command have AoE ATA command
> set sector to 1?
> >
> > I have my suspicions but I would like to hear from others...
>
> What are your suspicions?  Looking at ATA 6 draft rev. 3b section 18.5 I
> see also see that sector count is NA on input.
>
> --
>   Ed Cashin
>   ecas...@coraid.com
>
>
>
------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134791&iu=/4140/ostg.clktrk
_______________________________________________
Aoetools-discuss mailing list
Aoetools-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/aoetools-discuss

Reply via email to