Re: PS/2 Esdi patch #8

[A Duston <hald@sound.net>]

Jens Axboe wrote:

> On Thu, May 24 2001, Paul Gortmaker wrote:
> > Hal Duston wrote:
> >
> > > http://www.sound.net/~hald/projects/ps2esdi/ps2esdi-2.4.4-patch4
> > >
> > > Hal Duston
> > > hald@sound.net
> >
> > You PS/2 ESDI guys might want to set the max sectors for your
> > driver - old default used to be 128, currently 255 (which maybe
> > hardware can handle ok?) - the xd and hd drivers were broken until
> > a similar fix was added to them.
> >
> > Probably makes sense for driver to set it regardless, seeing
> > as default (MAX_SECTORS) has changed several times over last
> > few months.  At least then it will be under driver control
> > and not at the mercy of some global value.
>
> You might want to assign that max_sect array too, otherwise it's just
> going to waste space :-)
>
> Take a look at how ps2esdi handles requests -- always processing just
> the first segment. Alas, it doesn't matter how big the request is.

OK, obviously I am still missing something here from when I got the
driver booting again.  Presumably something with current_nr_sectors,
vs nr_sectors, maybe?  I thought it was odd that all the transfers were
exactly 2 blocks. I'll go ahead and take this one.  I will also go ahead
and check to see how much data the hardware can transfer at once
as well, but I expect it is quite a bit.  I am still working on getting a

group of folks to test patch #5 to see if it works as well.  Anyone
with appropriate hardware willing to test can contact me.  All I have
access to is my thinkpad 700 PS/2, and a 50Z that is a 286.

Thanks,
Hal Duston
hald@sound.net