Andre Hinrichs wrote:
>> Hi!
>> I'm very glad to say "It works!".
>> Doug, did Holger have the right idea?
>From the conversations I had with him, no, he did it differently. I
never looked at his code. Once I got a handle on what the problem was,
I sent him a test patch or two until it clicked that I wasn't honoring
the p->transinfo[tindex].user_offset in aic7xxx_done() inside of the
section of code that scans the device capabilities from the results of
the INQUIRY command. When we have a non-ultra 2 controller and we don't
know if the device is Wide or not, the value of goal_offset can change.
So, I was setting that to the "correct" value based on the result of the
wide bits in the INQUIRY data. I was forgetting that it we set it to 0
that meant we didn't want sync at all and I was accidentally wiping it
out in that case. I just had to honor the user_offset == 0 case of
setup and then things worked.
--
Doug Ledford <dledford at redhat.com>
Opinions expressed are my own, but
they should be everybody's.
To Unsubscribe: send mail to majordomo at FreeBSD.org
with "unsubscribe aic7xxx" in the body of the message