Re: 2024 X.Org Foundation Membership deadline for voting in the election

On Tue, 26 Mar 2024 11:42:48 -0400 Christopher Michael wrote: > The 2024 X.Org Foundation membership renewal period has been extended > one additional week and elections will start the following week on 01 > April 2024. > > Please note that only current members can vote in the upcoming

Re: [Nouveau] [PATCH] x86/mm/kmmio: correctly handle kzalloc return

On Tue, 20 Oct 2020 14:13:44 +0530 Mugilraj Dhavachelvan wrote: > Replacing return value -1 to error code > > Signed-off-by: Mugilraj Dhavachelvan > --- > arch/x86/mm/kmmio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/mm/kmmio.c b/arch/x86/mm/kmmio.c >

Re: [Nouveau] [PATCH 09/35] x86: Convert remaining uses of pr_warning to pr_warn

> Hi, seems fine to me, even though I haven't been involved in the kernel side for years. For the hunks quoted below *only*: Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk> > diff --git a/arch/x86/mm/kmmio.c b/arch/x86/mm/kmmio.c > index afc47f5c9531..ad70518cdcc7

Re: [Nouveau] K2200 Special Setup Question

On Tue, 22 Dec 2015 22:50:14 +0100 ternaryd wrote: > Hi, > > I'm planning to use a K2200 board with an Eizo > monitor at 10 bits color depth together with a > standard monitor on a debian sid system. Hi, I don't know anything about K2200, but I suppose I can give what I

Re: [Nouveau] [PATCH] Add Option DRI3 to allow to disable DRI3 under EXA.

On Sat, 18 Jul 2015 21:09:46 +1000 Dave Airlie airl...@gmail.com wrote: Hm, a total removal would at least make me and my users rather unhappy atm., as without glamor no hw accel at all on = maxwell. modesetting and nouveau are not on par feature-wise atm. E.g., modesetting as of the next

Re: [Nouveau] Logo Nouveau

On Tue, 18 Nov 2014 12:48:37 +0100 Martin Peres martin.pe...@free.fr wrote: With this done, I guess delivering the logo as an svg file along with its licence (have you had time to look at the creative commons?) would be a great step. It would also be nice to have one logo without nouveau

Re: [Nouveau] MmioTrace: Using the Instruction Decoder, etc.

On Fri, 25 Oct 2013 17:19:56 +0400 Eugene Shatokhin euspec...@gmail.com wrote: Hi, 2013/10/25 Pekka Paalanen p...@iki.fi ... We could use some comments from the real reverse-engineers. I used to be mostly a tool writer. Yes, if some experts could share their knowledge

Re: [Nouveau] MmioTrace: Using the Instruction Decoder, etc.

On Sat, 19 Oct 2013 17:12:20 +0400 Eugene Shatokhin euspec...@gmail.com wrote: Hi, Ah, you are not using the ftrace framework nor relayfs? Mmiotrace used to be relayfs at one point and then converted to ftrace. Yes, I considered these when I started working on KernelStrider but finally

Re: [Nouveau] MmioTrace: Using the Instruction Decoder, etc.

be possible in proprietary drivers, but you can look at the instruction decoding code in mmiotrace, which digs up the type and size of access and the value. That has been enough so far. Thanks, pq 2013/10/17 Pekka Paalanen p...@iki.fi On Mon, 14 Oct 2013 22:45:09 +0400 Eugene Shatokhin

Re: [Nouveau] MmioTrace: Using the Instruction Decoder, etc.

On Mon, 14 Oct 2013 22:45:09 +0400 Eugene Shatokhin euspec...@gmail.com wrote: Hi, There is an interesting TODO item on MmioTraceDeveloper page: kprobes has a generic instruction decoding facility, use that instead of homebrewn (or KVM), and use emulation instead of page faulting

Re: [Nouveau] how do I unsubscribe from the Nouveau malling list?

On Sat, 12 Jan 2013 17:50:12 -0600 intok in...@linuxmail.org wrote: don't remember the password and the password reminder on the site doesn't seem to work. http://lists.freedesktop.org/mailman/listinfo/nouveau Unsubcribing does not seem to require a password, just use the Unsubscribe or edit

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

