lkml.org 
[lkml]   [2003]   [Nov]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRE: [ANNOUNCE] QLogic qla2xxx driver update available (v8.00.00b6).
Date
From
Jens,

> > o Given the ISPs inability to handle data segments that cross
> > 32-bit page boundaries, and the overhead in defensive logic
> > within the driver to prevent these cases (compare the 6.x
> > code to the 8.x code and you will see what I mean), by
> > disabling clustering we guarantee that a single SG element
> > never crosses a 4GB boundary.
>
> That is not, however. Linux will never give you a segment crossing a
> 4GB boundary, exactly because most hardware cannot handle this.
>
> > Now, in 2.6 there are some significant changes. For one, with the
> > block layer rewrite and the ability to limit segment boundaries of a
> > block queue request with the blk_queue_segment_boundary() call, a LLDD
> > need not concern itself with any defensive fast-path logic to handle
> > the 4GB cross.
>
> Ditto 2.4. See BH_PHYS_4G().
>

Yes, but if I recall, these checks did not enter until late in the 2.4
branch. The original (engineering) intent of the 6.x series branch
was to support all 2.4 kernels, though we are slowly (formally) moving
away from the restrictions.

> > So, we're left with the benefits of the overhead of this merge
> > process done by the block layer. I'm certainly receptive to the
> > notion of reexamining the use of clustering given some solid data
> > points showing an (significant -- this is subjective) increase in
> > performance and/or a resounding 'yeah, enable it!' from those in
> > the block-layer 'know.'
>
> In 2.6 clustering comes for free, so it would be silly not to enable
> it.
>

Thanks for the clarification, I'll add the parameter back into the
template for the 8.x series driver.

Regards,
Andrew Vasquez
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:58    [W:0.043 / U:0.060 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site