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:   Fri, 26 Apr 2019 11:05:17 +1000
From:   David Gibson <david@gibson.dropbear.id.au>
To:     Christoph Hellwig <hch@lst.de>
Cc:     Jens Axboe <axboe@kernel.dk>,
        Michael Ellerman <michael@ellerman.id.au>,
        Paul Mackerras <paulus@samba.org>,
        linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
        Nick Piggin <npiggin@gmail.com>
Subject: Re: powerpc hugepage leak caused by 576ed913 "block: use
 bio_add_page in bio_iov_iter_get_pages"
Message-ID: <20190426010517.GA7378@umbus.fritz.box>
References: <20190423054131.GB31496@umbus.fritz.box>
 <20190425061958.GA7881@lst.de>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="45Z9DzgjV8m4Oswq"
Content-Disposition: inline
In-Reply-To: <20190425061958.GA7881@lst.de>
User-Agent: Mutt/1.11.3 (2019-02-01)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


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

On Thu, Apr 25, 2019 at 08:19:58AM +0200, Christoph Hellwig wrote:
> Just curious:  What exact trees do you see this with?  This area
> changed a lot with the multipage bvec support, and subsequent fixes.

So, I tried it with 576ed913 itself and with 576ed913^ to verify that
it didn't happen there.  The problem also occurred with Linus' tree as
of when I started bisecting, which appears to have been 444fe991.
Actually, come to that, here's the whole bisect log in case it's
helpful:

# git bisect log
git bisect start
# good: [bebc6082da0a9f5d47a1ea2edc099bf671058bd4] Linux 4.14
git bisect good bebc6082da0a9f5d47a1ea2edc099bf671058bd4
# bad: [444fe991353987c1c9bc5ab1f903d01f1b4ad415] Merge tag 'riscv-for-linu=
s-5.1-rc6' of git://git.kernel.org/pub/scm/linux/kernel/git/palmer/riscv-li=
nux
git bisect bad 444fe991353987c1c9bc5ab1f903d01f1b4ad415
# good: [399c4129eba6145924ab90363352b7bdcd554751] Merge tag 'pxa-for-4.19-=
dma_slave_map' of https://github.com/rjarzmik/linux
git bisect good 399c4129eba6145924ab90363352b7bdcd554751
# bad: [73b6f96cbc0162787bcbdac5f538167084c8d605] Merge branch 'drm-fixes-4=
=2E20' of git://people.freedesktop.org/~agd5f/linux into drm-fixes
git bisect bad 73b6f96cbc0162787bcbdac5f538167084c8d605
# good: [85a585918fb4122ad26b6febaec5c3c90bf2535c] Merge tag 'loadpin-secur=
ity-next' of https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux int=
o next-loadpin
git bisect good 85a585918fb4122ad26b6febaec5c3c90bf2535c
# bad: [3acbd2de6bc3af215c6ed7732dfc097d1e238503] Merge tag 'sound-4.20-rc1=
' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound
git bisect bad 3acbd2de6bc3af215c6ed7732dfc097d1e238503
# good: [8f18da47211554f1ef674fef627c05f23b75a8e0] Merge branch 'master' of=
 git://git.kernel.org/pub/scm/linux/kernel/git/klassert/ipsec-next