On Wed, 30 May 2012 20:55:49 -0700 rm.ric...@jacob21819.net (Robert M. Riches Jr.) wrote: I very doubt that rmmod and modprobe will help you in any way since nouveau is KMS module and it stays in use the whole time. Something will be broken anyway. But you can try and see what's happening,

Re: [Nouveau] Question about NV18 and GBM library.

-function OpenGL - writing the infrastructure for using EGL/OpenGL on the framebuffer offered by DRM and maybe something else I overlooked. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http

Re: [Nouveau] [PATCH] master-compat: Compile i915 DRM for out-of-tree. Optimus laptops need it.

. Or some other upstream tree. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] better way to test optimus laptop

, and do you have a hw mux? http://nouveau.freedesktop.org/wiki/Optimus -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH] master-compat: Compile i915 DRM for out-of-tree. Optimus laptops need it.

sanity checks in the kernel config system. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [Mesa-dev] RFC: ctx-Driver.Map/UnmapTextureImage() hooks

On Mon, 18 Jul 2011 08:09:17 -0600 Brian Paul bri...@vmware.com wrote: On 07/15/2011 02:59 PM, Pekka Paalanen wrote: On Fri, 15 Jul 2011 12:22:41 -0600 Brian Paulbri...@vmware.com wrote: On 07/15/2011 10:07 AM, Dave Airlie wrote: On Fri, Jul 15, 2011 at 4:10 AM, Brian Paulbrian.e.p

Re: [Nouveau] GeForce 5500 crashes driver

information, I'd say very unlikely. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] NV46 disconnected VGA

instructions in http://nouveau.freedesktop.org/wiki/KernelModeSetting And keep the simplistic xorg.conf for now. HTH, Cheers -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org

Re: [Nouveau] Nouveau_drv problem and KMS nouveau problem... please HELP!

right, and tell exactly what you used. Provide a complete kernel log, without any plymouth or X. Just trying to get the basic fbcon on Nouveau. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http

Re: [Nouveau] [PATCH] drm: Remove DRIVER_DATE and CORE_DATE

between nouveau/linux-2.6 and upstream kernel trees is a burden. Is there a better way to add revision information to an out-of-tree built kernel module? Or maybe this is not useful at all? Cheers. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau

Re: [Nouveau] Display goes off when framebuffer enabled and Kernel Panic with fbset

you provide kernel logs, please, do not filter them. Nouveau possibly interacts with lots of other drivers. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman

Re: [Nouveau] Display goes off when framebuffer enabled and Kernel Panic with fbset

(version ) for libdrm and xf86-video-nouveau. That is what I do with Gentoo. Cheers. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] updating the source tree

On Thu, 07 Oct 2010 20:07:37 +0200 Grzesiek Sójka p...@pfu.pl wrote: On 10/07/10 19:51, Pekka Paalanen wrote: On Thu, 07 Oct 2010 19:34:35 +0200 Grzesiek Sójkap...@pfu.pl wrote: I downloaded the nouveau/linux-2.6 using the following: git clone --depth 1 git://anongit.freedesktop.org

Re: [Nouveau] OT: compilation

, it is CONFIG_LOCALVERSION_AUTO as far as I recall, disable that. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] Trouble with building xf86-video-nouveau

. Nouveau DDX does not yet have a fix. There is already a bug report: https://bugs.freedesktop.org/show_bug.cgi?id=30482 In the mean time, you just need to revert to an older (preferrably released) X server. -- Pekka Paalanen http://www.iki.fi/pq

Re: [Nouveau] GeForce FX5200 dual DVI Samsung 204b

arbiter, if not touching the hardware. Cheers. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] dual head

monitors without X? Or do you want multi-seat X? What is your use case? -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] GeForce FX5200 dual DVI Samsung 204b

EndSection Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] GeForce FX5200 dual DVI Samsung 204b

On Tue, 28 Sep 2010 12:43:12 +0200 Grzesiek Sójka p...@pfu.pl wrote: On 09/27/10 23:41, Pekka Paalanen wrote: Are you saying that forcing a low-refresh-rate mode via video= kernel argument does not help? But you still do get a good picture on the framebuffer console, do you

