[Intel-gfx] duplicated patch on drm-intel-nightly

Jani Nikula jani.nikula at intel.com
Mon Feb 22 09:34:10 UTC 2016


On Sat, 20 Feb 2016, Rodrigo Vivi <rodrigo.vivi at gmail.com> wrote:
> We have a duplicated patch on drm-intel-nightly
>
> commits d7006964d and cc1de6e80

[I snipped the extra trailing 'q' off the 2nd commit id.]

> causing:
>
> drivers/gpu/drm//amd/amdgpu/amdgpu_ttm.c:818:6: error: redefinition of
> ‘amdgpu_ttm_tt_affect_userptr’
>  bool amdgpu_ttm_tt_affect_userptr(struct ttm_tt *ttm, unsigned long start,
>       ^
> drivers/gpu/drm//amd/amdgpu/amdgpu_ttm.c:799:6: note: previous
> definition of ‘amdgpu_ttm_tt_affect_userptr’ was here
>  bool amdgpu_ttm_tt_affect_userptr(struct ttm_tt *ttm, unsigned long start,
>
> One cc'ed stable and other didn't. So I'm not sure how this
> duplication happened or in which tree, so I'm pinging you all that
> know better how to identify and solve this issue.

There's nothing unusual here. The patch was apparently first applied to
drm-next, and then cherry-picked to drm-fixes and this one is now in
Linus' master. We do this all the time in i915.

The build fail is, or rather was, in our integration tree. I thought I
had screwed up the conflict resolution [1], but turns out the fail was
in a silent conflict. The function got added twice. It's now fixed in
our tree.

Heads up Dave, I presume this will happen when drm-next gets merged to
Linus' tree in the next merge window. (Or perhaps in linux-next
already.)


BR,
Jani.

[1] http://mid.gmane.org/87ziuwvj0o.fsf@intel.com

-- 
Jani Nikula, Intel Open Source Technology Center


More information about the Intel-gfx mailing list