Re: port-evbarm/54696: Kernel panic in bus_dma.c on Raspberry Pi 3B/3B+

[Nick Hudson <nick.hudson%gmx.co.uk@localhost>]

The following reply was made to PR port-evbarm/54696; it has been noted by GNATS.

From: Nick Hudson <nick.hudson%gmx.co.uk@localhost>
To: gnats-bugs%netbsd.org@localhost, port-evbarm-maintainer%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, dhop%nwlink.com@localhost
Cc: 
Subject: Re: port-evbarm/54696: Kernel panic in bus_dma.c on Raspberry Pi
 3B/3B+
Date: Sat, 23 Nov 2019 10:25:20 +0000

 This is a multi-part message in MIME format.
 --------------7B5C974A279740D3C01B23C8
 Content-Type: text/plain; charset=utf-8; format=flowed
 Content-Transfer-Encoding: quoted-printable
 
 On 14/11/2019 22:35, sc.dying%gmail.com@localhost wrote:
 [snip]
 
 >   But the transfer length of isoc xfer is sum of xfer->ux_frlengths[i]
 >   where i =3D 0.. xfer->ux_nframes - 1, not zero.
 >   dwc2 should do this usb_syncmem if xfertype =3D=3D UE_{INTERRUPT,BULK}=
 ,
 >   UE_CONTROL that has data phase (that is, len !=3D 0), or if xfertype =
 =3D=3D
 >   UE_ISOCHRONOUS, should do usb_syncmem for each chunk of ux_frlengths.
 >
 >   However, the uaudio works with dwc2 even if avobe usb_syncmem is ignor=
 ed.
 >   I think dwc2 does usb_syncmem for isoc data buf somewhere else, but
 >   I'm not sure where it is.
 
 Please try this patch
 
 thanks,
 Nick
 
 --------------7B5C974A279740D3C01B23C8
 Content-Type: text/plain; charset=UTF-8;
  name="dwc2.c.diff"
 Content-Transfer-Encoding: base64
 Content-Disposition: attachment;
  filename="dwc2.c.diff"
 
 SW5kZXg6IHN5cy9leHRlcm5hbC9ic2QvZHdjMi9kd2MyLmMKPT09PT09PT09PT09PT09PT09
 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1Mg
 ZmlsZTogL2N2c3Jvb3Qvc3JjL3N5cy9leHRlcm5hbC9ic2QvZHdjMi9kd2MyLmMsdgpyZXRy
 aWV2aW5nIHJldmlzaW9uIDEuNTkKZGlmZiAtdSAtcCAtcjEuNTkgZHdjMi5jCi0tLSBzeXMv
 ZXh0ZXJuYWwvYnNkL2R3YzIvZHdjMi5jCTE5IE1hciAyMDE5IDA4OjE3OjQ2IC0wMDAwCTEu
 NTkKKysrIHN5cy9leHRlcm5hbC9ic2QvZHdjMi9kd2MyLmMJMjMgTm92IDIwMTkgMTA6MjQ6
 NTcgLTAwMDAKQEAgLTk0Nyw3ICs5NDcsNyBAQCBkd2MyX2RldmljZV9zdGFydChzdHJ1Y3Qg
 dXNiZF94ZmVyICp4ZmVyCiAJdWludDMyX3Qgb2ZmID0gMDsKIAlpbnQgcmV0dmFsLCBlcnI7
 CiAJaW50IGFsbG9jX2JhbmR3aWR0aCA9IDA7Ci0JaW50IGk7CisJc2l6ZV90IGk7CiAKIAlE
 UFJJTlRGTigxLCAieGZlcj0lcCBwaXBlPSVwXG4iLCB4ZmVyLCB4ZmVyLT51eF9waXBlKTsK
 IApAQCAtOTg3LDYgKzk4NywxMyBAQCBkd2MyX2RldmljZV9zdGFydChzdHJ1Y3QgdXNiZF94
 ZmVyICp4ZmVyCiAJCURQUklOVEZOKDMsICJyZXEgPSAlcCBkbWEgPSAlIiBQUkl4QlVTQURE
 UiAiIGxlbiAlZCBkaXIgJXNcbiIsCiAJCSAgICBLRVJOQUREUigmZHBpcGUtPnJlcV9kbWEs
 IDApLCBETUFBRERSKCZkcGlwZS0+cmVxX2RtYSwgMCksCiAJCSAgICBsZW4sIGRpciA9PSBV
 RV9ESVJfSU4gPyAiaW4iIDogIm91dCIpOworCX0gZWxzZSBpZiAoeGZlcnR5cGUgPT0gVUVf
 SVNPQ0hST05PVVMpIHsKKwkJRFBSSU5URk4oMywgInhmZXI9JXAgbmZyYW1lcz0lZCBmbGFn
 cz0lZCBhZGRyPSVkIGVuZHB0PSVkLCIKKwkJICAgICIgbXBzPSVkIGRpciAlc1xuIiwgeGZl
 ciwgeGZlci0+dXhfdXhfbmZyYW1lcywgeGZlci0+dXhfZmxhZ3MsIGFkZHIsCisJCSAgICBl
 cG51bSwgbXBzLCBkaXIgPT0gVVRfUkVBRCA/ICJpbiIgOiJvdXQiKTsKKworCQlmb3IgKGxl
 biA9IGkgPSAwOyBpIDwgeGZlci0+dXhfbmZyYW1lczsgaSsrKQorCQkJbGVuICs9IHhmZXIt
 PnV4X2ZybGVuZ3Roc1tpXTsKIAl9IGVsc2UgewogCQlEUFJJTlRGTigzLCAieGZlcj0lcCBs
 ZW49JWQgZmxhZ3M9JWQgYWRkcj0lZCBlbmRwdD0lZCwiCiAJCSAgICAiIG1wcz0lZCBkaXIg
 JXNcbiIsIHhmZXIsIHhmZXItPnV4X2xlbmd0aCwgeGZlci0+dXhfZmxhZ3MsIGFkZHIsCg==
 --------------7B5C974A279740D3C01B23C8--