Re: Switchover to CAM ATA?

On Mon, Apr 26, 2010 at 09:18:07AM -0600, Scott Long wrote:
> On Apr 26, 2010, at 6:51 AM, Alexander Motin wrote:
> > Marius Strobl wrote:
> >> As noted earlier, pc98 and sparc64 need ada(4)/CAM ATA to perform
> >> geometry translation as done by ad_firmware_geom_adjust() for ad(4),
> >> which the following patch hooks up to both:
> >> http://people.freebsd.org/~marius/ata_disk_firmware_geom_adjust.diff
> >> You preferred to implement such functionality via XPT_CALC_GEOMETRY
> >> though (I'm still not convinced that it makes sense to put this
> >> functionality into every ATA SIM the same way it is done for SCSI
> >> rather than letting ada(4) handle it the same way for all SIMs
> >> however). Have you looked into implementing XPT_CALC_GEOMETRY for
> >> ATA CAM or is it okay to commit the above patch?
> > 
> > Sorry, I have forgotten about this.
> > 
> > I don't have better idea. For ATA translation seems indeed more
> > platform- then controller-specific. May be I would just preferred to see
> > this hack to be done inside XPT_CALC_GEOMETRY handler, as it is done now
> > for PC98 SCSI. But looking that whole this topic is quite crappy and
> > hopefully going to die sometimes, I won't argue much against committing
> > this as-is for now.
> 
> Put this into XPT_CALC_GEOMETRY.  There's no point in perpetuating the mistakes of the ata driver.
> Give me a day or two to think of a reasonable way to do it right.
> 

Did you get further with this approach?

Marius