Date: Sat, 31 Oct 2015 04:55:04 -0500 From: Jason Harmening <jason.harmening@gmail.com> To: Ganbold Tsagaankhuu <ganbold@gmail.com>, "Jason A. Harmening" <jah@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r289759 - in head/sys/arm: arm include Message-ID: <56348FF8.3010602@gmail.com> In-Reply-To: <CAGtf9xOx_iSNqsPT7OdTmAy-ER_wFzOC9YxFigpQ%2B0wLj7=ytQ@mail.gmail.com> References: <201510221638.t9MGc1cc053885@repo.freebsd.org> <CAGtf9xOx_iSNqsPT7OdTmAy-ER_wFzOC9YxFigpQ%2B0wLj7=ytQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --4nUDo2LCQnhKi1QM4Cf5mN1cWsfkcxHK3 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 10/31/15 03:21, Ganbold Tsagaankhuu wrote: > On Fri, Oct 23, 2015 at 12:38 AM, Jason A. Harmening <jah@freebsd.org> > wrote: >=20 >> Author: jah >> Date: Thu Oct 22 16:38:01 2015 >> New Revision: 289759 >> URL: https://svnweb.freebsd.org/changeset/base/289759 >> >> Log: >> Use pmap_quick* functions in armv6 busdma, for bounce buffers and ca= che >> maintenance. This makes it safe to sync buffers that have no VA mappi= ng >> associated with the busdma map, but may have other mappings, possibly = on >> different CPUs. This also makes it safe to sync unmapped bounce buffe= rs in >> non-sleepable thread contexts. >> >> Similar to r286787 for x86, this treats userspace buffers the same a= s >> unmapped buffers and no longer borrows the UVA for sync operations. >> >> Submitted by: Svatopluk Kraus <onwahe@gmail.com> (earlier >> revision) >> Tested by: Svatopluk Kraus >> Differential Revision: https://reviews.freebsd.org/D3869 >=20 >=20 >=20 > It seems I can't boot Odroid C1 with this change. >=20 > http://pastebin.ca/3227678 >=20 > r289758 works for me. >=20 > Am I missing something? >=20 > thanks, >=20 > Ganbold >=20 > Hmmm, the fault address of 0x20 and the fact that this is happening during mi_startup() make me wonder if the per-cpu pageframes used by pmap_quick* haven't been initialized yet. I wonder if changing the order of qpages_init in sys/arm/arm/pmap-v6[-new].c to something other than SI_ORDER_ANY would he= lp? It seems like we'd want SI_ORDER_FOURTH or SI_ORDER_MIDDLE, since mp_start() is SI_ORDER_THIRD. It would be nice to know what's calling bus_dmamap_sync() in this case. I can't figure that out, but maybe that's because I haven't had coffee y= et. Unfortunately I'm going to have limited time to help debug this over the next week, as I'm moving across the country starting (hopefully) today. --Jason --4nUDo2LCQnhKi1QM4Cf5mN1cWsfkcxHK3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQF8BAEBCgBmBQJWNI/4XxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRENkY3MTQyREU0MTU4MTgyRkZDNUU2ODVC QjlGOEJGOTkyODQxRDFCAAoJELufi/mShB0bg1IH/jQQrZm6C/egXU5rqQNw80aX ZFmJ8fqcWRifyoycEzRu2Rc+4V0vOFQxrwztA9JHbao43jZfkQRcbpvgHVc7sv9U eqjkzfi8j6cn5jUBCqOrh1+sSbjOUjxmUvm/DYAjkNgzh1j0eYcLHIoJ5483nVpv J1TnqmG2uQik2OhYDVD7W12mubQEqFbb+5gPLWSeoi8aMLPUEEBhVKR7ftsCr9F3 L+NY6O+zkI1KDCkMw4y5rCJRZm+y5ERcrh+rBJFtoKpZqoWxGtbzyAc/IUlqBRvN xcmKBhNHumuNGCIqq61ug8GQpt+pc+kDMqiGV1sQNK2gbpVtXYS6R26ocA1o5kE= =hFMs -----END PGP SIGNATURE----- --4nUDo2LCQnhKi1QM4Cf5mN1cWsfkcxHK3--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?56348FF8.3010602>