[Intel-gfx] [PULL] topic/i915-ttm

Thomas Zimmermann tzimmermann at suse.de
Mon Jun 14 07:21:38 UTC 2021


Hi

Am 14.06.21 um 08:46 schrieb Joonas Lahtinen:
> Quoting Thomas Zimmermann (2021-06-13 21:54:03)
>> Hi Joonas
>>
>> Am 11.06.21 um 13:13 schrieb Joonas Lahtinen:
>>> Quoting Joonas Lahtinen (2021-06-11 13:40:56)
>>>> Quoting Maarten Lankhorst (2021-06-11 12:27:15)
>>>>> Pull request for drm-misc-next and drm-intel-gt-next.
>>>>>
>>>>> topic/i915-ttm-2021-06-11:
>>>>> drm-misc and drm-intel pull request for topic/i915-ttm:
>>>>> - Convert i915 lmem handling to ttm.
>>>>> - Add a patch to temporarily add a driver_private member to vma_node.
>>>>> - Use this to allow mixed object mmap handling for i915.
>>>>> The following changes since commit 1bd8a7dc28c1c410f1ceefae1f2a97c06d1a67c2:
>>>>>
>>>>>     Merge tag 'exynos-drm-next-for-v5.14' of git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos into drm-next (2021-06-11 14:19:12 +1000)
>>>>
>>>> This base is not in drm-misc-next or drm-intel-gt-next, so effectively
>>>> we would end up pulling 478 extra commits from drm-next as a result. And
>>>> also causing all the warnings for those commits. I don't think we should
>>>> do that?
>>>>
>>>> The common ancestor would be ccd1950c2f7e38ae45aeefb99a08b39407cd6c63
>>>> "Merge tag 'drm-intel-gt-next-2021-05-28' of git://anongit.freedesktop.org/drm/drm-intel into drm-next"
>>>> Should we re-do the topic branch based on that?
>>>
>>> This problem seems to come from the fact that only the PR from yesterday
>>> that got merged to drm-next had the dependency patches. The previous
>>> backmerge of drm-next was requested too early.
>>>
>>> I've solved this with least hassle by backmerging drm-next again and
>>> then applying the PR to drm-intel-gt-next.
>>>
>>> I think drm-misc-next should do the same (exact commit was
>>> 1bd8a7dc28c1c410f1ceefae1f2a97c06d1a67c2).
>>
>> I did a backmerge from drm-next recently and drm-misc-next can merge the
>> patches in tags/topic/i915-ttm-2021-06-11 without additions.
>>
>> I assume you to updated drm-intel-gt-next without redoing the PR?
> 
> Correct.

Some patches landed in drm-next. I now did another backmerge and then 
merged the topic branch.

Best regards
Thomas

> 
> Regards, Joonas
> 
>>
>> Best regards
>> Thomas
>>
>>>
>>> Regards, Joonas
>>>
>>>> However the DIM docs[1] indeed do say: "For topic branches shared within
>>>> the gpu/drm subsystem, base it on the latest drm-next branch." I think
>>>> the docs don't take into account the current period where drm-next is
>>>> being actively updated as we speak.
>>>>
>>>> Should we update the docs to use 'git merge-base' or something else?
>>>>
>>>> Regards, Joonas
>>>>
>>>> [1]: https://drm.pages.freedesktop.org/maintainer-tools/dim.html#cross-subsystem-topic-branches
>>>>
>>>>>
>>>>> are available in the Git repository at:
>>>>>
>>>>>     git://anongit.freedesktop.org/drm/drm-misc tags/topic/i915-ttm-2021-06-11
>>>>>
>>>>> for you to fetch changes up to cf3e3e86d77970211e0983130e896ae242601003:
>>>>>
>>>>>     drm/i915: Use ttm mmap handling for ttm bo's. (2021-06-11 10:53:25 +0200)
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> drm-misc and drm-intel pull request for topic/i915-ttm:
>>>>> - Convert i915 lmem handling to ttm.
>>>>> - Add a patch to temporarily add a driver_private member to vma_node.
>>>>> - Use this to allow mixed object mmap handling for i915.
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> Maarten Lankhorst (2):
>>>>>         drm/vma: Add a driver_private member to vma_node.
>>>>>         drm/i915: Use ttm mmap handling for ttm bo's.
>>>>>
>>>>> Thomas Hellström (2):
>>>>>         drm/i915/ttm: Introduce a TTM i915 gem object backend
>>>>>         drm/i915/lmem: Verify checks for lmem residency
>>>>>
>>>>>    drivers/gpu/drm/drm_gem.c                          |   9 -
>>>>>    drivers/gpu/drm/i915/Makefile                      |   1 +
>>>>>    drivers/gpu/drm/i915/display/intel_display.c       |   2 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_create.c         |   9 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_lmem.c           | 126 ++--
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_lmem.h           |   5 -
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_mman.c           |  83 ++-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_object.c         | 143 +++--
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_object.h         |  19 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_object_types.h   |  30 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_pages.c          |   3 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_region.c         |   6 +-
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_ttm.c            | 647 +++++++++++++++++++++
>>>>>    drivers/gpu/drm/i915/gem/i915_gem_ttm.h            |  48 ++
>>>>>    drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c |  90 +--
>>>>>    drivers/gpu/drm/i915/gt/intel_region_lmem.c        |   3 +-
>>>>>    drivers/gpu/drm/i915/i915_gem.c                    |   5 +-
>>>>>    drivers/gpu/drm/i915/intel_memory_region.c         |   1 -
>>>>>    drivers/gpu/drm/i915/intel_memory_region.h         |   1 -
>>>>>    drivers/gpu/drm/i915/intel_region_ttm.c            |   8 +-
>>>>>    drivers/gpu/drm/i915/intel_region_ttm.h            |  11 +-
>>>>>    drivers/gpu/drm/i915/selftests/igt_mmap.c          |  25 +-
>>>>>    drivers/gpu/drm/i915/selftests/igt_mmap.h          |  12 +-
>>>>>    include/drm/drm_vma_manager.h                      |   2 +-
>>>>>    24 files changed, 1039 insertions(+), 250 deletions(-)
>>>>>    create mode 100644 drivers/gpu/drm/i915/gem/i915_gem_ttm.c
>>>>>    create mode 100644 drivers/gpu/drm/i915/gem/i915_gem_ttm.h
>>
>> -- 
>> Thomas Zimmermann
>> Graphics Driver Developer
>> SUSE Software Solutions Germany GmbH
>> Maxfeldstr. 5, 90409 Nürnberg, Germany
>> (HRB 36809, AG Nürnberg)
>> Geschäftsführer: Felix Imendörffer
>>

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20210614/08a4ff97/attachment.sig>


More information about the Intel-gfx mailing list