Re: [Nouveau] GeForce FX5200 dual DVI Samsung 204b

video= kernel argument does not help? But you still do get a good picture on the framebuffer console, do you not? In that case, X is overriding what kernel thinks is the best mode, so you should clean up the X config. If wiping the X config does not help, post kernel and X logs. Cheers. -- Pekka

Re: [Nouveau] [PATCH] x86, mmiotrace: add support for tracing STOS instruction

On Sat, 31 Jul 2010 22:51:01 +0200 Marcin Slusarz marcin.slus...@gmail.com wrote: Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc: Pekka Paalanen p...@iki.fi I have not checked the correctness of this patch, but otherwise: Acked-by: Pekka Paalanen p...@iki.fi --- arch/x86/mm

Re: [Nouveau] 2 cards in, but work only separately

). So is that supposed to be that way currently or I missed to pass some parameters or bug, or..? What more should I post here if anyone interested in this? -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org

Re: [Nouveau] Why do we disable local IRQ around nouveau_fence_update?

-unsafe spinlocks. I'm not sure how it relates to ISR bottom halves, though. Note, that also irq-unsafe spinlocks disable preemption, which might be enough to disturb audio. my 2c -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau

Re: [Nouveau] Kernel module fails to initialize on AMD751 based system with NV34

that a conflicting driver is loading: [3.772842] vga16fb: mapped to 0xc00a [3.773752] fb0: VGA16 VGA frame buffer device vga16fb is specifically mentioned as conflicting in TroubleShooting. Could you try without vga16fb? Thanks. -- Pekka Paalanen http://www.iki.fi/pq

Re: [Nouveau] Kernel module fails to initialize on AMD751 based system with NV34

these are not loaded? (This is a basic troubleshooting question, http://nouveau.freedesktop.org/wiki/TroubleShooting ) A complete kernel log would have shown, if these bad drivers load. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau

Re: [Nouveau] xf86-video-nouveau-git and xorg-server-git

, not errors. If your build fails, the reason is something else. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] Blank screen at startup (conflict with VESA VGA)

that recently: http://people.freedesktop.org/~cbrill/dri-log/index.php?date=2010-03-21channel=nouveaushow_html=truehighlight_names=update=Updatedate=2010-03-21 I haven't heard anything about it since. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau

Re: [Nouveau] Blank screen at startup (conflict with VESA VGA)

? The hand-over should work: when nouveaufb initialises, vesafb gets automatically deactivated. In any case, complete kernel and X logs will help to diagnose your situation. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau

Re: [Nouveau] Gallium driver and compatibility issues

broken applications to debug on their own. When this situation changes, it will certainly be announced to the world. But not yet. That is the reason why end users should not be trying the 3D part of the driver set. This is also why developers are so hostile to 3D issues. -- Pekka Paalanen http

Re: [Nouveau] Gallium driver and compatibility issues

is what people are actually doing on their free time. Trying to kick their butt can only make them ignorant or angry. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman

Re: [Nouveau] no start with latest modules from git

- 70d0a48b6c3f1a817bf850acd3bae48d063e56b9 libdrm - 3130f94c6ee32668cb9f0b96b6c8e308a7bb3b11 mesa - e16f0c14f353cc04ad6cbcf99e3b95ccb1d2c06b And where is the kernel / nouveau drm ? This is probably even more important than the rest :) So upgrade it too with http://cgit.freedesktop.org/nouveau/linux-2.6/ -- Pekka Paalanen

Re: [Nouveau] [PATCH] drm/nouveau: don't hold spin lock while calling kzalloc with GFP_KERNEL

, under the spinlock, a call to a function that re-locks the spinlock, hence hangs. Sparse does warn about exiting a function without releasing a spinlock in every path, i.e., inconsistent lock behaviour. Thanks. -- Pekka Paalanen http://www.iki.fi/pq

Re: [Nouveau] [PATCH] drm/nouveau: don't hold spin lock while calling kzalloc with GFP_KERNEL

. Cheers. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] 2.6.33 vs. NVIDIA GForce GT 220

. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH] drm/nouveau: Add module options to disable acceleration.

should not be initialized to zero, since they default to zero. checkpatch.pl complains: ERROR: do not initialise externals to 0 or NULL -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org

