From: Daniel Vetter <daniel@ffwll.ch>
To: James Simmons <jsimmons@infradead.org>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 00/23] kill drm cruft with fire
Date: Tue, 13 Dec 2011 19:32:10 +0100 [thread overview]
Message-ID: <20111213183210.GE4125@phenom.ffwll.local> (raw)
In-Reply-To: <alpine.LFD.2.02.1112071418210.27583@casper.infradead.org>
On Wed, Dec 07, 2011 at 02:19:39PM +0000, James Simmons wrote:
>
> > > >> Testing this on via would be awesome! Iirc I haven't changed anything in
> > > >> the via specific patches, but if it's more convenient you can also
> > > >> directly test my branch:
> > > >>
> > > >> http://cgit.freedesktop.org/~danvet/drm/log/?h=kill-with-fire
> > > >
> > > > Okay I tried the patches and it locked up the openchrome X server. I'm
> > > > going to try your branch tonight to see if it makes any difference. If it
> > > > still fails I will have to track down what the problem is.
> > >
> > > If you can bisect the issue, that would be awesome. Meanwhile my sis
> > > card arrived, so I'm hopefully get around to test that part of the
> > > series rsn. I'm traveling atm though, so response time will suffer a
> > > bit.
> >
> > Any updates on testing results?
>
> Yes I do. I'm using the patches you posted. Patches 1 to 10 work with no
> problems. Ist patch 11 that breaks the openchrome xorg driver. Have to
> do some digging to find the exact problem.
Thanks a lot for digging through these patches. As you've noted in patch
11, I don't set release_idlelock anywhere which results in the via driver
deadlocking when clients close. The same holds for the equivalent sis
patch. Updated patch series pushed to my kill-with-fire branch. Please
test and yell at me if it's still broken.
Yours, Daniel
--
Daniel Vetter
Mail: daniel@ffwll.ch
Mobile: +41 (0)79 365 57 48
prev parent reply other threads:[~2011-12-13 18:30 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-27 11:07 [PATCH 00/23] kill drm cruft with fire Daniel Vetter
2011-10-27 11:07 ` [PATCH 01/23] drm/sis: track obj->drm_fd relations in the driver Daniel Vetter
2011-10-27 11:07 ` [PATCH 02/23] drm/via: " Daniel Vetter
2011-12-07 16:21 ` James Simmons
2011-10-27 11:07 ` [PATCH 03/23] drm/sman: kill owner tracking interface functions Daniel Vetter
2011-12-07 16:21 ` James Simmons
2011-10-27 11:07 ` [PATCH 04/23] drm/sman: rip out owner tracking Daniel Vetter
2011-12-07 16:22 ` James Simmons
2011-10-27 11:07 ` [PATCH 05/23] drm/via: track user->memblock mapping with idr Daniel Vetter
2011-12-07 16:22 ` James Simmons
2011-10-27 11:07 ` [PATCH 06/23] drm/sis: " Daniel Vetter
2011-12-07 16:22 ` James Simmons
2011-10-27 11:07 ` [PATCH 07/23] drm/sman: kill user_hash_tab Daniel Vetter
2011-12-07 16:22 ` James Simmons
2011-10-27 11:07 ` [PATCH 08/23] drm/via: use drm_mm instead of drm_sman Daniel Vetter
2011-10-27 21:00 ` Daniel Vetter
2011-12-07 16:28 ` James Simmons
2011-12-12 22:51 ` Daniel Vetter
2011-12-07 16:23 ` James Simmons
2011-10-27 11:07 ` [PATCH 09/23] drm/sis: " Daniel Vetter
2011-10-28 22:47 ` Tormod Volden
2011-10-28 23:05 ` [PATCH] drm/sis: Fix building with CONFIG_FB_SIS/CONFIG_FB_SIS_MODULE Tormod Volden
2011-10-29 0:25 ` [PATCH 09/23] drm/sis: use drm_mm instead of drm_sman Daniel Vetter
2011-10-29 6:52 ` Tormod Volden
2011-10-29 9:44 ` Daniel Vetter
2011-12-07 16:23 ` James Simmons
2011-10-27 11:07 ` [PATCH 10/23] drm: kill drm_sman Daniel Vetter
2011-12-07 16:24 ` James Simmons
2011-10-27 11:07 ` [PATCH 11/23] drm/via: clean up reclaim_buffers Daniel Vetter
2011-12-08 0:26 ` James Simmons
2011-10-27 11:07 ` [PATCH 12/23] drm/sis: " Daniel Vetter
2011-10-27 11:07 ` [PATCH 13/23] drm: kill reclaim_buffers_idlelocked functions Daniel Vetter
2011-10-27 11:07 ` [PATCH 14/23] drm/i810: cleanup reclaim_buffers Daniel Vetter
2011-10-27 11:07 ` [PATCH 15/23] drm: kill reclaim_buffers_locked Daniel Vetter
2011-10-27 11:07 ` [PATCH 16/23] drm/savage: clean up reclaim_buffers Daniel Vetter
2011-11-06 10:18 ` Tormod Volden
2011-10-27 11:07 ` [PATCH 17/23] drm: kill reclaim_buffers callback Daniel Vetter
2011-10-27 11:07 ` [PATCH 18/23] drm: ditch strange DRIVER_DMA_QUEUE only error bail-out Daniel Vetter
2011-10-27 11:07 ` [PATCH 19/23] drm: kill dma queue support Daniel Vetter
2011-10-27 11:07 ` [PATCH 20/23] drm: unconditionally clean up dma buffers of closing clients Daniel Vetter
2011-10-27 11:07 ` [PATCH 21/23] drm: kill i915/i830 ids from drm_pciids.h Daniel Vetter
2011-10-27 11:07 ` [PATCH 22/23] drm: kill procfs support Daniel Vetter
2011-10-27 11:07 ` [PATCH 23/23] drm: kill a few unused remnants from drm_memory.c Daniel Vetter
2011-11-11 15:33 ` [PATCH 00/23] kill drm cruft with fire James Simmons
2011-11-11 17:07 ` Daniel Vetter
2011-11-14 16:10 ` James Simmons
2011-11-16 16:17 ` Daniel Vetter
2011-12-05 10:14 ` Daniel Vetter
2011-12-07 14:19 ` James Simmons
2011-12-13 18:32 ` Daniel Vetter [this message]
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=20111213183210.GE4125@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=jsimmons@infradead.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox