From mboxrd@z Thu Jan  1 00:00:00 1970
Message-ID: <47AC7675.2030603@gmx.de>
Date: Fri,  8 Feb 2008 16:34:13 +0100
From: Kernel Panic <cinap_lenrek@gmx.de>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;
	rv:1.7.12) Gecko/20050915
MIME-Version: 1.0
To: Fans of the OS Plan 9 from Bell Labs <9fans@cse.psu.edu>
Subject: Re: [9fans] VIA VT8237 SATA/RAID i/o errors, dma doesnt work (VIA
	Rhine II)
References: <675fc52d4267e9ef68bedb3cfb468f58@quanstro.net>
In-Reply-To: <675fc52d4267e9ef68bedb3cfb468f58@quanstro.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: quoted-printable
Topicbox-Message-UUID: 4be1da26-ead3-11e9-9d60-3106f5b1d025

erik quanstrom wrote:

>>I changed sdata.c to:
>>
>>return SDretry if starting dmarequest fails,
>>return SDretry and reseting controller if dma times out,
>>return SDretry on request is done and error signaled,
>>
>>and added a counter that resets the controller if a request
>>was 1000 times retried...
>>   =20
>>
>
>you might want to loop internally instead of counting to 1000.
>(which may be too high.  i would think that 2 or 4 would do it.
>have you seen differently?)  if more than one thread is doing i/o
>to the disk and you have a bad sector, it may take an arbitrary
>amount of time before you discover this fact.
> =20
>
ok, will lower that number and test...

>>The machine ran the whole night with bodth drives in DMA and heavy load=
.
>>vacing, pingflooded from outside, ddtest on both drives and still alive=
! :-)
>>   =20
>>
>
>good deal.  via's product page
>www.via.com.tw/en/products/chipsets/southbridge/vt8237/specs.jsp
>indicates that the nic & sata controller are both integrated into the sb=
.
>perhaps this extra southbridge configuration (pci.c) would make the
>sdata hack unnecessary?
> =20
>
I'm using a famous/cheap/bad realtek PCI card.
The driver (ethervt6102.c) for the internal nic assumes that his=20
Ringdescriptor-structures
fit into the cache line size of the card and that is not met:

if((cls =3D pcicfgr8(p, PciCLS)) =3D=3D 0 || cls =3D=3D 0xFF)
    cls =3D 0x10;
ctlr->cls =3D cls*4;
if(ctlr->cls < sizeof(Ds)){
        print("vt6102: cls %d < sizeof(Ds)\n", ctlr->cls);
        free(ctlr);
        continue;
}

pci -v

0.18.0:	net  02.00.00 1106/3065  10 0:0000f001 256 1:fdffe000 256
	VIA Technology VT6102 Rhine II PCI Fast Ethernet Controller


I think Llu=C3=ADs Batlle <viriketo@gmail.com> has the exact same problem=
:

http://9fans.net/archive/2008/02/128

But this had low priority for me, In the worst case, would install a=20
3com PCI card before
using it in production.

>static void
>viaset(Pcidev *p, uchar link, uchar irq)
>{
>	uchar pirq;
>
>	pirq =3D pcicfgr8(p, 0x55 + (link >> 1));
>	pirq &=3D (link & 1)? 0x0f: 0xf0;
>	pirq |=3D (link & 1)? (irq << 4): (irq & 15);
>	pcicfgw8(p, 0x55 + (link>>1), pirq);
>	if(p->vid !=3D 0x3177 && p->vid !=3D 0x3227)
>		return;
>
>	/* Set transition time to max */
>	pcicfgw8(p, 0xec, pcicfgr8(p, 0xec) & ~4);
>	pcicfgw8(p, 0xe4, pcicfgr8(p, 0xe4) & ~0x80);
>	pcicfgw8(p, 0xe5, pcicfgr8(p, 0xe5) | 0x80);
>
>//	if(pcicfgr8(p, 0x81) & 0x80)
>//		acpiregs =3D pcicfgr32(p, 0x88) & 0xff00;
>}
>
> =20
>
thanks, i'll try!

>- erik
> =20
>
cinap