[Nouveau] Nouveau on a Jetson TK1 problems

2016-10-16 Thread ben
I hope this is a good mailing list for this problem. I have a Jetson TK1 board and I am trying to get the nouveau drivers running on it without success. I am using the following: Nouveau linux kernel 4.8.0-rc4+ Nouveau DRM git master xf86-video-nouveau git master Gnurou xserver git branch GK20

Re: [Nouveau] CRTC video scanout position for high precision vblank timestamping?

2010-12-29 Thread Ben Skeggs
Francisco Jerez will reply, he might know > > the exact registers. > > > > I wish you much fun and happiness. > > > > Sincerely, > > > > Maarten. > > > > -- > > Far away from the primal instinct, the song seems to fade away, the >

Re: [Nouveau] CRTC video scanout position for high precision vblank timestamping?

2010-12-29 Thread Ben Skeggs
Sorry for the top-posting. Also confirmed on fermi (nvc0). Ben. Sent from my iPhone On 29/12/2010, at 22:26, Francisco Jerez wrote: > Mario Kleiner writes: > >> On 12/29/2010 10:44 AM, Ben Skeggs wrote: >>> On Wed, 2010-12-29 at 10:37 +0100, Maarten Maathuis wrote: &g

Re: [Nouveau] [PATCH] change temp sensor defaults to neutral ones

2011-01-07 Thread Ben Skeggs
On Thu, 2011-01-06 at 20:54 +0100, Lucas Stach wrote: > The attached patch deletes some defaults which are not really defaults > and changes the standard values to calculation neutral ones. > > I discussed the matter with mupuf on irc and we agreed on this solution. Thanks,

Re: [Nouveau] [PATCH] fix hwmon device binding

2011-01-07 Thread Ben Skeggs
On Thu, 2011-01-06 at 20:54 +0100, Lucas Stach wrote: > The following patch fixes the creation directory of the hwmon files, to > get our directory hierarchy in line with all other hwmon drivers. Thanks, pushed. Ben. > > --lynxeye > __

Re: [Nouveau] [PATCH] drm/nouveau: correctly pair hwmon_init and hwmon_fini

2011-01-30 Thread Ben Skeggs
On Sun, 2011-01-30 at 11:06 +0100, Lucas Stach wrote: > Hi Ben, > Hey Lucas, Thanks, pushed :) Ben. > please push. This fixes > https://bugs.freedesktop.org/show_bug.cgi?id=33434 where I messed up. > > -- Lucas ___ Nouveau ma

Re: [Nouveau] [PATCH 1/2] drm/nouveau: decode PFIFO DMA_PUSHER error codes

2011-02-13 Thread Ben Skeggs
On Sun, 2011-02-13 at 20:46 +0100, Marcin Slusarz wrote: > Signed-off-by: Marcin Slusarz Thanks, pushed. Ben. > --- > drivers/gpu/drm/nouveau/nv04_fifo.c | 17 ++--- > 1 files changed, 14 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/nouvea

Re: [Nouveau] [PATCH 2/2] drm/nv50: fix typos in CCACHE error reporting

2011-02-13 Thread Ben Skeggs
On Sun, 2011-02-13 at 20:46 +0100, Marcin Slusarz wrote: > The code was supposed to print registers around 0x405018 (which is read > earlier), not 0x405818. Thanks, pushed. Ben. > > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nv50_graph.c |8

Re: [Nouveau] [PATCH] fix drmSetMaster error reporting

2011-02-13 Thread Ben Skeggs
On Sun, 2011-02-13 at 20:46 +0100, Marcin Slusarz wrote: > On error, drmSetMaster returns -1 and sets errno. Pushed. > --- > src/nv_driver.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/nv_driver.c b/src/nv_driver.c > index 3e009fc..754739c 100644 > --- a/sr

Re: [Nouveau] [PATCH] drm/nouveau: Fix pageflip event

2011-02-16 Thread Ben Skeggs
On Wed, Feb 16, 2011 at 7:04 PM, Benjamin Franzke wrote: > Assign correct event when initializing nouveau_page_flip_state. Ah yes, I though that was an oddity when I was getting nv50/nvc0 page flipping done. Pushed to nouveau git, Thanks! > --- >  drivers/gpu/drm/nouveau/nouveau_display.c |    2

Re: [Nouveau] [PATCH 1/3] Update configure.ac with autoupdate

2011-02-16 Thread Ben Skeggs
On Thu, 2011-02-17 at 03:47 +0100, Francesco Marella wrote: > Signed-off-by: Francesco Marella Thank you, pushed all 3 patches. Ben. > --- > configure.ac | 15 --- > 1 files changed, 8 insertions(+), 7 deletions(-) > > diff --git a/configure.ac b/configure.

Re: [Nouveau] CCACHE and VFETCH FAULTs causing lockups

2011-03-01 Thread Ben Skeggs
> Any ideas? Not really. If this commit *is* the cause, the problem is still somewhere else. That commit just makes sure PTEs are marked invalid, so if it's causing your faults, then previously the GPU would still have been reading

Re: [Nouveau] CCACHE and VFETCH FAULTs causing lockups

