Re: [PATCH] speed up SATA

From: Marc Bevand
Date: Tue Mar 30 2004 - 08:51:05 EST


On 30 Mar 2004, Jens Axboe wrote:
| On Tue, Mar 30 2004, Marc Bevand wrote:
| [...]
| > As other people were complaining that the 32MB max request size might be too
| > high, I did give a try to 1MB (by replacing "65534" by "2046" in the patch).
| > There is no visible differences between 32MB and 1MB.
|
| As suspected. BTW, you want to use 2048 there, not 2046. The 64K-2
| (which could be 64K-1) is just due to ->max_sectors being an unsigned
| short currently.

Okay, I thought the 2 sectors were used to store some extra informations.

I also forgot to mention that there does not seem to have any *latency*
differences between 32MB and 1MB. Strange...

--
Marc Bevand http://www.epita.fr/~bevand_m
Computer Science School EPITA - System, Network and Security Dept.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/