Re: [Nouveau] [Discussion] User controls for PowerManagement

loss are mutually exclusive, since changing power modes is not free nor instantenous. Or is it? How much of the card you need to stop to change clocks and volts? Do you need to sync to vblank to prevent visual glitches? -- Pekka Paalanen http://www.iki.fi/pq

Re: [Nouveau] [Discussion] User controls for PowerManagement

temporary, it should probably go into debugfs. That way one can have the code in the kernel proper, not fear about freezing it, and prevent people from finding it by accident. And it should be guarded by an EXPERIMENTAL, DANGEROUS Kconfig option in any case. -- Pekka Paalanen http://www.iki.fi/pq

Re: [Nouveau] Removal of Non-KMS support

comments? -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] NV_PFIFO_INTR_DMA_PUSHER

a certain packet format. The pusher interrupt says the given stream is corrupt, the format is broken. This is either a code bug (e.g. BEGIN_RING with incorrect number of following commands), or (as in this case apparently) data corruption. That's all I think I know. -- Pekka Paalanen http

[Nouveau] Nouveau ctxprogs (Re: [git pull] drm)

On Thu, 10 Dec 2009 15:33:13 -0500 C. Bergström cbergst...@pathscale.com wrote: Pekka Paalanen The big question is what we call ctxprogs: binary blobs that are clearly executable, running somewhere in the GPU. No-one seems to know, if those are copyrightable, or if they can

Re: [Nouveau] [PATCH] nouveau: add ratelimit to nouveau_graph_dump_trap_info

On Fri, 27 Nov 2009 23:31:13 +0100 Xavier Chantry shinin...@gmail.com wrote: Often when things get stuck spinning, nouveau_pgraph_intr_error and nouveau_graph_dump_trap_info gets called many times, flooding logs with PGRAPH_ERROR and filling up the disk. Pekka Paalanen pointed me

[Nouveau] [PATCH] drm/nouveau: fix array overflow

Noticed by sparse: in nouveau_dp_link_train(), the array 'status' was defined as 2 bytes, yet it was accessed for 3 bytes. Fix it by making the array size 3. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_dp.c |3 ++- 1 files changed, 2 insertions(+), 1

Re: [Nouveau] drm git branches revisited

for testing latest drm stuff with an eye to the next Linus kernel. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] git - compile error ratelimit - missing include linux/ratelimit.h

to go. Aha, that problem wasn't there with 2.6.31-rc9 which is what Nouveau is still based on. I'll push a fix shortly, thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org

Re: [Nouveau] git - compile error ratelimit - missing include linux/ratelimit.h

to go. Okay, a fix pushed. Let me know, if it is still broken. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] mandriva 2010.1 cooker test report

does the above failed mean, exactly? - suspend to ram : ok - sees my two dvi heads : i made no test with 2 screens -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman

Re: [Nouveau] mandriva 2010.1 cooker test report

cc'd. btw. you can test monitor power saving on the command line, e.g. $ xset dpms force off For more states, see: xset help -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org

Re: [Nouveau] Introduction and discussion - Nouveau for Ubuntu Lucid

page per cpu, plus there can be some spurious kernel messages while using gdb. http://nouveau.freedesktop.org/wiki/MmioTrace It would also help if you have an easy way to switch between the Nvidia proprietary driver and Nouveau. -- Pekka Paalanen http://www.iki.fi/pq

[Nouveau] [PATCH 1/2] drm/nouveau: rework bios delays

milliseconds. call_lvds_manufacturer_script() contained a bug in the wait: the delay was executed in microseconds, when it should be milliseconds. Fixed. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bios.c | 20 +++- 1 files changed, 11 insertions

[Nouveau] [PATCH 2/2] drm/nouveau: fix init_condition_time() reporting

The function reported bogus wait time, if the condition was never fulfilled. Fix it. Also do some style fixes in this function and fix a typo. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bios.c | 13 + 1 files changed, 9 insertions(+), 4 deletions

Re: [Nouveau] [PATCH 3/3] nv10/exa: Replace celcius with celsius.

-f a8,x8r8g8b8,a8r8g8b8 and all tests pass before and after. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH] drm/nouveau: Add DRM_NOUVEAU_DEBUG option

