From: Christian Zigotzky <chzigotzky@xenosoft.de>
To: Ilia Mirkin <imirkin@alum.mit.edu>,
Maling list - DRI developers <dri-devel@lists.freedesktop.org>,
alexdeucher@gmail.com
Subject: Hardware 3D acceleration doesn't work anymore with the latest git kernel
Date: Fri, 24 Nov 2017 15:29:29 +0100 [thread overview]
Message-ID: <348d480d-aa6c-4861-8db7-7dc09fed9c27@xenosoft.de> (raw)
In-Reply-To: <CAKb7UvgLvFHULb26-_BnjJWSzVR0YxWPkiQoBmyZ3KpcRDx+Lw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 768 bytes --]
Hi All,
I bisected today and the first bad commit is:
a4dec819c8bba6365eb893a4ca88db4dd1210110 (drm/ttm: Add helper functions
to populate/map in one call (v2)) [1]
Please find attached the git bisect log.
Thanks,
Christian
[1]
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a4dec819c8bba6365eb893a4ca88db4dd1210110
On 22.11.2017 14:45, Ilia Mirkin wrote:
>
> I thought I covered that...
>
> "I'd recommend doing a proper bisect (search for "git bisect", there
> are tons of guides). That will identify the commit that's actually
> responsible."
>
> -ilia
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
[-- Attachment #2: bisect-4.15-radeon --]
[-- Type: text/plain, Size: 4410 bytes --]
git bisect start
git bisect good 894025f24bd028942da3e602b87d9f7223109b14 (Linux 4.15 alpha1 Mon Nov 13 21:14:07 2017 -0800)
git bisect bad e60e1ee60630cafef5e430c2ae364877e061d980
Output:
Bisecting: 2555 revisions left to test after this (roughly 12 steps)
[5bbcc0f595fadb4cac0eddc4401035ec0bd95b09] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
git bisect good
Output:
Bisecting: 1277 revisions left to test after this (roughly 10 steps)
[9a45ddaaa674aa103cd74a0df9a3f9c2c8fb3124] drm/nouveau/mmu: implement page table cache
git bisect bad
Output:
Bisecting: 638 revisions left to test after this (roughly 9 steps)
[d6c0511300dcff19969844495ba293c4efb50b42] drm/i915/execlists: Distinguish the incomplete context notifies
git bisect bad
Output:
Bisecting: 373 revisions left to test after this (roughly 8 steps)
[ae7617f0ef1820be033eef93859a6bb6174a843f] drm/i915: Allow optimized platform checks
git bisect good
Output:
Bisecting: 186 revisions left to test after this (roughly 8 steps)
[6e2e216fadd80b4280783bb78e543593ebf2cb69] drm/amdgpu:use formal register to trigger hdp invalidate
git bisect bad
Output:
Bisecting: 93 revisions left to test after this (roughly 7 steps)
[b72cf4fca2bb786e20864b5e8755105aa9626fb4] drm/amdgpu: move taking mmap_sem into get_user_pages v2
git bisect bad
Output:
Bisecting: 46 revisions left to test after this (roughly 6 steps)
[c5927537dd5706b17affa8aeea28c7b19c8fbb42] drm/amd: Remove null check before kfree
git bisect bad
Output:
Bisecting: 22 revisions left to test after this (roughly 5 steps)
[e719d5169f75ead3c05329b4125afb67b4f33eba] drm/amd/include: Add hdmi_redriver_set to atomfirmware
git bisect good
Output:
Bisecting: 11 revisions left to test after this (roughly 4 steps)
[925d5d798f465671c6b8011e80c636da46ef1a16] drm/amdgpu/gfx8: apply dynamic cu mask to APUs as well
git bisect bad
Output:
Bisecting: 5 revisions left to test after this (roughly 3 steps)
[db95e2185523ee9d46a13ceee37bffe8442d2e1c] drm/amdgpu: Add debugfs file for VBIOS and version
git bisect bad
Output:
Bisecting: 2 revisions left to test after this (roughly 1 step)
[7405e0dad4c75b33976ddd997513635d7a0204b1] drm/amd/amdgpu: Use new TTM populate/map helper function
CC drivers/gpu/drm/ttm/ttm_page_alloc.o
drivers/gpu/drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of ‘ttm_populate_and_map_pages’
int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/ttm/ttm_page_alloc.c:49:0:
./include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of ‘ttm_populate_and_map_pages’ was here
static inline int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of ‘ttm_unmap_and_unpopulate_pages’
void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt *tt)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/ttm/ttm_page_alloc.c:49:0:
./include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of ‘ttm_unmap_and_unpopulate_pages’ was here
static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt *tt)
git bisect bad
Output:
Bisecting: 0 revisions left to test after this (roughly 0 steps)
[a4dec819c8bba6365eb893a4ca88db4dd1210110] drm/ttm: Add helper functions to populate/map in one call (v2)
git bisect bad
Output:
a4dec819c8bba6365eb893a4ca88db4dd1210110 is the first bad commit
commit a4dec819c8bba6365eb893a4ca88db4dd1210110
Author: Tom St Denis <tom.stdenis@amd.com>
Date: Fri Aug 18 10:04:57 2017 -0400
drm/ttm: Add helper functions to populate/map in one call (v2)
These functions replace a section of common code found
in radeon/amdgpu drivers (and possibly others) as part
of the ttm_tt_*populate() callbacks.
v2: squash in fix for sw iommu from Tom
Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
:040000 040000 04b995fa3def96d5e983d0d5bc728e00b7617fe9 616ef24927f707cfaeecfc4748630f49b52adb42 M drivers
:040000 040000 61abf936f7ea760e19c5cef8a214ffada5ea3aeb 595278390bc27db92848ce8aaa255a1201c2af75 M include
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-11-24 15:14 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-20 18:08 Hardware 3D acceleration doesn't work anymore with the latest git kernel Christian Zigotzky
2017-11-21 4:38 ` Alex Deucher
2017-11-21 11:32 ` Christian Zigotzky
2017-11-22 13:20 ` Christian Zigotzky
2017-11-22 13:27 ` Ilia Mirkin
2017-11-22 13:40 ` Christian Zigotzky
2017-11-22 13:45 ` Ilia Mirkin
2017-11-22 14:53 ` Christian Zigotzky
2017-11-24 14:29 ` Christian Zigotzky [this message]
2017-11-24 16:09 ` Michel Dänzer
2017-11-24 19:08 ` Christian Zigotzky
2017-11-24 19:13 ` Christian Zigotzky
2017-11-24 19:14 ` Ilia Mirkin
2017-11-24 20:58 ` Christian Zigotzky
2017-11-27 11:02 ` Michel Dänzer
2017-11-27 11:50 ` Christian König
2017-11-27 12:02 ` Michel Dänzer
2017-11-27 12:17 ` Tom St Denis
2017-11-27 14:53 ` Michel Dänzer
2017-11-28 5:43 ` Christian Zigotzky
2017-11-28 9:40 ` Michel Dänzer
2017-11-28 23:48 ` Christian Zigotzky
2017-11-27 12:20 ` Christian König
2017-11-29 8:50 ` Michel Dänzer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=348d480d-aa6c-4861-8db7-7dc09fed9c27@xenosoft.de \
--to=chzigotzky@xenosoft.de \
--cc=alexdeucher@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=imirkin@alum.mit.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.