On Mon, Jun 12, 2017 at 9:25 AM, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > On Tue, May 23, 2017 at 11:30:44AM +0200, Greg Kroah-Hartman wrote: >> On Mon, May 22, 2017 at 07:49:50PM -0400, Ilia Mirkin wrote: >> > On Mon, May 22, 2017 at 6:57 PM, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote: >> > > Hi Greg, >> > > >> > > This patch appears to have misapplied. I'm not sure why cherry-pick >> > > got it wrong, but this is meant to go in the nv4a_chipset definition, >> > > not nv44_chipset. >> > > >> > > This comment also applies to the 4.4 and 4.11 cherry-picks you just did. >> > >> > Aha, I see what happened. f94773b9f5ecd1df7c88c2e921924dd41d2020cc >> > made it into v4.11. However by then it appears to have also made it >> > into drm-next as ac799acaa4d8db4f7dcd968b15c9596c80a4677f and thus the >> > drm merge into v4.12. So now you have two versions of the same commit >> > and are trying to cherry-pick both. >> >> Why in the world would you include the same patch in two different trees >> with two different git ids? I'm _really_ starting to hate the drm >> trees... >> >> > I'm guessing it's also the reason why your cherrypick of my other >> > commit failed (ad01a91a820cd9f0d880c407bf556b67298dcc93). >> >> That would make sense, but again, why is this happening? >> >> > And I might note that >> > "drm-nouveau-mmu-nv4a-use-nv04-mmu-rather-than-the-nv44-one.patch" was >> > already there before, so didn't make it into your "4.9-stable patches" >> > commit on the stable-queue git tree. >> >> Ok, so what should i do here? Drop all of the nouveau patches? Some of >> them? Which ones were wrong? >> >> totally confused and grumpy, > > Still confused, I guess all is ok in the stable trees? Not according to > some bug reports we are getting, 4.4 seems broken, Pointer? I'd be highly surprised if anyone were to run into the issue that my patch was addressing ("using old hardware"), or that my fix had caused problems for anyone. However I can confirm that the MMU pointer is set properly in the 4.4 tree for both the NV44 and NV44A chips. Note that there's a totally unrelated issue in the patch series ending in https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/commit/drivers/gpu/drm/nouveau?h=linux-4.4.y&id=e4add1cf6b4154804350c3385c6d447cff3570de but Ben is aware of that issue and has a patch. I suspect it should be making its way to stable-land ... eventually? On the bright side, that code was broken before too and led to kernel thread deadlocks on some hardware. Cheers, -ilia