|2 +- 5 files changed, 14 insertions(+), 5 deletions(-) Pushed, thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] master-compat branch in nouveau/linux-2.6

On Tue, 29 Sep 2009 19:46:32 +0300 Pekka Paalanen p...@iki.fi wrote: Hi, I would like to abandon the master-compat branch nouveau/linux-2.6 kernel tree. The difference to master branch is practically just the out-of-tree build Makefile. I'll find a new place for the Makefile and docs

Re: [Nouveau] [PATCH 3/3] drm/nouveau: Fix TV-out for the new semantics of drm_mode_vrefresh().

All three patches pushed. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

[Nouveau] master-compat branch in nouveau/linux-2.6

if not already there. If you do not like the idea, let me know why. Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

[Nouveau] [PATCH 2/2] drm/nouveau: unify nouveau_acpi.c code with the driver style

driver. To be able to use NV_ERROR(), pass struct drm_device * as argument to nouveau_dsm() instead of struct pci_dev *. Also, 'dev' in nouveau code is usually a drm_device and pdev is a pci_dev, so fix that too. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_acpi.c

[Nouveau] [PATCH 1/2] drm/nouveau: unify logging format with DRM core

Change NV_PRINTK() to use DRM_NAME and DRIVER_NAME, making it essentially generic. Print DRM_NAME in brackets, just like core DRM logging macros do. Convert two printk()'s into NV_* logging macro calls. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_acpi.c |2

Re: [Nouveau] [PATCH] drm/nouveau: fix WITHIN macro off by one

On Sun, 20 Sep 2009 13:29:24 +0200 Stephane Marchesin marche...@icps.u-strasbg.fr wrote: On Fri, Sep 18, 2009 at 20:15, Pekka Paalanen p...@iki.fi wrote: Equality in x = y + z is already out of range. X must be stricly less. Yeah, I did that back when the SIZE defines were, say, 0x1fff

[Nouveau] [PATCH] drm/nouveau: fix WITHIN macro off by one

Equality in x = y + z is already out of range. X must be stricly less. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bios.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.c b/drivers/gpu/drm/nouveau

[Nouveau] [PATCH 1/3] drm/nouveau: change channel regs mapping to ioremap

Use ioremap() for mapping the channel user regs (that are never exposed to user space) instead of drm_addmap(). This removes the last use cases of drm_addmap/drm_rmmap from Nouveau. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_channel.c | 13

Re: [Nouveau] Dropping compat from master-compat

On Wed, 9 Sep 2009 20:51:36 +0300 Pekka Paalanen p...@iki.fi wrote: Therefore, I will drop compatibility code from master-compat branch, and I will add a compile time check for the kernel version. The master-compat branch will be just like the master branch, except it contains the out-of-tree

Re: [Nouveau] Dropping compat from master-compat

On Wed, 9 Sep 2009 22:02:13 +0200 Andreas Radke a.ra...@arcor.de wrote: Am Wed, 9 Sep 2009 20:51:36 +0300 schrieb Pekka Paalanen p...@iki.fi: Therefore, I will drop compatibility code from master-compat branch, and I will add a compile time check for the kernel version. The master

Re: [Nouveau] [PATCH 4/4] drm/nouveau: fix signedness in nouveau_gpuobj_new()

On Sun, 6 Sep 2009 21:16:03 +0300 Pekka Paalanen p...@iki.fi wrote: The size parameter is usually unsigned, but here it was signed. Change 'int' to 'uint32_t' and propagate the signedness fixes to all callers. Fixes the sparse warning: nouveau_object.c:254:61: warning: incorrect type

Re: [Nouveau] [PATCH] drm/nouveau: fix ref leak in nouveau_gem_pushbuf_validate()

From ee49e66551a28550b69a1a0fc7f45ec2ab24c295 Mon Sep 17 00:00:00 2001 From: Pekka Paalanen p...@iki.fi Date: Tue, 8 Sep 2009 00:05:24 +0300 Subject: [PATCH] drm/nouveau: do not call ttm_bo_wait_unreserved() In nouveau_gem_puhsbuf_validate() path, ttm_bo_reserve() already calls

