Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 08:29:51 -0800 Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would recommend specifying as few speci

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread David Howells
Tejun Heo wrote: > > > My question to all of you: Should system_nrt_wq be made freezable, or > > > should I create a new workqueue that is both freezable and > > > non-reentrant? And if I do, which of the usages above should be > > > converted to the new workqueue? > > > > As far as keys are

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 09:41:34 -0500 (EST) Alan Stern wrote: > Folks: > > I recently uncovered a bug in the block layer. It uses a workqueue to > periodically probe removable drives for media or other state changes, > and the workqueue it uses is system_nrt_wq. > > The bug is that system_nrt_wq

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread David Howells
Alan Stern wrote: > My question to all of you: Should system_nrt_wq be made freezable, or > should I create a new workqueue that is both freezable and > non-reentrant? And if I do, which of the usages above should be > converted to the new workqueue? As far as keys are concerned, it's only f

Re: [PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Hillf Danton
On Thu, Feb 16, 2012 at 8:01 PM, Daniel Vetter wrote: > @@ -416,17 +417,20 @@ static inline int fault_in_pages_writeable(char __user > *uaddr, int size) >         * Writing zeroes into userspace here is OK, because we know that if >         * the zero gets there, we'll be overwriting it. >      

Re: [PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Dan Carpenter
On Thu, Feb 16, 2012 at 11:03:29AM +0100, Daniel Vetter wrote: > gcc seems to get uber-anal recently about these things. > Sorry, I should have said that it's not a gcc warning, it's a smatch thing. But also it's not uber-anal. It's the exact level of anality which is required to make the == -1

[Bug 42787] New: Video flickers on X1200 / RS690 over DVI

2012-02-16 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=42787 Summary: Video flickers on X1200 / RS690 over DVI Product: Drivers Version: 2.5 Kernel Version: 3.0.0 Platform: All OS/Version: Linux Tree: Mainline Status: NEW

[Bug 39832] HDA ATI HDMI: no sound with 3.0 - 2.6.39.3 works

2012-02-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=39832 --- Comment #19 from Jonathan Nieder 2012-02-17 05:38:13 --- (In reply to comment #18) > Is this still a problem in 3.2 / 3.3-rc1? Here's a report using 3.0 and 3.2.4: http://bugs.debian.org/659673 -- Configure bugmail: https://bugzilla.k

[PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Hillf Danton
On Thu, Feb 16, 2012 at 8:01 PM, Daniel Vetter wrote: > @@ -416,17 +417,20 @@ static inline int fault_in_pages_writeable(char __user > *uaddr, int size) > ? ? ? ? * Writing zeroes into userspace here is OK, because we know that if > ? ? ? ? * the zero gets there, we'll be overwriting it. > ? ? ?

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Benjamin Herrenschmidt
On Thu, 2012-02-16 at 09:26 +0100, Michel D?nzer wrote: > > > For that one, I'd try adding some more debugging output to > > > radeon_get_bios() to find out which method it ends up using to > retrieve > > > the ROM contents, and why it doesn't look like it's an ATOM BIOS. > > > > Is it an Apple ca

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Benjamin Herrenschmidt
On Thu, 2012-02-16 at 08:50 +0100, Michel D?nzer wrote: > > > The second case with no firmware is a bit more surprising, looks like > > > something bad happened on the PCI express bus or the kernel tried to > > > access something that the card rejected (target abort or PCIe > > > equivalent most li

[mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Chen Jie
? 2012?2?16? ??5:21?Chen Jie ??? > Hi, > > ? 2012?2?15? ??11:53?Jerome Glisse ??? >> To me it looks like the CP is trying to fetch memory but the >> GPU memory controller fail to fullfill cp request. Did you >> check the PCI configuration before & after (when things don't >> work) My best guest i

[mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Chen Jie
Hi, ? 2012?2?15? ??11:53?Jerome Glisse ??? > To me it looks like the CP is trying to fetch memory but the > GPU memory controller fail to fullfill cp request. Did you > check the PCI configuration before & after (when things don't > work) My best guest is PCI bus mastering is no properly working

Mode validation outside of connector or encoder (i.e. at CRTC or drm_driver level)

2012-02-16 Thread Matt Sealey
On Thu, Feb 16, 2012 at 9:51 AM, Adam Jackson wrote: > On Wed, 2012-02-15 at 22:43 -0600, Matt Sealey wrote: >> Quick question; if I want to validate a mode given to me by a >> connector/encoder as workable or not before I am going through the >> code to actually set that mode, how would I do this

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread David Howells
Tejun Heo wrote: > > > My question to all of you: Should system_nrt_wq be made freezable, or > > > should I create a new workqueue that is both freezable and > > > non-reentrant? And if I do, which of the usages above should be > > > converted to the new workqueue? > > > > As far as keys are

[Bug 45018] [bisected] rendering regression since added support for virtual address space on cayman v11

2012-02-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45018 --- Comment #18 from Alexandre Demers 2012-02-16 16:31:44 UTC --- Created attachment 57178 --> https://bugs.freedesktop.org/attachment.cgi?id=57178 dmesg with bo conflict Latest dmesg with the bo conflicts. -- Configure bugmail: https://bug

[Bug 45018] [bisected] rendering regression since added support for virtual address space on cayman v11

2012-02-16 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=45018 --- Comment #18 from Alexandre Demers 2012-02-16 16:31:44 UTC --- Created attachment 57178 --> https://bugs.freedesktop.org/attachment.cgi?id=57178 dmesg with bo conflict Latest dmesg with the bo conflicts. -- Configure bugmail: https://bug

[Bug 45018] [bisected] rendering regression since added support for virtual address space on cayman v11

2012-02-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45018 --- Comment #17 from Alexandre Demers 2012-02-16 16:19:08 PST --- I was not able to find the root of the problem. However, I have many message telling me the following: Radon :01:00.0: no 880214144000 via 0x02400 conflicts with (no

[PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
On Thu, Feb 16, 2012 at 09:32:08PM +0800, Hillf Danton wrote: > On Thu, Feb 16, 2012 at 8:01 PM, Daniel Vetter > wrote: > > @@ -416,17 +417,20 @@ static inline int fault_in_pages_writeable(char > > __user *uaddr, int size) > > ? ? ? ? * Writing zeroes into userspace here is OK, because we know t

[Bug 42611] [865g] webgl enabled pages crash Firefox and mesa

2012-02-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42611 --- Comment #5 from Cesare Leonardi 2012-02-16 15:54:01 UTC --- Will, have you already upgraded to Mesa 8 from experimental? The problem is not resolved for me (i855) and i do not reopen this bug only because your bug is specific to Drivers/DR

[Bug 42611] [865g] webgl enabled pages crash Firefox and mesa

2012-02-16 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=42611 --- Comment #5 from Cesare Leonardi 2012-02-16 15:54:01 UTC --- Will, have you already upgraded to Mesa 8 from experimental? The problem is not resolved for me (i855) and i do not reopen this bug only because your bug is specific to Drivers/DR

Kernel Display and Video API Consolidation mini-summit at ELC 2012 - Notes

2012-02-16 Thread Laurent Pinchart
Hello everybody, First of all, I would like to thank all the attendees for their participation in the mini-summit that helped make the meeting a success. Here are my consolidated notes that cover both the Linaro Connect meeting and the ELC meeting. They're also available at http://www.ideasonb

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread David Howells
Alan Stern wrote: > My question to all of you: Should system_nrt_wq be made freezable, or > should I create a new workqueue that is both freezable and > non-reentrant? And if I do, which of the usages above should be > converted to the new workqueue? As far as keys are concerned, it's only f

Re: Business proposal

2012-02-16 Thread Laurent Pinchart
Hi Koen, On Thursday 16 February 2012 23:46:51 Beel, Koen wrote: > Hi Laurent, > > Would it be possible to give us your number on which you can be reached next > Monday? I prefer to init the call from our side as we will join the call > with two people and we will be in a meeting room... (so not

Re: Mode validation outside of connector or encoder (i.e. at CRTC or drm_driver level)

2012-02-16 Thread Matt Sealey
On Thu, Feb 16, 2012 at 9:51 AM, Adam Jackson wrote: > On Wed, 2012-02-15 at 22:43 -0600, Matt Sealey wrote: >> Quick question; if I want to validate a mode given to me by a >> connector/encoder as workable or not before I am going through the >> code to actually set that mode, how would I do this

[RFC] drm: atomic mode set API

2012-02-16 Thread Ville Syrjälä
On Thu, Feb 16, 2012 at 12:12:49AM +0100, Daniel Vetter wrote: > On Wed, Feb 15, 2012 at 05:59:45PM -0500, Adam Jackson wrote: > > On 2/15/12 5:42 PM, Jesse Barnes wrote: > > > > >+#define DRM_SET_CONFIG_TEST (1<<0) /* don't change the config, just test > > >it for validity */ This would be OK f

[PATCH 2/2] nouveau: implement precise vblank timestamping

2012-02-16 Thread Ben Skeggs
On Wed, 2012-02-15 at 23:55 +0100, Mario Kleiner wrote: > From: Lucas Stach > > This patch implements the drivers hooks needed for precise vblank > timestamping. This is a complementary patch to Mario Kleiner's > patches to improve swap scheduling. With the complete > patchset applied nouveau wil

[PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Dan Carpenter
6 bytes Desc: Digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120216/007fc5b2/attachment.pgp>

[Bug 42787] New: Video flickers on X1200 / RS690 over DVI

2012-02-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42787 Summary: Video flickers on X1200 / RS690 over DVI Product: Drivers Version: 2.5 Kernel Version: 3.0.0 Platform: All OS/Version: Linux Tree: Mainline Status: NEW

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 08:29:51 -0800 Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would recommend specifying as few speci

[PATCH 03/14] drm: add helper to clflush a virtual address range

2012-02-16 Thread Daniel Vetter
Useful when the page is already mapped to copy date in/out. For -stable because the next patch (fixing phys obj pwrite) needs this little helper function. Cc: dri-devel at lists.freedesktop.org Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_cache.c | 23 +++ include/

[PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
drm/i915 wants to read/write more than one page in its fastpath and hence needs to prefault more than PAGE_SIZE bytes. I've checked the callsites and they all already clamp size when calling fault_in_pages_* to the same as for the subsequent __copy_to|from_user and hence don't rely on the implicit

[PATCH] extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
Hi all, drm/i915 has write/read paths to upload/download data to/from gpu buffer objects. For a bunch of reasons we have special fastpaths with decent setup costs, so when we fall back to the slow-path we don't fully recover to the fastest fast-path when grabbing our locks again. This is also in p

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 11:37:33AM -0500, Alan Stern wrote: > > Um. I don't think I can audit all the calls in the kernel that submit > > block requests and determine which ones need to be allowed while a > > system sleep is in progress. > > ??

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would recommend specifying as few special attribute as

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, (cc'ing Rafael and Jens) > > On Thu, Feb 16, 2012 at 09:41:34AM -0500, Alan Stern wrote: > > My question to all of you: Should system_nrt_wq be made freezable, or > > should I create a new workqueue that is both freezable and > > non-reentrant? An

[Bug 46161] New: [regression] Commit 356eb0aadb causes compiz redraw issues

2012-02-16 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=46161 Bug #: 46161 Summary: [regression] Commit 356eb0aadb causes compiz redraw issues Classification: Unclassified Product: Mesa Version: unspecified Platform: x86-64 (AMD6

[mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Jerome Glisse
On Thu, Feb 16, 2012 at 05:21:10PM +0800, Chen Jie wrote: > Hi, > > ? 2012?2?15? ??11:53?Jerome Glisse ??? > > To me it looks like the CP is trying to fetch memory but the > > GPU memory controller fail to fullfill cp request. Did you > > check the PCI configuration before & after (when things do

[PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Paul Menzel
text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120216/eac6e9df/attachment.pgp>

[PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Daniel Vetter
gcc seems to get uber-anal recently about these things. Reported-by: Dan Carpenter Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/i915_drv.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index b

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, Jeff. On Thu, Feb 16, 2012 at 01:59:45PM -0500, Jeff Layton wrote: > The other problem here is that we really ought to be submitting the > write completion handler to a workqueue that has WQ_MEM_RECLAIM set. > Since none of the public wq's have that then I guess we'll have to make > our own

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, Jeff. On Thu, Feb 16, 2012 at 01:59:45PM -0500, Jeff Layton wrote: > The other problem here is that we really ought to be submitting the > write completion handler to a workqueue that has WQ_MEM_RECLAIM set. > Since none of the public wq's have that then I guess we'll have to make > our own

[PATCH] Revert "drivers/gpu/drm/i915/intel_overlay.c needs seq_file.h"

2012-02-16 Thread Daniel Vetter
On Wed, Feb 15, 2012 at 06:53:55PM +0100, Danny Kukawka wrote: > This reverts commit e167976ee7f5fe4b80f7e8f55e087f6c67cf9562, > Since this was already fixed in commit > 3bd3c9329973a93fa3ef5e9840f2fd6fa2889e3f some days before this > commit cause seq_file.h to be included twice. > > Signed-off-by

Mode validation outside of connector or encoder (i.e. at CRTC or drm_driver level)

2012-02-16 Thread Adam Jackson
: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120216/c67623ea/attachment.pgp>

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Jeff Layton
On Thu, 16 Feb 2012 09:41:34 -0500 (EST) Alan Stern wrote: > Folks: > > I recently uncovered a bug in the block layer. It uses a workqueue to > periodically probe removable drives for media or other state changes, > and the workqueue it uses is system_nrt_wq. > > The bug is that system_nrt_wq

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
Folks: I recently uncovered a bug in the block layer. It uses a workqueue to periodically probe removable drives for media or other state changes, and the workqueue it uses is system_nrt_wq. The bug is that system_nrt_wq is not freezable, so it keeps on running even while the system is in the pr

drm/i915: switch ring->id to be a real id

2012-02-16 Thread Dan Carpenter
Hello Daniel Vetter, The patch 96154f2faba5: "drm/i915: switch ring->id to be a real id" from Dec 14, 2011, leads to the following warning: drivers/gpu/drm/i915/i915_debugfs.c:688 print_error_buffers() error: err->ring is never equal to -1 (wrong type 0 - 15). @@ -724,6 +724,7 @@ static

[git pull] drm fixes

2012-02-16 Thread Dave Airlie
Hi Linus, 3 radeon fixes, I have some exynos fixes to push later but I'll queue them separately once I've looked them over a bit. Dave. The following changes since commit ce5afed937f0a823d3b00c9459409c3f5f2fbd5d: Merge git://git.samba.org/sfrench/cifs-2.6 (2012-02-13 20:34:44 -0800) are av

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Michel Dänzer
On Don, 2012-02-16 at 19:02 +1100, Benjamin Herrenschmidt wrote: > On Thu, 2012-02-16 at 08:50 +0100, Michel D?nzer wrote: > > > > The second case with no firmware is a bit more surprising, looks like > > > > something bad happened on the PCI express bus or the kernel tried to > > > > access somet

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 11:37:33AM -0500, Alan Stern wrote: > > Um. I don't think I can audit all the calls in the kernel that submit > > block requests and determine which ones need to be allowed while a > > system sleep is in progress. > > ??

Re: [RFC] drm: atomic mode set API

2012-02-16 Thread Jesse Barnes
On Thu, 16 Feb 2012 04:46:39 -0800 (PST) Jakob Bornecrantz wrote: > - Original Message - > > Many of us really want (and need) a way to set the whole display > > configuration atomically, as well as test a global config. > > > > In talking with Rob and Alex here at ELC a bit, I think thi

[RFC] drm: atomic mode set API

2012-02-16 Thread Jesse Barnes
On Thu, 16 Feb 2012 04:46:39 -0800 (PST) Jakob Bornecrantz wrote: > - Original Message - > > Many of us really want (and need) a way to set the whole display > > configuration atomically, as well as test a global config. > > > > In talking with Rob and Alex here at ELC a bit, I think thi

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Michel Dänzer
On Don, 2012-02-16 at 01:05 +, acrux wrote: > On Thu, 16 Feb 2012 07:28:10 +1100 > Benjamin Herrenschmidt wrote: > > > On Wed, 2012-02-15 at 08:39 +0100, Michel D?nzer wrote: > > > > > > > Btw, to sum up the list of Power Architecture machines with > > > > > PCIE that aim to be a desktop/w

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, On Thu, Feb 16, 2012 at 11:37:33AM -0500, Alan Stern wrote: > Um. I don't think I can audit all the calls in the kernel that submit > block requests and determine which ones need to be allowed while a > system sleep is in progress. ??? we need to do that anyway and the ones which should g

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, On Thu, Feb 16, 2012 at 11:37:33AM -0500, Alan Stern wrote: > Um. I don't think I can audit all the calls in the kernel that submit > block requests and determine which ones need to be allowed while a > system sleep is in progress. ??? we need to do that anyway and the ones which should g

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, > > On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > > These should all be freezable and we might even be able to get away > > with WQ_UNBOUND for some of these. > > In general, I would recommend specifying as few special attribute as

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
On Thu, Feb 16, 2012 at 04:35:51PM +, David Howells wrote: > The key_garbage_collector work item is marked neither freezable nor > unfreezable that I can see. Heh, was too brief apparently. :) I was trying to say that if it doesn't require freezing, please don't put it on a freezable workqueu

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
On Thu, 16 Feb 2012, Tejun Heo wrote: > Hello, (cc'ing Rafael and Jens) > > On Thu, Feb 16, 2012 at 09:41:34AM -0500, Alan Stern wrote: > > My question to all of you: Should system_nrt_wq be made freezable, or > > should I create a new workqueue that is both freezable and > > non-reentrant? An

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
On Thu, Feb 16, 2012 at 04:35:51PM +, David Howells wrote: > The key_garbage_collector work item is marked neither freezable nor > unfreezable that I can see. Heh, was too brief apparently. :) I was trying to say that if it doesn't require freezing, please don't put it on a freezable workqueu

Re: [mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Jerome Glisse
On Thu, Feb 16, 2012 at 05:21:10PM +0800, Chen Jie wrote: > Hi, > > 在 2012年2月15日 下午11:53,Jerome Glisse 写道: > > To me it looks like the CP is trying to fetch memory but the > > GPU memory controller fail to fullfill cp request. Did you > > check the PCI configuration before & after (when things do

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > These should all be freezable and we might even be able to get away > with WQ_UNBOUND for some of these. In general, I would recommend specifying as few special attribute as possible. If WQ_UNBOUND is necessary (large amount

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, On Thu, Feb 16, 2012 at 10:27:28AM -0500, Jeff Layton wrote: > These should all be freezable and we might even be able to get away > with WQ_UNBOUND for some of these. In general, I would recommend specifying as few special attribute as possible. If WQ_UNBOUND is necessary (large amount

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
On Thu, Feb 16, 2012 at 03:22:24PM +, David Howells wrote: > Alan Stern wrote: > > > My question to all of you: Should system_nrt_wq be made freezable, or > > should I create a new workqueue that is both freezable and > > non-reentrant? And if I do, which of the usages above should be > >

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
On Thu, Feb 16, 2012 at 03:22:24PM +, David Howells wrote: > Alan Stern wrote: > > > My question to all of you: Should system_nrt_wq be made freezable, or > > should I create a new workqueue that is both freezable and > > non-reentrant? And if I do, which of the usages above should be > >

Re: system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, (cc'ing Rafael and Jens) On Thu, Feb 16, 2012 at 09:41:34AM -0500, Alan Stern wrote: > My question to all of you: Should system_nrt_wq be made freezable, or > should I create a new workqueue that is both freezable and > non-reentrant? And if I do, which of the usages above should be > c

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Tejun Heo
Hello, (cc'ing Rafael and Jens) On Thu, Feb 16, 2012 at 09:41:34AM -0500, Alan Stern wrote: > My question to all of you: Should system_nrt_wq be made freezable, or > should I create a new workqueue that is both freezable and > non-reentrant? And if I do, which of the usages above should be > c

Re: Mode validation outside of connector or encoder (i.e. at CRTC or drm_driver level)

2012-02-16 Thread Adam Jackson
On Wed, 2012-02-15 at 22:43 -0600, Matt Sealey wrote: > Quick question; if I want to validate a mode given to me by a > connector/encoder as workable or not before I am going through the > code to actually set that mode, how would I do this? I think this is an API bug. The crtc really should get

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Benjamin Herrenschmidt
On Wed, 2012-02-15 at 08:39 +0100, Michel D?nzer wrote: > > > Btw, to sum up the list of Power Architecture machines with PCIE that > > > aim to be a desktop/workstation: Apple iMac G5 (iSight), Apple PowerMac > > > Quad G5, YDL Powerstation 2x970MP and Acube Sam460ex . And the last > > > two, on

[PATCH 4/5] drm/vgem: getparam ioctl

2012-02-16 Thread Dave Airlie
On Wed, Feb 15, 2012 at 9:22 PM, Adam Jackson wrote: > On 2/8/12 6:19 PM, Ben Widawsky wrote: >> >> Similar to i915, it's nice to be able to query this device uniquely and >> get some info >> >> Signed-off-by: Ben Widawsky > > > So, this is actually not especially useful as written. ?You'd like to

Re: [PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
On Thu, Feb 16, 2012 at 09:32:08PM +0800, Hillf Danton wrote: > On Thu, Feb 16, 2012 at 8:01 PM, Daniel Vetter wrote: > > @@ -416,17 +417,20 @@ static inline int fault_in_pages_writeable(char > > __user *uaddr, int size) > >         * Writing zeroes into userspace here is OK, because we know that

system_nrt_wq, system suspend, and the freezer

2012-02-16 Thread Alan Stern
Folks: I recently uncovered a bug in the block layer. It uses a workqueue to periodically probe removable drives for media or other state changes, and the workqueue it uses is system_nrt_wq. The bug is that system_nrt_wq is not freezable, so it keeps on running even while the system is in the pr

Re: [PATCH 1/5] drm/vgem: virtual GEM provider

2012-02-16 Thread Jakob Bornecrantz
- Original Message - > From: Adam Jackson > > This is about as minimal of a virtual GEM service as possible. My > plan is to use this with non-native-3D hardware for buffer sharing > between X and DRI. > > The current drisw winsys assumes an unmodified X server, which means > it's hopel

[PATCH 1/5] drm/vgem: virtual GEM provider

2012-02-16 Thread Jakob Bornecrantz
- Original Message - > From: Adam Jackson > > This is about as minimal of a virtual GEM service as possible. My > plan is to use this with non-native-3D hardware for buffer sharing > between X and DRI. > > The current drisw winsys assumes an unmodified X server, which means > it's hopel

Re: [RFC] drm: atomic mode set API

2012-02-16 Thread Jakob Bornecrantz
- Original Message - > Many of us really want (and need) a way to set the whole display > configuration atomically, as well as test a global config. > > In talking with Rob and Alex here at ELC a bit, I think this may be > enough: > > diff --git a/include/drm/drm_mode.h b/include/drm/drm_

[RFC] drm: atomic mode set API

2012-02-16 Thread Jakob Bornecrantz
- Original Message - > Many of us really want (and need) a way to set the whole display > configuration atomically, as well as test a global config. > > In talking with Rob and Alex here at ELC a bit, I think this may be > enough: > > diff --git a/include/drm/drm_mode.h b/include/drm/drm_

Re: [RFC] drm: atomic mode set API

2012-02-16 Thread Ville Syrjälä
On Thu, Feb 16, 2012 at 12:12:49AM +0100, Daniel Vetter wrote: > On Wed, Feb 15, 2012 at 05:59:45PM -0500, Adam Jackson wrote: > > On 2/15/12 5:42 PM, Jesse Barnes wrote: > > > > >+#define DRM_SET_CONFIG_TEST (1<<0) /* don't change the config, just test > > >it for validity */ This would be OK f

[PATCH 03/14] drm: add helper to clflush a virtual address range

2012-02-16 Thread Daniel Vetter
Useful when the page is already mapped to copy date in/out. For -stable because the next patch (fixing phys obj pwrite) needs this little helper function. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_cache.c | 23 +++ include/drm

[PATCH] mm: extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
drm/i915 wants to read/write more than one page in its fastpath and hence needs to prefault more than PAGE_SIZE bytes. I've checked the callsites and they all already clamp size when calling fault_in_pages_* to the same as for the subsequent __copy_to|from_user and hence don't rely on the implicit

[PATCH] extend prefault helpers to fault in more than PAGE_SIZE

2012-02-16 Thread Daniel Vetter
Hi all, drm/i915 has write/read paths to upload/download data to/from gpu buffer objects. For a bunch of reasons we have special fastpaths with decent setup costs, so when we fall back to the slow-path we don't fully recover to the fastest fast-path when grabbing our locks again. This is also in p

[Bug 46161] New: [regression] Commit 356eb0aadb causes compiz redraw issues

2012-02-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=46161 Bug #: 46161 Summary: [regression] Commit 356eb0aadb causes compiz redraw issues Classification: Unclassified Product: Mesa Version: unspecified Platform: x86-64 (AMD6

Re: [PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Paul Menzel
Am Donnerstag, den 16.02.2012, 11:03 +0100 schrieb Daniel Vetter: > gcc seems to get uber-anal recently about these things. which was introduced by the following commit. 96154f2faba5: "drm/i915: switch ring->id to be a real id" > Reported-by: Dan Carpenter The URL of the report is the

Re: [mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Chen Jie
在 2012年2月16日 下午5:21,Chen Jie 写道: > Hi, > > 在 2012年2月15日 下午11:53,Jerome Glisse 写道: >> To me it looks like the CP is trying to fetch memory but the >> GPU memory controller fail to fullfill cp request. Did you >> check the PCI configuration before & after (when things don't >> work) My best guest i

[PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Daniel Vetter
gcc seems to get uber-anal recently about these things. Reported-by: Dan Carpenter Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/i915_drv.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index b

Re: [PATCH] Revert "drivers/gpu/drm/i915/intel_overlay.c needs seq_file.h"

2012-02-16 Thread Daniel Vetter
On Wed, Feb 15, 2012 at 06:53:55PM +0100, Danny Kukawka wrote: > This reverts commit e167976ee7f5fe4b80f7e8f55e087f6c67cf9562, > Since this was already fixed in commit > 3bd3c9329973a93fa3ef5e9840f2fd6fa2889e3f some days before this > commit cause seq_file.h to be included twice. > > Signed-off-by

[RFC] drm: atomic mode set API

2012-02-16 Thread Rob Clark
On Wed, Feb 15, 2012 at 4:59 PM, Adam Jackson wrote: > On 2/15/12 5:42 PM, Jesse Barnes wrote: > >> +#define DRM_SET_CONFIG_TEST (1<<0) /* don't change the config, just test >> it for validity */ >> + >> +struct drm_mode_set_config { >> + ? ? ? __u64 crtcs; >> + ? ? ? __u64 crtc_fbs; >> + ? ? ? __

[git pull] drm fixes

2012-02-16 Thread Dave Airlie
Hi Linus, 3 radeon fixes, I have some exynos fixes to push later but I'll queue them separately once I've looked them over a bit. Dave. The following changes since commit ce5afed937f0a823d3b00c9459409c3f5f2fbd5d: Merge git://git.samba.org/sfrench/cifs-2.6 (2012-02-13 20:34:44 -0800) are av

Re: [mipsel+rs780e]Occasionally "GPU lockup" after resuming from suspend.

2012-02-16 Thread Chen Jie
Hi, 在 2012年2月15日 下午11:53,Jerome Glisse 写道: > To me it looks like the CP is trying to fetch memory but the > GPU memory controller fail to fullfill cp request. Did you > check the PCI configuration before & after (when things don't > work) My best guest is PCI bus mastering is no properly working

linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread acrux
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On Thu, 16 Feb 2012 07:28:10 +1100 Benjamin Herrenschmidt wrote: > On Wed, 2012-02-15 at 08:39 +0100, Michel D?nzer wrote: > > > > > Btw, to sum up the list of Power Architecture machines with > > > > PCIE that aim to be a desktop/workstation: Appl

Re: linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Benjamin Herrenschmidt
On Thu, 2012-02-16 at 09:26 +0100, Michel Dänzer wrote: > > > For that one, I'd try adding some more debugging output to > > > radeon_get_bios() to find out which method it ends up using to > retrieve > > > the ROM contents, and why it doesn't look like it's an ATOM BIOS. > > > > Is it an Apple ca

re: drm/i915: switch ring->id to be a real id

2012-02-16 Thread Dan Carpenter
Hello Daniel Vetter, The patch 96154f2faba5: "drm/i915: switch ring->id to be a real id" from Dec 14, 2011, leads to the following warning: drivers/gpu/drm/i915/i915_debugfs.c:688 print_error_buffers() error: err->ring is never equal to -1 (wrong type 0 - 15). @@ -724,6 +724,7 @@ static

Re: linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Michel Dänzer
On Don, 2012-02-16 at 19:02 +1100, Benjamin Herrenschmidt wrote: > On Thu, 2012-02-16 at 08:50 +0100, Michel Dänzer wrote: > > > > The second case with no firmware is a bit more surprising, looks like > > > > something bad happened on the PCI express bus or the kernel tried to > > > > access somet

[BUG] Intel HD Graphic QM57 chipset: Dell U3011 monitor turns to power saving mode

2012-02-16 Thread Daniel Vetter
On Wed, Feb 15, 2012 at 03:16:52PM -0500, Mathieu Desnoyers wrote: > * Mathieu Desnoyers (mathieu.desnoyers at efficios.com) wrote: > > Hi Keith, > > > > * Keith Packard (keithp at keithp.com) wrote: > > > On Sat, 21 Jan 2012 13:56:21 -0500, Mathieu Desnoyers > > at efficios.com> wrote: > > > >

[RFC] drm: atomic mode set API

2012-02-16 Thread Daniel Vetter
On Wed, Feb 15, 2012 at 05:59:45PM -0500, Adam Jackson wrote: > On 2/15/12 5:42 PM, Jesse Barnes wrote: > > >+#define DRM_SET_CONFIG_TEST (1<<0) /* don't change the config, just test it > >for validity */ > >+ > >+struct drm_mode_set_config { > >+__u64 crtcs; > >+__u64 crtc_fbs; > >+_

Re: linux-3.3-rc2 and radeon kms failure on ppc32 with Radeon X1650PRO pcie

2012-02-16 Thread Benjamin Herrenschmidt
On Thu, 2012-02-16 at 08:50 +0100, Michel Dänzer wrote: > > > The second case with no firmware is a bit more surprising, looks like > > > something bad happened on the PCI express bus or the kernel tried to > > > access something that the card rejected (target abort or PCIe > > > equivalent most li