2011-03-07 Thread Ben Skeggs
On Mon, 2011-03-07 at 21:51 +, Maarten Maathuis wrote: > On Sun, Mar 6, 2011 at 2:24 PM, Ben Skeggs wrote: > > > > > > Sent from my iPhone > > > > On 07/03/2011, at 0:03, Maarten Maathuis wrote: > > > >> On Sun, Mar 6, 2011 at 1:44 PM, Ben Skeg

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-07 Thread Ben Skeggs
IBLE); > >> if (lazy && time_after_eq(jiffies, sleep_time)) > >> - schedule_timeout(1); > >> + schedule_hrtimeout(&t, HRTIMER_MODE_REL); > >> > >> if (intr && signal_pending(curre

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-07 Thread Ben Skeggs
On Tue, 2011-03-08 at 00:27 +0100, Marcin Slusarz wrote: > On Tue, Mar 08, 2011 at 12:22:56AM +0100, Marcin Slusarz wrote: > > On Tue, Mar 08, 2011 at 08:24:26AM +1000, Ben Skeggs wrote: > > > On Mon, 2011-03-07 at 18:18 +, Maarten Maathuis wrote: > > > > On Fri,

Re: [Nouveau] [PATCH 1/1] nv30: Fix parsing of perf table

2011-03-18 Thread Ben Skeggs
On Fri, 2011-03-18 at 00:46 +, Emil Velikov wrote: > Perf tables v 1.2 and 1.3 (seen on Geforce FX / 5) are not long enough to > store > the voltage label/id > > Signed-off-by: Emil Velikov > --- > drivers/gpu/drm/nouveau/nouveau_perf.c |6 +- > 1 files changed, 5 insertions(+), 1 d

Re: [Nouveau] [PATCH 1/1] nv50: improve nv50_pm_clock_get()

2011-03-18 Thread Ben Skeggs
ers actually run at double the core clock if the PLL is "off"? > + } else if (id == PLL_MEMORY) { > + NV_INFO(dev, "Memory PLL appears to be OFF\n"); > + return 100*1000; This, is

Re: [Nouveau] [PATCH 1/1] nv50: improve nv50_pm_clock_get ()

2011-03-18 Thread Ben Skeggs
On Fri, 2011-03-18 at 19:57 +, Emil Velikov wrote: > On Fri, 2011-03-18 at 22:04 +1000, Ben Skeggs wrote: > > On Fri, 2011-03-18 at 01:15 +, Emil Velikov wrote: > > > On some cards the memory and/or shader pll can be switched off/disabled > > > Check and retur

Re: [Nouveau] [PATCH 1/1] nv30: Fix parsing of perf table

2011-03-18 Thread Ben Skeggs
On Fri, 2011-03-18 at 20:19 +, Emil Velikov wrote: > Perf tables v 1.2 and 1.3 (seen on Geforce FX/ 5) are not long enough to > store the voltage label/id > > v2 - Remove comment from the code Thank you :) Pushed, with the log message line split to fit into 80 chars. Ben. >

Re: [Nouveau] [PATCH] Associate memtimings with performance levels on cards <= nv98

2011-04-12 Thread Ben Skeggs
On Tue, 2011-04-12 at 01:18 +0200, Martin Peres wrote: > Signed-off-by: Martin Peres Hey, Can you please fix the commit title to be "drm/nouveau: " > --- > drivers/gpu/drm/nouveau/nouveau_drv.h | 26 ++ > drivers/gpu/drm/nouveau/nouveau_mem.c |3 ++- > drivers/gp

Re: [Nouveau] [PATCH] drm/nouveau: Associate memtimings with performance levels on cards <= nv98

2011-04-13 Thread Ben Skeggs
commit name with "drm/nouveau: ". Thank you. Did you see the rest of the comments? Ben. > ___ > Nouveau mailing list > Nouveau@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/nouveau __

Re: [Nouveau] [PATCH] drm/nvc0/pm: read core/shader/memory clocks (more) correctly

2011-04-17 Thread Ben Skeggs
ent(2)) & 0xfff) * 1000; > + perflvl->core = (ROM16(subent(3)) & 0xfff) * 1000; > + perflvl->shader = (ROM16(subent(3)) & 0xfff) / 2 * 1000; > + perflvl->memory = (ROM16(subent(5)) & 0xfff) * 1000;

Re: [Nouveau] [PATCH] drm/nvc0/pm: read core/shader/memory clocks (more) correctly

2011-04-17 Thread Ben Skeggs
On Mon, 2011-04-18 at 00:54 +0200, Martin Peres wrote: > Le 18/04/2011 00:32, Ben Skeggs a écrit : > > On Sun, 2011-04-17 at 17:10 +0200, Martin Peres wrote: > >> --- > >> drivers/gpu/drm/nouveau/nouveau_perf.c |6 +++--- > >> 1 files changed, 3 insertio

Re: [Nouveau] [PATCH] drm/nouveau: Fix a complier warning in the card init error path

2011-04-17 Thread Ben Skeggs
of engine > > initialization routines failed and this change will break it... > > > > Marcin > > Really? It wasn't intentional. I didn't see the e referenced above. It's used above, but I don't think it's actually an error. The

Re: [Nouveau] [PATCH] drm/nouveau: Free nv04 instmem ramin heap at card takedown

2011-04-17 Thread Ben Skeggs
On Sun, 2011-04-17 at 16:15 -0400, Jimmy Rentz wrote: > Add a missing nv04 instmem ramin heap shutdown call. Thank you, pushed this and the vga_ram patch. Ben. > > Signed-off-by: Jimmy Rentz > > diff --git a/drivers/gpu/drm/nouveau/nv04_instmem.c > b/drivers/gpu/drm/nouv

