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:   Mon, 3 Dec 2018 22:55:58 +0100
From:   Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>
To:     Christophe Leroy <christophe.leroy@c-s.fr>
Cc:     Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Paul Mackerras <paulus@samba.org>,
        Michael Ellerman <mpe@ellerman.id.au>, j.neuschaefer@gmx.net,
        linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH v1 03/13] powerpc/mm/32s: rework mmu_mapin_ram()
Message-ID: <20181203215558.GK15324@latitude>
References: <cover.1543517818.git.christophe.leroy@c-s.fr>
 <8835330baa77d88e0267b0b1215b78c991e6d17a.1543517818.git.christophe.leroy@c-s.fr>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="3607uds81ZQvwCD0"
Content-Disposition: inline
In-Reply-To: <8835330baa77d88e0267b0b1215b78c991e6d17a.1543517818.git.christophe.leroy@c-s.fr>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--3607uds81ZQvwCD0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

On Thu, Nov 29, 2018 at 07:00:16PM +0000, Christophe Leroy wrote:
> This patch reworks mmu_mapin_ram() to be more generic and map as much
> blocks as possible. It now supports blocks not starting at address 0.
>=20
> It scans DBATs array to find free ones instead of forcing the use of
> BAT2 and BAT3.
>=20
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> ---

I've just tested this series on my Wii, and starting from this patch
(03/13), it hangs at the following lines of output:

[    0.000000] printk: bootconsole [udbg0] enabled
[    0.000000] Total memory =3D 319MB; using 1024kB for hash table (at (ptr=
val))

Before this patch it looks like this and boots to userspace:

[    0.000000] printk: bootconsole [udbg0] enabled
[    0.000000] Total memory =3D 319MB; using 1024kB for hash table (at (ptr=
val))
[    0.000000] Linux version 4.20.0-rc5-wii-00022-gfbb911b84755 (jn@longitu=
de) (gcc version 8.2.0 (Debian 8.2.0-9)) #1337 PREEMPT Mon Dec 3 21:49:02 C=
ET 2018
ug_udbg_init: early -> final
usbgecko_udbg: ready
[    0.000000] Using wii machine description
=2E..

I've tested at patch 1, 2, 3, 4, and 13, so I don't know if it works
somewhere in the middle, but probably not.

(And in case you're wondering about the 22 in the version string: Those
are mostly patches that give me a serial console.)

I'm not sure what is going on, because I haven't looked closely at the
patches or tried to debug the problem. If you have some debugging tips,
I can try them.


Jonathan Neusch=C3=A4fer

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

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

iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAlwFpmUACgkQCDBEmo7z
X9tjQhAAtgGfhvYKAZs5YH93HGyareHmVqRSSg59aSnlMwaTZhLNHTix+xkwCgRQ
i8mr6oA83S5B79HehInblEfrDh0FGluPBrUfhK/bTQOmJ4Cqk8ur4MHIEmBGiy3X
2Py2zXCeuK4K7YOPCqfJmmuPTo+qviQe8ya2i1TPyZ1IqwqnLJ5jstLBs8Yyfm4o
y8azYiRZ43cJyhZ7bI1I6L4Y8oFYG06TLSe8yRaRHWO6iOhbEcv1ANUZ0Lz/Azn2
n0xWB4GhcS+cCJNzzYHjZqXQWWtWElDbuuXYh5kgUQAHZWgFNzCMjm5pd1WysiJk
rxUyaaHTShibBgBz8KRMfXpJBUg6Ux4t86t2JdopbZCJ5fnXY8QFHEMCVpcak7v9
YDlX8ABaxC0s2ykxE9HFF6nrlkYQUk0pmbeqqEZN/vUV1lAQr2tlVap9s2ZwJdIP
HUhhA12LBkOJx5q31wNip2k0cuvM6IjdHSrImjTNb37yHqsGaXCu+TWaWSmzXA/B
f3/7TLY+chXqxUCawpPgeYz53fy4Y9opNjD3hzDzzFD9PcRFzyixl7fM9wlgE37w
D7UNNZ18ky7LKMuL+5RpbH41dGILkku3tPHflc+h0v+RCJacL3h+uDr3+w7Y8br7
D0zb7tVOQZwK8rbIoRzLPgsuU/aE2+3oC6fdLM2utgF7WwG/jlo=
=J3qj
-----END PGP SIGNATURE-----

--3607uds81ZQvwCD0--