Hey everybody,
so, here is a proposal of what we could change in order to allow
patches to land faster, more reliably and to increase the overall bus
factor in terms of nouveau kernel maintenance.
But let's start with the current situation:
At the moment contributors have to send patches to the
On Wed, Aug 4, 2021 at 11:10 PM Arnd Bergmann wrote:
>
> On Wed, Aug 4, 2021 at 8:59 PM Karol Herbst wrote:
> > On Wed, Aug 4, 2021 at 4:43 PM Karol Herbst wrote:
> > > On Wed, Aug 4, 2021 at 4:19 PM Arnd Bergmann wrote:
> > > > On Wed, Aug 4, 20
On Wed, Aug 4, 2021 at 4:43 PM Karol Herbst wrote:
>
> On Wed, Aug 4, 2021 at 4:19 PM Arnd Bergmann wrote:
> >
> > On Wed, Aug 4, 2021 at 4:10 PM Karol Herbst wrote:
> > >
> > > playing around a little bit with this, I think the original "select
>
On Wed, Aug 4, 2021 at 4:19 PM Arnd Bergmann wrote:
>
> On Wed, Aug 4, 2021 at 4:10 PM Karol Herbst wrote:
> >
> > playing around a little bit with this, I think the original "select
> > BACKLIGHT_CLASS_DEVICE" is fine. Atm we kind of have this weird mix o
playing around a little bit with this, I think the original "select
BACKLIGHT_CLASS_DEVICE" is fine. Atm we kind of have this weird mix of
drivers selecting and others depending on it. We could of course convert
everything over to depend, and break those cycling dependency issues with
this.
Anyway
functions with the official version.
> The behavior remains unchanged.
>
> Cc: Steven Rostedt
> Cc: Ingo Molnar
> Cc: Karol Herbst
> Cc: Pekka Paalanen
> Cc: Dave Hansen
> Cc: Andy Lutomirski
> Cc: Peter Zijlstra
> Cc: Thomas Gleixner
> Cc: Borislav Petkov
On Thu, Jul 22, 2021 at 5:10 AM Ratchanan Srirattanamet
wrote:
>
> The call site of nouveau_dsm_pci_probe() uses single set of output
> variables for all invocations. So, we must not write anything to them
> until we think this is an NVIDIA device of interest. Otherwise, if we
> are called with an
On Sat, Jul 24, 2021 at 4:05 PM Arnd Bergmann wrote:
>
> On Sat, Jul 24, 2021 at 2:52 PM Karol Herbst wrote:
> >
> > On Sat, Jul 24, 2021 at 2:10 PM Karol Herbst wrote:
> > >
> > > On Sat, Jul 24, 2021 at 1:56 PM Arnd Bergmann wrote:
> > > >
On Sat, Jul 24, 2021 at 2:10 PM Karol Herbst wrote:
>
> On Sat, Jul 24, 2021 at 1:56 PM Arnd Bergmann wrote:
> >
> > On Sat, Jul 24, 2021 at 11:55 AM Karol Herbst wrote:
> > >
> > > On Sat, Jul 24, 2021 at 8:55 AM Arnd Bergmann wrote:
> > > >
On Sat, Jul 24, 2021 at 1:56 PM Arnd Bergmann wrote:
>
> On Sat, Jul 24, 2021 at 11:55 AM Karol Herbst wrote:
> >
> > On Sat, Jul 24, 2021 at 8:55 AM Arnd Bergmann wrote:
> > >
> > > On Sat, Jul 24, 2021 at 12:47 AM Karol Herbst wrote:
> > > >
On Sat, Jul 24, 2021 at 8:55 AM Arnd Bergmann wrote:
>
> On Sat, Jul 24, 2021 at 12:47 AM Karol Herbst wrote:
> >
> > In the past this only led to compilation issues. Also the small amount of
> > extra .text shouldn't really matter compared to the ent
ri-de...@lists.freedesktop.org
Fixes: 6eca310e8924 ("drm/nouveau/kms/nv50-: Add basic DPCD backlight support
for nouveau")
Signed-off-by: Karol Herbst
---
drivers/gpu/drm/nouveau/Kbuild | 2 +-
drivers/gpu/drm/nouveau/Kconfig | 13 ++-
drivers/gpu/drm/nouveau/
On Fri, Jul 23, 2021 at 8:40 PM Arnd Bergmann wrote:
>
> On Fri, Jul 23, 2021 at 6:34 PM Karol Herbst wrote:
> > On Fri, Jul 23, 2021 at 6:31 PM Randy Dunlap wrote:
> > > On 7/23/21 8:15 AM, Karol Herbst wrote:
> > > > On Fri, Jul 23, 2021 at 5:1
On Fri, Jul 23, 2021 at 6:31 PM Randy Dunlap wrote:
>
> On 7/23/21 8:15 AM, Karol Herbst wrote:
> > On Fri, Jul 23, 2021 at 5:10 PM Randy Dunlap wrote:
> >>
> >> On 7/23/21 2:15 AM, Arnd Bergmann wrote:
> >>> From: Arnd Bergmann
> >>>
&g
On Fri, Jul 23, 2021 at 5:10 PM Randy Dunlap wrote:
>
> On 7/23/21 2:15 AM, Arnd Bergmann wrote:
> > From: Arnd Bergmann
> >
> > When the backlight support is disabled, the driver fails to build:
> >
> > drivers/gpu/drm/nouveau/dispnv50/disp.c: In function
> > 'nv50_sor_atomic_disable':
> > driv
On Fri, Jul 23, 2021 at 12:10 PM Karol Herbst wrote:
>
> On Fri, Jul 23, 2021 at 11:24 AM Daniel Vetter wrote:
> >
> > On Fri, Jul 23, 2021 at 11:15 AM Arnd Bergmann wrote:
> > >
> > > From: Arnd Bergmann
> > >
> > > When the ba
On Fri, Jul 23, 2021 at 11:24 AM Daniel Vetter wrote:
>
> On Fri, Jul 23, 2021 at 11:15 AM Arnd Bergmann wrote:
> >
> > From: Arnd Bergmann
> >
> > When the backlight support is disabled, the driver fails to build:
> >
> > drivers/gpu/drm/nouveau/dispnv50/disp.c: In function
> > 'nv50_sor_atomi
On Thu, Jul 22, 2021 at 9:54 PM Karol Herbst wrote:
>
> On Thu, Jul 22, 2021 at 9:49 PM Ratchanan Srirattanamet
> wrote:
> >
> > Hello,
> >
> > เมื่อ 22/7/64 เวลา 23:36 Karol Herbst เขียนว่า:
> > > hey, thanks for the patch. But I am a bit confused
On Thu, Jul 22, 2021 at 9:49 PM Ratchanan Srirattanamet
wrote:
>
> Hello,
>
> เมื่อ 22/7/64 เวลา 23:36 Karol Herbst เขียนว่า:
> > hey, thanks for the patch. But I am a bit confused on why that patch
> > actually helps. It should only be called for nvidia GPUs, but are we
&
hey thanks for the report.
This is a known issue and the fix is pending in drm-mist-fixes and
should land in 5.14 soonish.
On Thu, Jul 22, 2021 at 9:29 PM Ondrej Zary wrote:
>
> Hello,
> nouveau is broken again:
>
> [ 58.795794] BUG: kernel NULL pointer dereference, address: 017c
> [ 58.
hey, thanks for the patch. But I am a bit confused on why that patch
actually helps. It should only be called for nvidia GPUs, but are we
ending up checking it for AMD GPUs as well?
Mind posting the output of lspci -tvnn?
On Thu, Jul 22, 2021 at 5:10 AM Ratchanan Srirattanamet
wrote:
>
> The cal
actually.. it was already found in this thread:
https://lore.kernel.org/lkml/YOC4uekpD7iA3xPi@Red/T/ fixes incoming.
On Wed, Jul 21, 2021 at 3:13 PM Karol Herbst wrote:
>
> fyi: I am able to reproduce it on my machine and will figure out
> what's wrong. Thanks!
>
> On Mon, J
fyi: I am able to reproduce it on my machine and will figure out
what's wrong. Thanks!
On Mon, Jul 19, 2021 at 10:53 PM Sid Boyce wrote:
>
> Also sent this to kernel mailing list.
>
> I saw an earlier kernel mailing list post (week 8 - 15 July) that so far
> hasn't made it into 5.14-rc.
>
>
> GPU
hen.
>
As long as it doesn't touch code I don't see why not. Just in case:
Reviewed-by: Karol Herbst
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
On Tue, Jun 22, 2021 at 5:14 PM Cai Huoqing wrote:
>
> fix the warning- variable 'dev' set but not used
>
the patch title needs a prefix to indicate which subsystem it belongs
to. Check git log for examples.
With that fixed: Reviewed-by: Karol Herbst
> Sign
all three nouveau patches are
Reviewed-by: Karol Herbst
and I don't think anybody would mind if those get into through other
trees, but maybe drm-mist would be a good place for it if other
patches involve other drm drivers?
On Wed, Jun 2, 2021 at 1:16 AM Gustavo A. R. Silva
wrote:
&
t; spin_unlock_irqrestore(&mc->lock, flags);
> }
>
> -void
> +static void
> tu102_mc_intr_mask(struct nvkm_mc *base, u32 mask, u32 intr)
> {
> struct tu102_mc *mc = tu102_mc(base);
> --
> 2.31.1
>
> _
On Mon, May 17, 2021 at 11:17 AM Thierry Reding wrote:
>
> On Mon, May 17, 2021 at 10:56:29AM +0200, Thierry Reding wrote:
> > On Tue, May 11, 2021 at 06:35:53PM +0200, Karol Herbst wrote:
> > > If ttm_bo_init fails it will already call ttm_bo_put, so we don't h
On Thu, May 13, 2021 at 5:25 PM Jeremy Cline wrote:
>
> On Tue, May 11, 2021 at 06:35:53PM +0200, Karol Herbst wrote:
> > If ttm_bo_init fails it will already call ttm_bo_put, so we don't have to
> > do it
rm/nouveau: Initialize GEM object before TTM object"
Cc: Thierry Reding
Signed-off-by: Karol Herbst
---
drivers/gpu/drm/nouveau/nouveau_gem.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c
b/drivers/gpu/drm/nouveau/nouveau_gem.c
index c88cbb85f10
I've notified the gitlab admins, but maybe you can use the social
login (google, github, twitter, etc..) for the moment or would that
require the confirmation email as well?
On Thu, Apr 22, 2021 at 3:44 PM o1bigtenor wrote:
>
> Greetings
>
> In the process of setting up an account at the https://
On Wed, Apr 21, 2021 at 3:57 PM Dan Carpenter wrote:
>
> Hello Oscar Salvador,
>
> The patch bfb96e4c344e: "drm/nouveau/hwmon: Remove old code, add
> .write/.read operations" from May 18, 2017, leads to the following
> static checker warning:
>
> drivers/gpu/drm/nouveau/nouveau_hwmon.c:507 nou
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/dispnv50/disp.c:2599:1: warning: no previous
> prototype for ‘nv50_display_create’ [-Wmissing-prototypes]
>
> Cc: Be
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nouveau_ioc32.c:2: warning: Cannot understand *
> file mga_ioc32.c
>
> Cc: Ben Skeggs
> Cc: David Airlie
&
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nouveau_ioc32.c:52: warning: Function parameter or
> member 'filp' not described in 'nouveau_compat_ioc
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nouveau_svm.c: In function ‘nouveau_pfns_map’:
> drivers/gpu/drm/nouveau/nouveau_svm.c:810:6: warning: variable ‘ret’ set but
> not used [-Wunused-but-set-variable]
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following build error:
>
> drivers/gpu/drm/nouveau/dispnv50/disp.c:2530:1: error: conflicting types for
> ‘nv50_display_fini’
> In file included from drivers/gpu/drm/nouveau/
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/dispnv50/headc57d.c:173:1: warning: no previous
> prototype for ‘headc57d_olut’ [-Wmissing-prototypes]
>
> Cc: Be
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/dispnv50/disp.c: In function ‘nv50_mstm_cleanup’:
> drivers/gpu/drm/nouveau/dispnv50/disp.c:1357:6: warning: variable ‘ret’ set
> but not used [-Wunused-but-set-vari
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:38 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/dispnv04/crtc.c:462: warning: Function parameter or
> member 'crtc' not described in 'nv_crtc_mode_set_re
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:37 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nouveau_bo.c: In function ‘nouveau_ttm_tt_populate’:
> drivers/gpu/drm/nouveau/nouveau_bo.c:1228:17: warning: variable ‘
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:37 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nvkm/subdev/volt/gk20a.c:53: warning: Function
> parameter or member 'speedo' not described in 'gk20a_vo
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:37 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c:992: warning: Function
> parameter or member 'gr' not described in 'gf100_gr_wait_
On Fri, Apr 16, 2021 at 4:37 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/dispnv50/disp.c:1381:6: warning: variable ‘ret’ set
> but not used [-Wunused-but-set-variable]
>
not a big fan of just ignoring return codes, I'd rather see it handl
Reviewed-by: Karol Herbst
On Fri, Apr 16, 2021 at 4:37 PM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c:584: warning: Function
> parameter or member 'init' not described in 'init_reserv
On Tue, Apr 13, 2021 at 1:17 PM Roy Spliet wrote:
>
> Op 13-04-2021 om 10:48 schreef Karol Herbst:
> > On Tue, Apr 13, 2021 at 10:24 AM Roy Spliet wrote:
> >>
> >> Op 13-04-2021 om 01:10 schreef Karol Herbst:
> >>> On Mon, Apr 12, 2021 at 9:36 PM Ro
On Tue, Apr 13, 2021 at 10:24 AM Roy Spliet wrote:
>
> Op 13-04-2021 om 01:10 schreef Karol Herbst:
> > On Mon, Apr 12, 2021 at 9:36 PM Roy Spliet wrote:
> >>
> >> Hello Aaron,
> >>
> >> Thanks for your insights. A follow-up query and some observ
On Mon, Apr 12, 2021 at 9:36 PM Roy Spliet wrote:
>
> Hello Aaron,
>
> Thanks for your insights. A follow-up query and some observations in-line.
>
> Op 12-04-2021 om 20:06 schreef Aaron Plattner:
> > On 4/10/21 1:48 PM, Roy Spliet wrote:
> >> Op 10-04-2021 om 20:23 schreef Lukas Wunner:
> >>> On
On Tue, Mar 30, 2021 at 8:28 AM wrote:
>
> Hi,
> I am currently the happy owner of a nforce 4 - SLI with two 7100gs graphics
> cards setup.
> Since I've read on your features matrix that such a setup is rather uncommon
> among developers, I would like to help you implementing SLI for old GeForce
ize != 1024)
> --
> 2.27.0
>
> ___
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Reviewed-by: Karol Herbst
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
; *
> * 32-bit ioctl compatibility routines for the MGA DRM.
> --
> 2.27.0
>
> ___
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
Reviewed-by: Karol Herbst
m_framebuffer *old_fb)
> return ret;
> }
>
> -/**
> +/*
> * Sets up registers for the given mode/adjusted_mode pair.
> *
> * The clocks, CRTCs and outputs attached to this CRTC must be off.
> --
> 2.27.0
>
> ___
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
Reviewed-by: Karol Herbst
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
e "nouveau_ioctl.h"
>
> -/**
> +/*
> * Called whenever a 32-bit process running under a 64-bit kernel
> * performs an ioctl on /dev/dri/card.
> *
> --
> 2.27.0
>
> _______
> dri-devel mailing list
> dri-de..
; TTM_PAGE_FLAG_SG);
>
> if (slave)
> return;
>
> drm = nouveau_bdev(bdev);
> - dev = drm->dev->dev;
>
> return ttm_pool_free(&drm->ttm.bdev.pool, ttm);
> }
> --
> 2.27.0
>
> ___
gt; _______
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Reviewed-by: Karol Herbst
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
t;
> ___
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
Reviewed-by: Karol Herbst
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau
c0)
> */
> static inline int
> @@ -58,7 +58,7 @@ gk20a_volt_get_cvb_voltage(int speedo, int s_scale, const
> struct cvb_coef *coef)
> return mv;
> }
>
> -/**
> +/*
> * cvb_t_mv =
> * ((c2 * speedo / s_sc
Reviewed-by: Karol Herbst
On Fri, Mar 19, 2021 at 9:24 AM Lee Jones wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c:584: warning: Function
> parameter or member 'init' not described in 'init_reserv
On Sun, Mar 14, 2021 at 11:04 PM wrote:
>
> https://forum.artixlinux.org/index.php/topic,2083.msg14006.html#msg14006
> https://forum.artixlinux.org/index.php/topic,2383.new.html#new
>
> It is going to blow up many more now, the freeze bug may take days, weeks or
> months to appear, now with this
On Thu, Mar 4, 2021 at 3:21 AM o1bigtenor wrote:
>
> Greetings
>
> Running debian testing, 2 nvidia gpus (GP107 and GF110) and 5 monitors
> (1 - 3840x2160 and 2 1920x1080 on the GP107 and 2 - 1920x1080 on the
> GF110) using one screen (7680x3000).
>
> I would still like to modify some things but c
On Wed, Mar 3, 2021 at 11:07 AM Tomek LECOCQ
wrote:
> Hello,
>
> I’ve already asked this on the Kernel Newbies mail list, but as developing
> nouveau seems to be kind of similar to what I want to achieve, I thought it
> would be a good idea to ask it here as well.
>
> I have a PCI-Express video c
Reviewed-by: Karol Herbst
On Thu, Jan 21, 2021 at 3:28 PM Bastian Beranek
wrote:
>
> Commit f844eb485eb056ad3b67e49f95cbc6c685a73db4 introduced a regression for
> NV50, which lead to visual artifacts, tearing and eventual crashes.
>
> In the changes of f844eb485eb056ad3b67e49f9
ohh, at first I thought this was a resend, but it's in a different file :)
On Thu, Jan 21, 2021 at 12:11 PM Bastian Beranek
wrote:
>
> Commit f844eb485eb056ad3b67e49f95cbc6c685a73db4 introduced a regression for
> NV50, which lead to visual artifacts, tearing and eventual crashes.
>
> In the chang
fyi, there is a patch which solves a maybe related issue on your GPU,
mind giving it a try before we dig further?
https://gitlab.freedesktop.org/drm/nouveau/-/issues/14#note_767791
On Thu, Jan 21, 2021 at 3:33 AM Jamie Heilman
wrote:
>
> Karol Herbst wrote:
> > On Wed, Jan 6, 202
Fixes a crash when trying to create a channel on e.g. Turing GPUs when
NOUVEAU_SVM_INIT was called before.
Fixes: eeaf06ac1a558 ("drm/nouveau/svm: initial support for shared virtual
memory")
Signed-off-by: Karol Herbst
---
drivers/gpu/drm/nouveau/nouveau_svm.c | 4
1 file
Reviewed-by: Karol Herbst
On Fri, Jan 15, 2021 at 11:51 PM wrote:
>
> From: Tom Rix
>
> Defining DEBUG should only be done in development.
> So remove DEBUG.
>
> Signed-off-by: Tom Rix
> ---
> kernel/trace/trace_mmiotrace.c | 2 --
> 1 file changed, 2 deleti
On Wed, Jan 6, 2021 at 4:25 AM Jamie Heilman
wrote:
>
> Jamie Heilman wrote:
> > Jamie Heilman wrote:
> > > Karol Herbst wrote:
> > > > do you think you'd be able to do a kernel bisect in order to pinpoint
> > > > the actual commit causing it? Tha
mhh, that PCI config stuff should really not happen all the time, but
it also doesn't appear to. The other thing I really don't know is, how
well the runpm works with tools like TLP if there isn't only an audio
device, but also the USB stuff and all the subdevices have to be
turned off all the time
do you think you'd be able to do a kernel bisect in order to pinpoint
the actual commit causing it? Thanks
On Sun, Dec 27, 2020 at 8:16 PM Jamie Heilman
wrote:
>
> Something between v5.8 and v5.9 has resulted in periodically losing video.
> Unfortunately, I can't reliably reproduce it, it seems t
On Tue, Dec 22, 2020 at 3:50 AM Kai-Heng Feng
wrote:
>
> On Tue, Dec 22, 2020 at 1:56 AM Ilia Mirkin wrote:
> >
> > On Mon, Dec 21, 2020 at 11:33 AM Kai-Heng Feng
> > wrote:
> > >
> > > [+Cc nouveau]
> > >
> > > On Fri, Dec 18, 2020 at 4:06 PM Takashi Iwai wrote:
> > > [snip]
> > > > > Quite po
uhm.. good question, but I think so. Might be best to ask on the xorg ML as well
On Mon, Dec 21, 2020 at 2:10 PM Nikunj goyal wrote:
>
> Hi everyone,
> I was surfing the web and i came across this program endless summer of code
> by Xorg and wanted to ask if it is still active?
> Thanks
>
> Rega
On Thu, Nov 26, 2020 at 4:28 PM Geert Uytterhoeven wrote:
>
> Hi Miguel,
>
> On Thu, Nov 26, 2020 at 3:54 PM Miguel Ojeda
> wrote:
> > On Wed, Nov 25, 2020 at 11:44 PM Edward Cree wrote:
> > > To make the intent clear, you have to first be certain that you
> > > understand the intent; otherwise
documentation.
Cc: sta...@vger.kernel.org
Cc: dri-de...@lists.freedesktop.org
Cc: Ben Skeggs
Reported-by: Mark Pearson
Tested-by: Mark Pearson
Signed-off-by: Karol Herbst
---
drivers/gpu/drm/nouveau/include/nvkm/subdev/bios/conn.h | 1 +
drivers/gpu/drm/nouveau/nouveau_connector.c | 1 +
2
Reviewed-by: Karol Herbst
On Thu, Nov 26, 2020 at 2:11 PM Christian König
wrote:
>
> This wasn't initialized for pre NV50 hardware.
>
> Signed-off-by: Christian König
> Reported-and-Tested-by: Mark Hounschell
> ---
> drivers/gpu/drm/nouveau/nouveau_bo.c | 2 +-
>
On Sun, Nov 15, 2020 at 6:43 PM Salvatore Bonaccorso wrote:
>
> Hi,
>
> On Fri, Aug 28, 2020 at 11:28:46AM +0200, Frantisek Hrbata wrote:
> > Unprivileged user can crash kernel by using DRM_IOCTL_NOUVEAU_CHANNEL_ALLOC
> > ioctl. This was reported by trinity[1] fuzzer.
> >
> > [ 71.073906] nouvea
On Fri, Nov 6, 2020 at 3:17 AM Jeremy Cline wrote:
>
> Make use of the devm_drm_dev_alloc() API to bind the lifetime of
> nouveau_drm structure to the drm_device. This is important because a
> reference to nouveau_drm is accessible from drm_device, which is
> provided to a number of DRM layer call
On Fri, Oct 30, 2020 at 3:37 AM Alistair Popple wrote:
>
> This is an initial series of patches to improve channel recovery on Turing
> GPUs
> with the goal of improving reliability enough to eventually enable SVM for
> Turing. It's likely follow up patches will be required to fully address
> pr
I sent a patch to the mailing list and wanted to have some review on
that from at least Ben, but no idea if Ben already picked it and if
it's good enough for sending it to stable yet.
___
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.f
On Thu, Oct 15, 2020 at 6:32 PM Karol Herbst wrote:
>
> Ben, I think this is like the 5th patch tackling this issue, I think
> we should merge one of those.
>
maybe I just confused that with reports, but it seems to turn up quite
a bit and maybe I should have pushed more of it as wel
Ben, I think this is like the 5th patch tackling this issue, I think
we should merge one of those.
On Thu, Oct 15, 2020 at 6:23 AM Keita Suzuki
wrote:
>
> struct pw_rail_t is allocated as an array in function nvios_iccsense_parse,
> and stored to a struct member of local variable. However, the ar
On Tue, Oct 13, 2020 at 3:55 PM Ilia Mirkin wrote:
>
> On Tue, Oct 13, 2020 at 8:01 AM Karol Herbst wrote:
> >
> > With this we try to detect if the endianess switch works and assume LE if
> > not. Suggested by Ben.
> >
> > Fixes: 51c05340e407 ("drm/nouv
ehh.. messed up sending it against the other thread, but mind testing
this patch and verify it fixes your issue? Thanks.
On Tue, Oct 13, 2020 at 2:01 PM Karol Herbst wrote:
>
> With this we try to detect if the endianess switch works and assume LE if
> not. Suggested by Ben.
With this we try to detect if the endianess switch works and assume LE if
not. Suggested by Ben.
Fixes: 51c05340e407 ("drm/nouveau/device: detect if changing endianness failed")
---
.../gpu/drm/nouveau/nvkm/engine/device/base.c | 39 ---
1 file changed, 26 insertions(+), 13 deleti
On Sat, Oct 10, 2020 at 12:23 AM Ilia Mirkin wrote:
>
> On Fri, Oct 9, 2020 at 5:54 PM Karol Herbst wrote:
> >
> > On Fri, Oct 9, 2020 at 11:35 PM Ondrej Zary wrote:
> > >
> > > Hello,
> > > I'm testing 5.9.0-rc8 and found that Riva TNT2 stopped
On Fri, Oct 9, 2020 at 11:35 PM Ondrej Zary wrote:
>
> Hello,
> I'm testing 5.9.0-rc8 and found that Riva TNT2 stopped working:
> [0.00] Linux version 5.9.0-rc8+ (zary@gsql) (gcc (Debian 8.3.0-6)
> 8.3.0, GNU ld (GNU Binutils for Debian) 2.31.1) #326 SMP Fri Oct 9 22:31:40
> CEST 2020
>
liar fixes?
Signed-off-by: Karol Herbst
---
drm/nouveau/nvkm/engine/fifo/gk104.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drm/nouveau/nvkm/engine/fifo/gk104.c
b/drm/nouveau/nvkm/engine/fifo/gk104.c
index 441483325..669153fba 100644
--- a/drm/nouveau/nvkm/engine/
we can't use nouveau_bo_ref here as no ttm object was allocated and
nouveau_bo_ref mainly deals with that. Simply deallocate the object.
Signed-off-by: Karol Herbst
---
drm/nouveau/nouveau_gem.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drm/nouveau/nouveau_ge
On Tue, Oct 6, 2020 at 7:53 PM Jeremy Cline wrote:
>
> On Tue, Oct 06, 2020 at 06:46:35PM +0200, Karol Herbst wrote:
> > On Tue, Oct 6, 2020 at 6:43 PM dann frazier
> > wrote:
> > >
> > > On Mon, Oct 05, 2020 at 03:52:30PM +0200, Karol Herbst wrote:
> >
On Tue, Oct 6, 2020 at 6:43 PM dann frazier wrote:
>
> On Mon, Oct 05, 2020 at 03:52:30PM +0200, Karol Herbst wrote:
> > Previously the code relied on device->pri to be NULL and to fail probing
> > later. We really should just return an error inside nvkm_device_ctor for
Previously the code relied on device->pri to be NULL and to fail probing
later. We really should just return an error inside nvkm_device_ctor for
unsupported GPUs.
Fixes: 24d5ff40a732 ("drm/nouveau/device: rework mmio mapping code to get rid
of second map")
Signed-off-by: Karol Her
On Thu, Oct 1, 2020 at 7:53 PM Sam Ravnborg wrote:
>
> Hi Karol.
>
> On Thu, Oct 01, 2020 at 01:36:52PM +0200, Karol Herbst wrote:
> > On Fri, Sep 25, 2020 at 11:27 PM Karol Herbst wrote:
> > >
> > > Hi everybody,
> > >
> > > I think it'
On Fri, Sep 25, 2020 at 11:27 PM Karol Herbst wrote:
>
> Hi everybody,
>
> I think it's time to finally move our wiki from the old infrastructure
> over to gitlab pages.
>
> This comes with several benefits:
> * full control through git over the ikiwiki pipeline (s
Hi everybody,
I think it's time to finally move our wiki from the old infrastructure
over to gitlab pages.
This comes with several benefits:
* full control through git over the ikiwiki pipeline (setup files,
plugins, system packages, ...)
* random users are able to create MRs against the wiki as
On Thu, Sep 24, 2020 at 8:18 PM Jeremy Cline wrote:
>
> On Thu, Sep 24, 2020 at 07:26:01PM +0200, Karol Herbst wrote:
> > On Thu, Sep 24, 2020 at 6:03 PM Jeremy Cline wrote:
> > >
> > > On Wed, Sep 23, 2020 at 11:36:54PM +0200, Karol Herbst wrote:
> > > &g
On Thu, Sep 24, 2020 at 7:26 PM Karol Herbst wrote:
>
> On Thu, Sep 24, 2020 at 6:03 PM Jeremy Cline wrote:
> >
> > On Wed, Sep 23, 2020 at 11:36:54PM +0200, Karol Herbst wrote:
> > > On Wed, Sep 23, 2020 at 10:39 PM Jeremy Cline wrote:
> > > >
> >
On Thu, Sep 24, 2020 at 6:03 PM Jeremy Cline wrote:
>
> On Wed, Sep 23, 2020 at 11:36:54PM +0200, Karol Herbst wrote:
> > On Wed, Sep 23, 2020 at 10:39 PM Jeremy Cline wrote:
> > >
> > > On Wed, Sep 23, 2020 at 09:02:45PM +0200, Karol Herbst wrote:
> > > &
On Thu, Sep 24, 2020 at 3:06 PM Roy Spliet wrote:
>
>
> Op 23-09-2020 om 22:36 schreef Karol Herbst:
> > On Wed, Sep 23, 2020 at 10:39 PM Jeremy Cline wrote:
> >>
> >> On Wed, Sep 23, 2020 at 09:02:45PM +0200, Karol Herbst wrote:
> >>> On Fri
On Wed, Sep 23, 2020 at 10:39 PM Jeremy Cline wrote:
>
> On Wed, Sep 23, 2020 at 09:02:45PM +0200, Karol Herbst wrote:
> > On Fri, Sep 11, 2020 at 6:21 PM Jeremy Cline wrote:
>
>
>
> > yeah, I think overall this file is a good idea and being able to get a
> >
On Fri, Sep 11, 2020 at 6:21 PM Jeremy Cline wrote:
>
> Other gpu drivers have some driver-specific documentation, so it would
> nice if nouveau did as well.
>
> This adds a bare-bones ReStructured Text document with sections for
> module parameter documentation, an overview of the driver architec
On Thu, Sep 17, 2020 at 4:11 PM Jeremy Cline wrote:
>
> On Wed, Sep 16, 2020 at 10:03:22PM +0200, Karol Herbst wrote:
> > On Wed, Sep 16, 2020 at 10:01 PM Karol Herbst wrote:
> > >
> > > On Wed, Sep 16, 2020 at 9:47 PM Jeremy Cline wrote:
> > > >
201 - 300 of 1155 matches
Mail list logo