Re: [Nouveau] [PATCH] drm/nouveau: Rename bit_table()

2011-04-18 Thread Ben Skeggs
Sorry for the top post, sent from in bed, on a mobile phone. Personally, I prefer how it is now. If people want it changed, I guess that's ok, but, bit_table_find() instead maybe? Ben. Sent from my iPhone On 18/04/2011, at 11:29, Emil Velikov wrote: > Rename bit_table() to find_b

Re: [Nouveau] Nvidia GT450M

2011-04-18 Thread Ben Skeggs
y to switch to nvidia card > I get a black screen. I supose, can be caused by this error... This error wouldn't get you a black screen. However, I'd say you've probably got an Optimus laptop, which means you're out of luck. Only the intel GPU is physically w

Re: [Nouveau] Nvidia GT450M

2011-04-19 Thread Ben Skeggs
On Tue, 2011-04-19 at 10:19 +0200, Marc Rios wrote: > Hi Ben, > > I know I have optimus, and we are trying a module to be able to switch > between screens, you can check it here: > git://github.com/awilliam/asus-switcheroo.git > > I will try to get better debug and see why

Re: [Nouveau] 2.6.39-rc1 nouveau(?) regression (bisected)

2011-04-21 Thread Ben Skeggs
ing request at > > > bc70b058 > > > > Here it is less obvious, but it was _probably_ a regular kernel > > pointer of the type 0x8801bc70b058 before the high bits were > > overwritten by a 0xffff. >

Re: [Nouveau] [PATCH] drm/nouveau: printk info about disabled modesetting

2011-05-01 Thread Ben Skeggs
his is useful. I don't find it hard when looking at people's logs to scroll to the top and look at the kernel commandline. Really, that should be done *anyway* as there are many more options that are relevant to what nouveau does/does

Re: [Nouveau] [PATCH] drm/nouveau: make cursor_set implementation consistent with other drivers

2011-05-01 Thread Ben Skeggs
On Sun, 2011-05-01 at 23:49 +0200, Marcin Slusarz wrote: > When xorg state tracker wants to hide the cursor it calls set_cursor > with NULL buffer_handle and size=0x0, but nouveau refuses to hide it > because size is not 64x64... which is a bit odd. Both radeon and intel > check buffer_handle befor

Re: [Nouveau] [PATCH] drm/nouveau: release vga_ram allocation before tearing down mm's

2011-05-08 Thread Ben Skeggs
On Sat, 2011-05-07 at 18:03 +0200, Daniel Vetter wrote: > Otherwise we have a use-after free. > > Tested-and-Reported-by: Bruno Prémont > Signed-off-by: Daniel Vetter Ah, we actually have a patch in the nouveau git tree fixing this already. I'll get this upstream ASAP. Ben.

[Nouveau] [TEST REQUEST] fermi (GeForce 4xx/5xx) ucode

2011-05-24 Thread Ben Skeggs
pare stablity/bugs/etc, at least initially. There's likely going to be a number of issues to sort out, so please let me know if you spot any! Thanks! Ben. ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] Nouveau Fermi Firmware

2011-06-02 Thread Ben Skeggs
Feb 12 2011 ? Is this correct? All three other firmware files are > written as binary files... I know a bit of python and bash but am > drawing a blank on this. Have you had a look at the trace itself and made sure there was more than a couple of writes to 0x41a1c4? The NVIDIA fuc blobs usua

Re: [Nouveau] [PATCH] drm/nouveau/pm: Prevent overflow in nouveau_perf_init()

2011-06-15 Thread Ben Skeggs
causing aditional problems. > > Add a simple check in order to prevent the case I've pushed this. I'm not entirely certain we shouldn't just bail out completely if this is the case, I suspect that if there's this many, the VBIOS image is probably very screwed. This'll do

Re: [Nouveau] [PATCH] drm/nouveau/pm: Prevent overflow in nouveau_perf_init()

2011-06-16 Thread Ben Skeggs
On Thu, 2011-06-16 at 23:40 +0100, Emil Velikov wrote: > On Thu, 16 Jun 2011 04:43:59 +0100, Ben Skeggs wrote: > > > On Sat, 2011-06-11 at 13:30 +0100, Emil Velikov wrote: > >> While parsing the perf table, there is no check if > >> the num of entries read from t

Re: [Nouveau] gpu lockup detection and fallback to noaccel

2011-06-19 Thread Ben Skeggs
t that important > as I thought. Hey, I'd be interested in seeing the approach you've taken at least. I'm not convinced this is something we want exactly, my fear is that a lot of bugs will end up covered over with people not noticing. But, lets see :) Ben. > > Marcin

Re: [Nouveau] [PATCH] drm/nouveau: set ptimer to count in ns on all chipset at the exception of nv40

2011-07-03 Thread Ben Skeggs
Apologies for top-posting. Martin, As per our convo on irc earlier, pushed code achieving this functionality. Let me know if there's issues :) Ben. Sent from my iPhone On 01/07/2011, at 10:34, Martin Peres wrote: > From: Martin Peres > > May cause problems with your laptop

Re: [Nouveau] [PATCH] drm/nouveau: restore subchannel<->object binding on resume

2011-07-17 Thread Ben Skeggs
the cause I believe. I've pushed a fix to git. Ben. > > Reported-by: Andrew Randrianasulu > Tested-by: Andrew Randrianasulu > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nouveau_dma.c |3 +-- > drivers/gpu/drm/nouveau/nouveau_dma.h |

