Re: [PATCH v4 2/2] drm/i915: On fb alloc failure, unref gem object where it gets refed

Daniel Vetter
2015-07-06T13:52:43Z

On Mon, Jul 06, 2015 at 02:59:02PM +0200, Lukas Wunner wrote:
> Hi Daniel,
>
> On Mon, Jul 06, 2015 at 09:41:51AM +0200, Daniel Vetter wrote:
> > Please keep a record of the changes you do to the patch so I know what to
> > look out for. Just reving the patch revision alone doesn't add much
> > information for reviewers/maintainers.
>
> There's a changelog in the first patch of this 2 patch series
> (subject "[PATCH v4 1/2] drm/i915: Fix failure paths around
> initial fbdev allocation"), it says:
>
> "v4:
> * Lock struct mutex on unref. (Chris Wilson)"

Ah, I was looking for the v4 changelog for patch 2. Either make a small
v4: rebased note or just don't call patch 2 v4 to avoid confusion in the
future.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation