Hi Jani, On Mon, Aug 31, 2015 at 10:15:07PM +0300, Jani Nikula wrote: > On Sat, 29 Aug 2015, Lukas Wunner <lukas@xxxxxxxxx> wrote: > > the patch set I've posted August 12 included 3 commits which fix bugs > > in i915. These bugs should be fixed independently of MacBook Pro GPU > > switching, please consider merging them: > > [...] > > drm/i915: Fix failure paths around initial fbdev allocation > > http://patchwork.freedesktop.org/patch/53673/ > > drm/i915: On fb alloc failure, unref gem object where it gets refed > > http://patchwork.freedesktop.org/patch/53674/ > > Sorry about that. Unfortunately the target is moving fast, and they no > longer apply. Please resend on top of current nightly. Alright, coming up in separate e-mails are the above 2 patches rebased on drm-intel-nightly as of this morning. I didn't have to make any changes to the code, if they didn't apply cleanly to your tree it was probably just because of changed diff context. To ease reviewing I've also pushed them to GitHub: https://github.com/l1k/linux/commit/f0cd66427039ce1bdc61460a9d833e6d858cff3e https://github.com/l1k/linux/commit/521e48fc5fc8d211ed2847070120ff4032b7a383 Briefly, the story of the 2 patches is this: - I had originally reported the issue on June 3: http://lists.freedesktop.org/archives/intel-gfx/2015-June/067965.html - Tvrtko came up with a patch which I've tested successfully: https://patchwork.freedesktop.org/patch/53207/ - However Ville responded to Tvrtko's patch: "I find it rather unexpected that the function drops the passed reference on error. My usual rule is: do nothing on error, if possible." (see comment section of patchwork link) Tvrtko answered that he didn't have time to look into this further and I wanted it fixed, so I submitted a set of 2 patches, consisting of an adjusted version of Tvrtko's patch, plus another one by me to address Ville's remarks. So you can either merge Tvrtko's single patch or the 2 patches from me, whichever you prefer. Or request something completely different. Thanks, Lukas _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