Re: [Nouveau] [PATCH 2/2] g3dvl: disable iDCT for now

2011-07-26 Thread Ben Skeggs
On Wed, 2011-07-20 at 13:48 -0500, Bryan Cain wrote: > --- > src/vl_hwmc.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) The obvious question I have for this is, why? Ben. > > diff --git a/src/vl_hwmc.c b/src/vl_hwmc.c > index 6e9e0c7..8e24786 100644 >

Re: [Nouveau] [PATCH 2/2] g3dvl: disable iDCT for now

2011-07-27 Thread Ben Skeggs
On Wed, 2011-07-27 at 14:08 -0500, Bryan Cain wrote: > On 07/27/2011 02:42 AM, Ben Skeggs wrote: > > On Wed, 2011-07-20 at 13:48 -0500, Bryan Cain wrote: > >> --- > >> src/vl_hwmc.c |4 ++-- > >> 1 files changed, 2 insertions(+), 2 deletions(-) > > Th

Re: [Nouveau] [PATCH] drm/nv41/pm: Write to correct registers on nv41_fan_set()

2011-08-04 Thread Ben Skeggs
fix into the original commit. Ben. > > diff --git a/drivers/gpu/drm/nouveau/nv40_pm.c > b/drivers/gpu/drm/nouveau/nv40_pm.c > index c5ad31c..aa9fded 100644 > --- a/drivers/gpu/drm/nouveau/nv40_pm.c > +++ b/drivers/gpu/drm/nouveau/nv40_pm.c > @@ -380,7 +380,7 @@ nv41_pm_fanspee

[Nouveau] NVIDIA (including Optimus) laptop owners - please read!

2011-10-06 Thread Ben Skeggs
;nouveau.vbios=ACPI" in your kernel options, and then: mount -t debugfs debugfs /sys/kernel/debug cp /sys/kernel/debug/dri/0/vbios.rom ~/ Any questions, feel free to respond to this email or jump into #nouveau on freenode and ping me (darktama). Thanks in advance! Ben. [1] http://nou

Re: [Nouveau] NVIDIA (including Optimus) laptop owners - please read!

2011-10-08 Thread Ben Skeggs
ivers installed. > Here below some infos: Hmm, from the below info it looks very likely there's *no* outputs wired up to the NVIDIA GPU on your system. But, can you attach your dmesg output so I can take a look. Thank you, Ben. > > # lspci -vnnn|grep VGA > 00:02.0 VGA com

Re: [Nouveau] [PATCH] drm/nouveau: initialize chan->fence.lock before use

2011-10-09 Thread Ben Skeggs
ses > > fence lock. Apologies, I missed this the first time around. I've queued it up and will push today. Thanks, Ben. > > > > BUG: spinlock bad magic on CPU#0, test/24134 > > lock: 88019f90dba8, .magic: , .owner: /-1, .owner_cpu: 0 > > Pid: 24134,

Re: [Nouveau] [PATCH] Import EVO commands into envytools

2011-10-09 Thread Ben Skeggs
On Mon, 2011-10-10 at 00:40 +0200, Maxim Levitsky wrote: > --- > rnndb/nv_evo.xml | 311 > ++ > 1 files changed, 311 insertions(+), 0 deletions(-) > create mode 100644 rnndb/nv_evo.xml Nice work, comments inlined where appropriate below. > >

Re: [Nouveau] [PATCH 01/10]: nouveau: assorted fixes

2011-10-09 Thread Ben Skeggs
et you the same functionality. Thanks! Ben. > > Here is my patch queue I accumulated over quite a long time. > Patches 1-6 are bugfixes, and rest is mostly RFC. > > Comments are welcome. > > Best regards, > Maxim Levitsky > _

Re: [Nouveau] NVIDIA (including Optimus) laptop owners - please read!

2011-10-10 Thread Ben Skeggs
On Mon, 2011-10-10 at 07:55 +0200, Gianluca Cecchi wrote: > On Sun, Oct 9, 2011 at 5:38 AM, Ben Skeggs wrote: > > Hmm, from the below info it looks very likely there's *no* outputs wired > > up to the NVIDIA GPU on your system. But, can you attach your dmesg > > o

Re: [Nouveau] NVIDIA (including Optimus) laptop owners - please read!

2011-10-10 Thread Ben Skeggs
On Mon, 2011-10-10 at 02:09 -0400, Eric Appleman wrote: > Ben, Hey, > most Optimus laptops have no physical output for the Nvidia GPU. > Oddballs like the Dell XPS line have the HDMI ports connected to the GPU > and Asus laptops have the MXDS/MXMX mux for the LVDS. Yep, I

Re: [Nouveau] Problems configuring an NVIDIA GeForce GT 520 with a 1366x768 screen

2011-10-10 Thread Ben Skeggs
me as previous Fermi chips, huge chunks of it were redesigned. Ben. [1] http://nouveau.freedesktop.org/wiki/InstallDRM#out-of-treebuild.28alternative.29 > - Carlos Solís > ___ > Nouveau mailing list > Nouveau@lists.freedeskt

Re: [Nouveau] geforce 9800m gt HDMI

2011-10-11 Thread Ben Skeggs
ot expected to work at all with current nouveau, it's being worked on as I get a chance. Ben. > > > > -- > Michele Cane > > ___ > Nouveau mailing list > Nouveau@lists.freedesktop.o

Re: [Nouveau] "enable ctxprog xfer only when we need it to save power" introduces big performance regression

2011-10-29 Thread Ben Skeggs
___ > > Nouveau mailing list > > Nouveau@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/nouveau > > > > Before this patch, what ensured that context switching was enabled? It was enabled by default during graph init, this was removed with th

Re: [Nouveau] Option "Rotate" is not used

2011-11-08 Thread Ben Skeggs
ange its orientation: > > Option "Rotate" "CCW" > > Can this be done with nouveau? BTW, I can't say for sure that nouveau > isn't rotating the screen image because the workstation is remote. Sure it can, nouveau is a randr 1.2 driver - so adding 'Optio

Re: [Nouveau] Bisect: i2c regression in 63daa9589cb88617d9cc8a0b0282564682a975b7

2012-01-02 Thread Ben Skeggs
delays. I doubled T_RISEFALL and T_HOLD, and the > adt7473 chip could be detected & used again. I haven't tested other > combination, nor tried to link those delays with any datasheet. Apologies for the breakage, and thanks for looking into the cause! Could you identify whether both

Re: [Nouveau] properly check for _DSM function support and Optimus support for vga_switcheroo

2012-01-09 Thread Ben Skeggs
On Mon, 2012-01-09 at 00:13 +0100, Lekensteyn wrote: > Any activity on this? The newly attached patch contains one difference > with the previous > one: a dummy function has been added to avoid an error if ACPI is > disabled. Dave, Any thoughts on this? Ben. > > Regards, &g

Re: [Nouveau] [PATCH RFC] drm/nouveau/dp: fix bad comparison in dp_link_train_commit()

2012-02-05 Thread Ben Skeggs
On Fri, 2012-02-03 at 11:13 -0500, Xi Wang wrote: > The comparison (lpre == DP_TRAIN_PRE_EMPHASIS_9_5) is always false: > lpre is initialized as (lane & 0x0c) >> 2, which is at most 3, while > DP_TRAIN_PRE_EMPHASIS_9_5 is defined as (3 << 3). I've picked up the patch.

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

2012-02-15 Thread Ben Skeggs
matting issues such as "if(blah)" instead of "if (blah)" in a few places. Ben. > > Kudos to Mario for his many helpful comments and testing. > > Signed-off-by: Lucas Stach > Reviewed-by: Mario Kleiner > Tested-by: Mario Kleiner > --- > drivers/gpu/drm/n

Re: [Nouveau] [Patches][nouveau/ddx]: Improvements to bufferswap implementation and timestamping

2012-02-28 Thread Ben Skeggs
ebase the ported-to-new-libdrm ddx on top of it all and get that ready. Ben. > > here a set of patches against the nouveau-ddx. This is an extended and > revised set, based on Francisco Jerez feedback from autumn last year. > > [1/9] Makes pageflipping work again on X-Server 1

Re: [Nouveau] [PATCH] drm/nv50/vm: Prevent kernel freeze

2012-02-29 Thread Ben Skeggs
On Mon, 2012-02-27 at 21:01 +, Emil Velikov wrote: > Commit 'drm/nv50-nvc0/vm: support unsnooped system memory' caused > a regression on certain nv50 IGP due to not passing the correct > target type to nv50_vm_addr() Thanks for this, I've pushed it to the nouveau tree.

Re: [Nouveau] Nouveau: NV43 unknown ROM tables? + division by zero

2012-04-03 Thread Ben Skeggs
ttached at the moment, used somewhere else. > > Looks better, indeed. > > Could you send us your vbios please ? Better yet, add "nouveau.vbios=PRAMIN" to your kernel boot options. Ben. > > Thanks by advance, > > Martin > __

Re: [Nouveau] [PATCH 2/5] drm/nouveau: base fence timeout on time of emission

2012-04-23 Thread Ben Skeggs
compute operations) in the future. Ben. > > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nouveau_fence.c |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c > b/drivers/gpu/drm/nouv

Re: [Nouveau] [WIP PATCH] dri/nouveau: Add S3TC support for nv20.

2012-04-24 Thread Ben Skeggs
can give me some feedback on the style etc., please do. > 4) S3TC texture seem to be supported also on nv10, I have nv11 somewhere so I > might be able to add support to it too, Curro, are you able to take a look over this and give Viktor some feedback? You're probably best acquainted w

Re: [Nouveau] [RFC PATCH 5/5] drm/nouveau: gpu lockup recovery

2012-04-24 Thread Ben Skeggs
ngine that's involved.. It's going to be useless. The problem could be anything. > > Yes, when this patchset will be applied, I'm going to work on improving > error reporting. Assuming you're not talking about a register-dump style lockup report like above, this co

Re: [Nouveau] rendering problems with latest ddx driver (nv50)

2012-04-24 Thread Ben Skeggs
oked at this, so it may just be a question of precision. Ben. > ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH v2 4/4] drm/nouveau: gpu lockup recovery

2012-04-26 Thread Ben Skeggs
I am, however, considering pulling the vm flush timeout error propagation and break-out-of-waits-on-signals that builds on it. As we really do need to become better at having killable processes if things go wrong :) Ben. > > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nou

Re: [Nouveau] [PATCH] nouveau: fix channel closing

