Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Tue, 18 Dec 2018 04:05:38 +0100
From:   Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>
To:     Christophe Leroy <christophe.leroy@c-s.fr>
Cc:     Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>,
        linuxppc-dev@lists.ozlabs.org, Paul Mackerras <paulus@samba.org>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 03/13] powerpc/mm/32s: rework mmu_mapin_ram()
Message-ID: <20181218030538.GA24278@latitude>
References: <cover.1543517818.git.christophe.leroy@c-s.fr>
 <8835330baa77d88e0267b0b1215b78c991e6d17a.1543517818.git.christophe.leroy@c-s.fr>
 <20181203215558.GK15324@latitude>
 <84624899-bbda-9f08-2527-151fddbd1b01@c-s.fr>
 <a253e9f8-1e0a-1596-e0ea-54261ac62212@c-s.fr>
 <20181217012837.GT15324@latitude>
 <18ba3a7c-ebfa-66aa-e231-b56985d0e89a@c-s.fr>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="ReaqsoxgOBHFXBhH"
Content-Disposition: inline
In-Reply-To: <18ba3a7c-ebfa-66aa-e231-b56985d0e89a@c-s.fr>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--ReaqsoxgOBHFXBhH
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Dec 17, 2018 at 10:29:18AM +0100, Christophe Leroy wrote:
> > With patches 1-3:
> > [    0.000000] setbat(0, c0000000, 00000000, 01000000, 311)
> > [    0.000000] setbat(2, c1000000, 01000000, 00800000, 311)
> > [    0.000000] setbat(4, d0000000, 10000000, 02000000, 791)
>=20
> What we see is that BAT0 is not used in the origin. I have always wondered
> the reason, maybe there is something odd behind and BAT0 shall no ne used.
>=20
> Could you try and modify find_free_bat() so that it starts at b =3D 1 ins=
tead
> of b =3D 0 ?

In this case, setbat is called with index 2, 3, and 4, but the Wii still
doesn't boot.

> > According to arch/powerpc/include/asm/book3s/32/hash.h,
> >   - 0x591 =3D _PAGE_RW | _PAGE_ACCESSED | _PAGE_DIRTY | _PAGE_COHERENT =
| _PAGE_PRESENT
> >   - 0x311 =3D _PAGE_EXEC | _PAGE_ACCESSED | _PAGE_COHERENT | _PAGE_PRES=
ENT
> >   - 0x791 =3D _PAGE_RW | _PAGE_EXEC | _PAGE_ACCESSED | _PAGE_DIRTY | _P=
AGE_COHERENT | _PAGE_PRESENT
> >=20
>=20
> Yes, patch 1 added _PAGE_EXEC which explains this 0x200.
> Do you confirm it still works well with only patch 1 ?

Patch 1 alone boots to userspace.


Jonathan

--ReaqsoxgOBHFXBhH
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAlwYY/kACgkQCDBEmo7z
X9srjQ//YkSw5N9QwnyF4TE1jp1QLg5wnwHtbepzoxOfaZ4xDejB9bVXT6+smkhH
aUa+AVVkkBwiBv6Ce6X4DHaYTHxu2kXoDJWt3pN28ZZ99F64oNgnHbQpy+XWJ1sJ
mc/MQlaIDsoDjwniZQjr+gp55rZkrDVdI2GaAbVeFUlGEMKYvIGmM/w+K8X/K87N
eBJHfXM1ztnfoUqeto9H/DsajuYqxXAuGqIHyw9KqRTnUYpCXztdgfOmyiR/QlRO
9O4W9xv1eS5FiOR0XwC/QBBhE2rgOp7uRbk9d6eh8ii4edrO3oe/61xFF9E+uRJf
EP2fDPUnsHY83l8UosYcillYbsZBlIbjojflXTuyqbb+HSRl8lFxl0JDsokkCrUi
dYwbB/up8ljHlfb9Zku/h4LhOpYklWYNeAMcAJZY6W2ewiQkazIw2NA5o6YFyG1N
dowfvYPUGqbfG7OtqWbhRkfTYlliLojFpifjH1jpkwm7jDOcno4eATKojfDf0qUt
40vXg/P3UCvX+T5jkr92peYwL5S5ChCWNZQ9GlvpigFCnP0BY5NKY+KqBqDpzBuM
Z7VrVZ4WIQkoebpW90gt0DAofubFJII+4pINM/HPuiBBkq/FyFyRRfEl58DO+kqT
KorhpRuKkUb+z4bI44+6A1WM6HlORjXBpDBPe/vkqLNXAbOtEkU=
=UInt
-----END PGP SIGNATURE-----

--ReaqsoxgOBHFXBhH--