git bisect good 8f18da47211554f1ef674fef627c05f23b75a8e0
# bad: [0d1b82cd8ac2e8856ae9045c97782ac1c359929c] Merge branch 'ras-core-fo=
r-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
git bisect bad 0d1b82cd8ac2e8856ae9045c97782ac1c359929c
# bad: [1650ac53066577a5e83fe3e9d992c9311597ff8c] Merge tag 'mmc-v4.20' of =
git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc
git bisect bad 1650ac53066577a5e83fe3e9d992c9311597ff8c
# bad: [6ab9e09238fdfd742fe23b81e2d385a1cab49d9b] Merge tag 'for-4.20/block=
-20181021' of git://git.kernel.dk/linux-block
git bisect bad 6ab9e09238fdfd742fe23b81e2d385a1cab49d9b
# good: [528985117126f11beea339cf39120ee99da04cd2] Merge tag 'arm64-upstrea=
m' of git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux
git bisect good 528985117126f11beea339cf39120ee99da04cd2
# bad: [2cf99bbd106f89fc72f778e8ad9d5538f1ef939b] lightnvm: pblk: add helpe=
rs for chunk addresses
git bisect bad 2cf99bbd106f89fc72f778e8ad9d5538f1ef939b
# bad: [33b14f67a4e1eabd219fd6543da8f15ed86b641c] nvme: register ns_id attr=
ibutes as default sysfs groups
git bisect bad 33b14f67a4e1eabd219fd6543da8f15ed86b641c
# bad: [27ca1d4ed04ea29dc77b47190a3cc82697023e76] block: move req_gap_back_=
merge to blk.h
git bisect bad 27ca1d4ed04ea29dc77b47190a3cc82697023e76
# bad: [07b05bcc3213ac9f8c28c9d835b4bf3d5798cc60] blkcg: convert blkg_looku=
p_create to find closest blkg
git bisect bad 07b05bcc3213ac9f8c28c9d835b4bf3d5798cc60
# good: [cbeb869a3d1110450186b738199963c5e68c2a71] block, bfq: correctly ch=
arge and reset entity service in all cases
git bisect good cbeb869a3d1110450186b738199963c5e68c2a71
# bad: [576ed9135489c723fb39b97c4e2c73428d06dd78] block: use bio_add_page i=
n bio_iov_iter_get_pages
git bisect bad 576ed9135489c723fb39b97c4e2c73428d06dd78
# good: [c8765de0adfcaaf4ffb2d951e07444f00ffa9453] blok, bfq: do not plug I=
/O if all queues are weight-raised
git bisect good c8765de0adfcaaf4ffb2d951e07444f00ffa9453
# first bad commit: [576ed9135489c723fb39b97c4e2c73428d06dd78] block: use b=
io_add_page in bio_iov_iter_get_pages

The problem also occurred with the RHEL8 downstream kernel tree.
That's based on 4.18, but has 576ed913 backported.

> So I'd be really curious if it can be reproduced with Jens' latest
> block for-5.2 tree (which should be in latest linux-next).

I'll see if I can try that when I next get access to the machine.

--=20
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

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

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

iQIzBAEBCAAdFiEEdfRlhq5hpmzETofcbDjKyiDZs5IFAlzCWU0ACgkQbDjKyiDZ
s5LH3BAAn2hSOM3+dwLehrQVwc4k+3c4Hmw+7p5wrgOgIGkH9MDR/SUzNxOZkhM4
hzhHZfwOQ4R+t3eJpWyW2QrY7uBrRryyYnvZPu9yigZvuj0B5cfEgbV6UiyZVIyI
L1xdYdKkd0Yajne4mf+FfR/td1YL50A+qrh62VbQvoUVXFTFJeWCR1Klm1VGdQ2o
RYL/89BmDzyN2YWDK2CrWTn/GKFGnBwRoUoKl0ibjHeRFaTI4S2PkqtIGJTSWHPo
fEhWv26SrFuXceGSywEJxk/GMF+V0xFOGpMvhoIM5pC28xq5z03MEQ4CYJd8Or75
TMaXE4ksK41U3ITTEqpoqqy4V/h4FbihtUPzM8oOj8tVJMPVGBpTI5281VMYIp7p
POHrBnwyKDYyPVtkXRMs93s3O+CBFAWRp8Kgc+08BaZCTMtSjeSJkVxeL9nUG8xM
6IQCe1pByeEK8HDmwdf45qgmmQYir3Ceb9toznp2mt7F5RRRSy0GVs643FQi1Ref
crg/BkVU4rlgkbkcWdRBZRxdWbpZ90yT9pE7+5aaBs3fdL9/FfXYorokpKlI98WG
w+ZK+Z3vqr2ncxDtOmY/hZeKj5KgeJ/SFo26PAWORbN6aiG4HziO4+fkdt521vNA
hqua9mghyAwunKrUhkzfv/ECm9saThllOriBpBNGlvvzFK3EjNU=
=I9iT
-----END PGP SIGNATURE-----

--45Z9DzgjV8m4Oswq--