2012-05-02 Thread Ben Skeggs
On Tue, 2012-05-01 at 23:24 +0200, Marcin Slusarz wrote: > Restore code lost in libdrm_nouveau rewrite. Thanks, pushed. > --- > nouveau/nouveau.c | 17 - > 1 files changed, 12 insertions(+), 5 deletions(-) > > diff --git a/nouveau/nouveau.c b/nouveau/nouveau.c > index f0bc2c3..

Re: [Nouveau] [PATCH v2 4/4] drm/nouveau: gpu lockup recovery

2012-05-02 Thread Ben Skeggs
On Sat, 2012-04-28 at 16:49 +0200, Marcin Slusarz wrote: > On Thu, Apr 26, 2012 at 05:32:29PM +1000, Ben Skeggs wrote: > > On Wed, 2012-04-25 at 23:20 +0200, Marcin Slusarz wrote: > > > Overall idea: > > > Detect lockups by watching for timeouts (vm flush / fence), retu

Re: [Nouveau] [PATCH v2 4/4] drm/nouveau: gpu lockup recovery

2012-05-02 Thread Ben Skeggs
On Wed, 2012-05-02 at 15:33 +0200, Martin Peres wrote: > On 02/05/2012 13:28, Ben Skeggs wrote: > > Right, again, I don't disagree :) I think we can improve a lot on the > > big-hammer-suspend-the-gpu solution though, and instead reset only the > > faulting engine

Re: [Nouveau] NV43: Native resolution not available on Dell 2007FP

2012-05-04 Thread Ben Skeggs
On Saturday, 5 May 2012, Mark Hills wrote: > On Sat, 5 May 2012, Maarten Maathuis wrote: > > > On Fri, May 4, 2012 at 5:55 PM, Mark Hills wrote: > > > I have a Dell 2007FP monitor. NV43 (GeForce 6600) can not use the > native > > > resolution. > > > > > > 1600x1200 is listed under "DDC gathered M

Re: [Nouveau] [kernel PATCH v2] drm/nv50: decode PGRAPH DATA_ERROR = 0x24

2012-05-08 Thread Ben Skeggs
On Sun, 2012-05-06 at 23:08 +0200, Marcin Slusarz wrote: > Signed-off-by: Marcin Slusarz Pushed, thanks! > --- > v2: sync name to envytools > --- > drivers/gpu/drm/nouveau/nv50_graph.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nv50_gra

Re: [Nouveau] [PATCH 1/4] nv30: Fix generic passing to fragment program in NV34

2012-05-11 Thread Ben Skeggs
Am Samstag, den 12.05.2012, 03:42 +0200 schrieb Roy Spliet: > fixes over 800 piglit cases in quick-driver suite for NV34 > > Signed-off-by: Roy Spliet > --- > src/gallium/drivers/nv30/nv30_fragprog.c |2 +- > src/gallium/drivers/nv30/nv30_state.h|2 +- > src/gallium/drivers/nv30/nvfx

Re: [Nouveau] [Patches] mesa/nv30: Diverse set of patches that improve NV3x render quality V2

2012-05-11 Thread Ben Skeggs
t; [2/4]: Fixes shader compiler assertion errors, as some source registers do > not exist for certain operations. Fixes several piglit tests when mesa is > compiled with --enable-debug > [3/4]: unchanged 2/3 pushed to mesa git! Thanks again! Ben. > [4/4]: Dropped, requires a more subs

Re: [Nouveau] nouveau_subdev & misc patches

2012-05-20 Thread Ben Skeggs
overflow. Hey, Thanks for these! I've pulled most of them directly for the moment, comments on specific patches will follow. Cheers, Ben. > > Please review and comment > > --- > drivers/gpu/drm/nouveau/nouveau_acpi.c |3 ++- > drivers/gp

Re: [Nouveau] [PATCH 03/14] drm/nouveau/device: Simplify init, fini fuction

2012-05-20 Thread Ben Skeggs
On Mon, May 21, 2012 at 12:14:52AM +0100, Emil Velikov wrote: > Removes unnecessary goto statement I dropped these changes from this patch for the moment, I might want them yet. > fini subdevices before destroying them I took this hunk, and reworded the commit message to match. > > Signed-off-by

Re: [Nouveau] [PATCH 14/14] drm/nouveau/therm: Rework nouveau_therm_create()

2012-05-20 Thread Ben Skeggs
On Mon, May 21, 2012 at 12:15:03AM +0100, Emil Velikov wrote: > It contains a few changes mainly targeting the following > * Therm table is present in BIT vbios > * Parse the vbios table before hooking temp_get(), as it contains the therm > sensor calibration data > * Add dummy_therm_temp_get()

Re: [Nouveau] [PATCH 14/14] drm/nouveau/therm: Rework nouveau_therm_create()

2012-05-21 Thread Ben Skeggs
On Mon, May 21, 2012 at 07:13:54AM +0100, Emil Velikov wrote: > On Mon, 21 May 2012 07:30:32 +0100, Ben Skeggs wrote: > > >On Mon, May 21, 2012 at 12:15:03AM +0100, Emil Velikov wrote: > >>It contains a few changes mainly targeting the following > >> * Therm

Re: [Nouveau] [PATCH] drm/nouveau/nvd9: Fix GPIO initialisation sequence.

2012-05-21 Thread Ben Skeggs
On Mon, May 21, 2012 at 02:51:50PM +0200, Marcin Kościelnicki wrote: > Signed-off-by: Marcin Kościelnicki Applied, thanks! > --- > drivers/gpu/drm/nouveau/nouveau_gpio.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/nouveau/nouveau_gpio.c > b/drive

