Date: Sat, 13 Aug 2005 11:26:22 +0200 From: =?ISO-8859-1?Q?S=F8ren_Schmidt?= <sos@FreeBSD.org> To: Vadim Chekan <vchekan@cox.net> Cc: current@FreeBSD.org Subject: Re: [Patch] Serial ATAPI is not recognized Message-ID: <DABE4116-EE59-463C-8AB6-D4E2F66B9E64@FreeBSD.org> In-Reply-To: <42FCE289.4070901@cox.net> References: <42FBEA3A.5090809@cox.net> <22CD48F3-747D-48EB-A5CF-633FB78BCA8D@FreeBSD.org> <42FCE289.4070901@cox.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On 12/08/2005, at 19:55, Vadim Chekan wrote: > S=F8ren Schmidt wrote: >> >> SATA/ATAPI is not supported yet, and the above patch is by far =20 >> not enough for all systems to work. >> In fact you are just lucky on your system, for the heck of it I =20 >> tried it here and my test system lost all its SATA disks :) >> Support is in the works and will hit -current when its stable =20 >> enough for general usage.... >> > Then I'll be one of the first who check this support. > Thanks for your job! > > BTW, I was a little bit puzzled by the code in > ata-chipset.c:ata_achi_allocate() > /* XXX SOS this is a hack to satisfy various legacy cruft */ > ch->r_io[ATA_CYL_LSB].res =3D ctlr->r_res2; > ch->r_io[ATA_CYL_LSB].offset =3D ATA_AHCI_P_SIG + 1 + offset; > ch->r_io[ATA_CYL_LSB].res =3D ctlr->r_res2; > ch->r_io[ATA_CYL_MSB].offset =3D ATA_AHCI_P_SIG + 3 + offset; > > > If I understand the logic of the code correct, in the line before =20 > last should be ATA_CYL_MSB, not ATA_CYL_LSB. Looks like it's typo =20 > and ch->r_io[ATA_CYL_MSB].res is never initialized by this code. =20 > Neither I can find where it is initialized at all. But somehow it =20 > works !? Thats a bug alright, however its not used as the AHCI support doesn't =20= understand ATAPI yet, so the code should newer be used (but it will =20 fail to recon ATAPI devices). - S=F8ren
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?DABE4116-EE59-463C-8AB6-D4E2F66B9E64>