[Nouveau] [PATCH 1/4] drm/nouveau: add reg_debug module parameter

-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_drv.c |6 drivers/gpu/drm/nouveau/nouveau_drv.h | 20 +++ drivers/gpu/drm/nouveau/nouveau_hw.h | 42 +--- 3 files changed, 48 insertions(+), 20 deletions(-) diff --git a/drivers

[Nouveau] [PATCH 3/4] drm/nouveau: remove NVRead and NVWrite

These functions were effectively nv_rd32 and nv_wr32 with a debug print, and they were never used. Remove them. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_drv.c |7 --- drivers/gpu/drm/nouveau/nouveau_drv.h | 11 +-- drivers/gpu/drm/nouveau

[Nouveau] [PATCH 4/4] drm/nouveau: fix signedness in nouveau_gpuobj_new()

The size parameter is usually unsigned, but here it was signed. Change 'int' to 'uint32_t' and propagate the signedness fixes to all callers. Fixes the sparse warning: nouveau_object.c:254:61: warning: incorrect type in argument 3 (different signedness). Signed-off-by: Pekka Paalanen p...@iki.fi

Re: [Nouveau] [PATCH] drm/nouveau: init some list_heads

/nouveau -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] Debugging help needed

with that. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCHv2 2/2] drm/nouveau: Make the tv_norm parameter description more useful.

On Tue, 25 Aug 2009 15:41:27 +0200 Francisco Jerez curroje...@riseup.net wrote: Signed-off-by: Francisco Jerez curroje...@riseup.net --- drivers/gpu/drm/nouveau/nouveau_drv.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) Both pushed, thanks. -- Pekka Paalanen http

Re: [Nouveau] [PATCH] drm/i2c/ch7006: Fix the scale parameter handling.

On Tue, 25 Aug 2009 16:23:43 +0200 Francisco Jerez curroje...@riseup.net wrote: Signed-off-by: Francisco Jerez curroje...@riseup.net --- drivers/gpu/drm/i2c/ch7006_drv.c | 14 ++ 1 files changed, 6 insertions(+), 8 deletions(-) Pushed, thanks. -- Pekka Paalanen http

Re: [Nouveau] [PATCH] drm/nouveau: rewrite nouveau_dma_wait()

to the beginning of the + * push buffer if we require it. we can never get GET == PUT + * here, so this is safe. + */ + chan-dma.free = get - chan-dma.cur - 1; } - return ret; + return 0; } + -- 1.6.4 -- Pekka Paalanen

Re: [Nouveau] [PATCH] drm/nouveau: rewrite nouveau_dma_wait()

-dma.free = get - chan-dma.cur - 1; should become equivalent to chan-dma.free = chan-dma.max - chan-dma.cur; but there is the -1. Or do we observe GET == dma.max + 1 just before it actually jumps? Or do we simply not know how it behaves and want to be safe? -- Pekka Paalanen http://www.iki.fi/pq

[Nouveau] Delays in DRM nouveau_bios.c

Hi, questions will follow. --- diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.c b/drivers/gpu/drm/nouveau/nouveau_bios.c index 99f7bd4..13b3fb1 100644 --- a/drivers/gpu/drm/nouveau/nouveau_bios.c +++ b/drivers/gpu/drm/nouveau/nouveau_bios.c @@ -40,8 +40,6 @@ #define BIOSLOG(sip, fmt,

[Nouveau] [PATCH 1/4] drm/nouveau: refactor nouveau_dma_wait()

A cleanup of nouveau_dma_wait(): extract a sub-function and eliminate two variables to improve readability. No functional changes. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_dma.c | 72 ++--- 1 files changed, 39 insertions(+), 33

[Nouveau] [PATCH 2/4] drm/nouveau: make WRITE_PUT a function

Make WRITE_PUT() an inline function, and pass chan as an argument. Update chan-dma.put in WRITE_PUT() instead of (almost) everywhere by hand afterwards. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_dma.c |5 ++--- drivers/gpu/drm/nouveau/nouveau_dma.h | 16

[Nouveau] [PATCH 2/2] drm/nouveau: fix bo access after CALL implementation