Re: [Nouveau] [PATCH] drm/nouveau/pm: fix oops on get/set_perflvl

2012-05-29 Thread Ben Skeggs
On Sun, May 27, 2012 at 10:00:41PM +0200, Marcin Slusarz wrote: > Regression from "drm/nouveau: very scary looking cleanup commit" Thanks, pushed. > > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nouveau_pm.c |6 -- > 1 files changed, 4 insertions(+), 2 deletions(-) >

Re: [Nouveau] still seeking way to recover card after cursor disappears or card locks up

2012-05-30 Thread Ben Skeggs
very much for that info/advice. You were > 100% right that rmmod refused to remove the nouveau module. > > Is there any other way to force a reset of a graphics card or > driver module other than OS shutdown/reboot? # echo 0 > /sys/class/vtconsole/vtcon1/bind; modprobe -r nouv

Re: [Nouveau] [PATCH] nv04/exa: Reset destination surface offset in the same call of NV04EXACopy

2012-06-08 Thread Ben Skeggs
On Thu, Jun 07, 2012 at 02:51:00PM +0200, Marcin Slusarz wrote: > On Sat, May 26, 2012 at 10:15:20PM +0200, Viktor Novotný wrote: > > Fixes FDO bug #48954. > > > > Signed-off-by: Viktor Novotný > > --- > > Reviewed-by: Marcin Slusarz Thanks a lot, nice catch. I've pushed the fix to nouveau git

Re: [Nouveau] [PATCH] drm/nv84: decode PCRYPT errors

2012-06-13 Thread Ben Skeggs
On Mon, Jun 11, 2012 at 12:21:12AM +0200, Marcin Slusarz wrote: > Pushed, thanks. > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nv84_crypt.c | 19 --- > 1 files changed, 16 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nv84_crypt.c

Re: [Nouveau] [PATCH] drm/nv50: rename INVALID_QUERY_OR_TEXTURE error to INVALID_OPERATION

2012-06-14 Thread Ben Skeggs
On Thu, Jun 14, 2012 at 11:58:08PM +0200, Marcin Slusarz wrote: > Current name is misleading, because this error can be triggered by other > conditions, like changing STRMOUT parameter without disabling STRMOUT first. > Applied, thanks. > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/no

Re: [Nouveau] Regression: Blank screen with kernel 3.4

2012-06-15 Thread Ben Skeggs
ee [1]), I won't attach anything here. But I'm happy to > provide anything you need, of course. Just let me know. Thanks! I'm aware of the issue and will look into it after the weekend. I don't have any effected hw, so i'll ping you for testing as needed :) Ben. > >

Re: [Nouveau] [REGRESSION] nouveau: Memory corruption using nva3 engine for 0xaf

2012-07-04 Thread Ben Skeggs
On Thu, Jul 05, 2012 at 08:31:13AM +0200, Henrik Rydberg wrote: > Hi Ben, Dave, Hey Henrik, > > Since 3.5-rc0, I have been experiencing occasional screen corruption > on my MacBookAir3,1, using a GeForce 320M (nv50, 0xaf). The X driver > version is xf86-video-nouvea-1.0.1-1 (arch)

Re: [Nouveau] [PATCH] drm/nvd0/disp: mask off high 16 bit of negative cursor x-coordinate

2012-07-26 Thread Ben Skeggs
On Fri, Jul 27, 2012 at 6:04 AM, Maarten Lankhorst wrote: > Op 26-07-12 20:53, Christoph Bumiller schreef: >> --- >> drivers/gpu/drm/nouveau/nvd0_display.c |2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/gpu/drm/nouveau/nvd0_display.c >> b/drivers/gpu/dr

[Nouveau] BIG changes to kernel module

2012-08-04 Thread Ben Skeggs
m off to relax a bit (read: hide) before the storm of bug reports :) Cheers, Ben. [1] http://cgit.freedesktop.org/nouveau/linux-2.6 ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH] nvc0: Add and enable vblank support

2012-08-04 Thread Ben Skeggs
is a sub-status of what you were using. Let me know if there's any issues. Ben. > > The changes are that I enable vblank by default, and offset > takes OFFSET_HIGH/LOW instead of something relative to notifier_bo. > > Signed-off-by: Maarten Lankhorst > > --- > d

Re: [Nouveau] [PATCH] nouveau: Do not use nva3 engine for 0xaf chipset

2012-08-04 Thread Ben Skeggs
which kills the symptoms. I've pushed this (with slightly modified commit message) to nouveau git. I'll get it to Linus' tree in a future -fixes merge. Thanks, Ben. > > Signed-off-by: Henrik Rydberg > --- > Hi Ben, > > this patch is still needed in 3.6-rc1,

Re: [Nouveau] BIG changes to kernel module

2012-08-06 Thread Ben Skeggs
On Sun, Aug 05, 2012 at 10:56:37PM +0200, Marcin Slusarz wrote: > On Sun, Aug 05, 2012 at 12:40:15PM +1000, Ben Skeggs wrote: > > Hey all, > > > > I've just pushed a major rework/rewrite of the Nouveau kernel module to > > git[1]. > > > > Details of

Re: [Nouveau] [bisected] nouveau: "Failed to idle channel x" after resume

2012-08-08 Thread Ben Skeggs
only to stop again. Starting Firefox after resuming > > > > > triggers the bugs every time, and bisecting leads to 03bd6efa > > > > > ("drm/nv50/fifo: use hardware channel kickoff functionality"). > > > > > > > > Hi Ben, > > >

Re: [Nouveau] [bisected] nouveau: "Failed to idle channel x" after resume

2012-08-08 Thread Ben Skeggs
On Wed, Aug 08, 2012 at 08:00:21AM +0200, Sven Joachim wrote: > On 2012-08-08 07:37 +0200, Ben Skeggs wrote: > > > On Mon, Aug 06, 2012 at 11:38:04PM +0300, Maxim Levitsky wrote: > >> On Sat, 2012-08-04 at 17:41 +0300, Maxim Levitsky wrote: > >> > On Mon, 2012-07

Re: [Nouveau] nouveau Macbook EDID fix upstreaming...

2012-08-19 Thread Ben Skeggs
d to eg 3.6-rc3? Not at this stage, no. I don't believe this is likely to be the correct fix. I should have access to one of the new mbp systems in the near future and I'll investigate it further then. Cheers, Ben. > > Many thanks, > Daniel > > --- [1] > >

Re: [Nouveau] [PATCH] drm/nouveau: restore hw accelerated buffer copies

2012-08-19 Thread Ben Skeggs
On Fri, Aug 17, 2012 at 11:47:44PM +0200, Marcin Slusarz wrote: > Regression from "drm/nve0: use async copy engine for ttm buffer moves > if available". Oops, my bad. Applied, thanks! > > Signed-off-by: Marcin Slusarz > --- > drivers/gpu/drm/nouveau/nouveau_bo.c |4 ++-- > 1 files changed,

Re: [Nouveau] [PATCH 01/10] drm/nv50: decode PGRAPH status registers on TLB flush fail

2012-08-19 Thread Ben Skeggs
On Sun, Aug 19, 2012 at 10:59:56PM +0200, Marcin Slusarz wrote: > Now it outputs: > nouveau E[ PGRAPH][:02:00.0] PGRAPH TLB flush idle timeout fail > nouveau E[ PGRAPH][:02:00.0] PGRAPH_STATUS: BUSY DISPATCH VFETCH > CCACHE_UNK4 STRMOUT_GSCHED_UNK5 UNK14XX UNK1CXX CLIPID ZCULL ENG2D UNK3

Re: [Nouveau] [PATCH 06/10] drm/nouveau: sparse fixes

2012-08-19 Thread Ben Skeggs
On Sun, Aug 19, 2012 at 11:00:01PM +0200, Marcin Slusarz wrote: > drivers/gpu/drm/nouveau/core/core/object.c:452:22: warning: non-ANSI function > declaration of function 'nouveau_object_debug' > > drivers/gpu/drm/nouveau/core/subdev/device/base.c:64:10: warning: Initializer > entry defined twice

Re: [Nouveau] [PATCH 08/10] drm/nouveau: use ioread32_native/iowrite32_native for fifo control registers

2012-08-19 Thread Ben Skeggs
On Sun, Aug 19, 2012 at 11:00:03PM +0200, Marcin Slusarz wrote: > Signed-off-by: Marcin Slusarz > --- > Do I miss something important why was it written this way? Nope, you didn't. It's a left-over from a *really* early revision of the userspace version of the kernel module. Thanks! > --- > dr

Re: [Nouveau] [PATCH 09/10] drm/nv50/evo: store iomem pointer in properly typed field

2012-08-19 Thread Ben Skeggs
On Sun, Aug 19, 2012 at 11:00:04PM +0200, Marcin Slusarz wrote: > Abuse of function pointer was noticed by sparse. I'm aware of this horror, however I didn't apply the patch. I have work in progress that'll get rid of this nasty stuff before the tree goes upstream, doing it this way was a quick ha

Re: [Nouveau] [PATCH 10/10] drm/nouveau: fix off-by-one bugs related to command submission in IB mode

2012-08-19 Thread Ben Skeggs
On Sun, Aug 19, 2012 at 11:02:00PM +0200, Marcin Slusarz wrote: > Commit "drm/nouveau: port all engines to new engine module format" changed > IB size calculation to be less wasteful, but didn't take into account already > existing off-by-one bugs :). > > So: > - ib_max is the last entry, so we ne

Re: [Nouveau] [PATCH 10/10] drm/nouveau: fix off-by-one bugs related to command submission in IB mode

2012-08-20 Thread Ben Skeggs
Am Dienstag, 21. August 2012 schrieb Marcin Slusarz : > On Mon, Aug 20, 2012 at 04:33:24PM +1000, Ben Skeggs wrote: > > On Sun, Aug 19, 2012 at 11:02:00PM +0200, Marcin Slusarz wrote: > > > Commit "drm/nouveau: port all engines to new engine module format" > chang

Re: [Nouveau] [PATCH] drm/nouveau: add default debug level override

2012-09-12 Thread Ben Skeggs
..fde06b0 100644 > --- a/drivers/gpu/drm/nouveau/core/core/option.c > +++ b/drivers/gpu/drm/nouveau/core/core/option.c > @@ -22,9 +22,16 @@ > * Authors: Ben Skeggs > */ > > +#include > #include > #include > > +MODULE_PARM_DESC(debug_level, "nv_

  1   2   3   4   5   6   7   8   9   10   >