TTM buffer object's virtual cookie may be an iomem cookie or a kernel virtual address. These need to be handled separately. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_gem.c | 21 ++--- 1 files changed, 14 insertions(+), 7 deletions(-) diff

[Nouveau] drm bo accessors etc. v2

Revised patch set v2. [PATCH 1/8] drm/nouveau: bo read/write wrappers for nv04_crtc.c [PATCH 2/8] drm/nouveau: use bo accessors for push buffers [PATCH 3/8] drm/nouveau: OUT_RINGp - optimize OUT_RING loops [PATCH 4/8] drm/nv50: proper notifier_bo access in nv50_display_vblank_crtc_handler()

[Nouveau] [PATCH 1/8] drm/nouveau: bo read/write wrappers for nv04_crtc.c

not cost much. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bo.c | 23 +++ drivers/gpu/drm/nouveau/nouveau_drv.h |2 ++ drivers/gpu/drm/nouveau/nv04_crtc.c |7 ++- 3 files changed, 27 insertions(+), 5 deletions(-) diff --git

[Nouveau] [PATCH 6/8] drm/nouveau: screen_base and lut must be iomem

This introduces nvbo_kmap_obj_iovirtual() for cases where the bo memory is guaranteed (or supposed) to be iomem. Fbcon assumes screen_base is iomem, and the crtc color LUT must reside in iomem. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_drv.h | 11

[Nouveau] [PATCH 5/8] drm/nouveau: access fbcon notifier via bo accessors

Notifier can live in system memory or VRAM, hence they must be accessed using the bo wrapper functions. Remove the redundant member nouveau_channel::m2mf_ntfy_map. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_dma.c |2 -- drivers/gpu/drm/nouveau

[Nouveau] [PATCH 7/8] drm/nouveau: remove redundant check in nouveau_bo_del_ttm()

ttm_bo_kunmap() already checks if the bo is not mapped. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bo.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c index

[Nouveau] [PATCH 8/8] drm/nouveau: remove mask in nv04_fbcon_imageblit()

this obfuscation. Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nv04_fbcon.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nv04_fbcon.c b/drivers/gpu/drm/nouveau/nv04_fbcon.c index 14fc87f..3d19492 100644 --- a/drivers

Re: [Nouveau] [PATCH 05/12] drm/nouveau: Use drm_encoder_slave instead of drm_encoder.

drm_device *dev, struct dcb_entry *entry); -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/nouveau

Re: [Nouveau] [PATCH 10/12] drm: Import driver for the ch7006 I2C TV encoder chip.

branch, we would have to remember to revert it when submitting Nouveau upstream. OTOH, we could apply it to master-compat branch for testing. How's that sound? Thanks. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau

Re: [Nouveau] I have a multihead setup

-KMS (as in user mode setting, UMS) does still exist, but the development is slowly moving away from it. You should stick to the #nouveau irc channel 24/7 if you can :-) Cheers. -- Pekka Paalanen http://www.iki.fi/pq/ ___ Nouveau mailing list Nouveau

[Nouveau] [PATCH 1/7] drm/nouveau: sparse cleanups for undeclared vs. static

Module parameters declarations moved to nouveau_drv.h. Ioctl table declarations moved to nouveau_drv.h. nouveau_ioc32.c: include nouveau_drv.h for nouveau_compat_ioctl(). Signed-off-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_bios.c |3 --- drivers/gpu/drm/nouveau

[Nouveau] [PATCH 2/7] drm/nouveau: collect switch blocks in nouveau_fbcon.c

The two large switch blocks are refactored into a function. While doing this, some inconsistencies are corrected: - in nouveau_fbcon_check_var(), case 30 was writing to a wrong structure. - in nouveau_fbcon_create(), case 16 was missing transp.length. Signed-off-by: Pekka Paalanen p...@iki.fi

[Nouveau] [PATCH 3/7] drm/nouveau: FB_ACCEL_I830 in fbcon is wrong

-by: Pekka Paalanen p...@iki.fi --- drivers/gpu/drm/nouveau/nouveau_fbcon.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fbcon.c b/drivers/gpu/drm/nouveau/nouveau_fbcon.c index 7ccadc0..228e763 100644 --- a/drivers/gpu/drm/nouveau

  1   2   >