Re: [PATCH] video: fbdev: arkfb: fix possible object reference leak

2023-10-06 Thread Ondrej Zary
return rc; > > @@ -1085,7 +1085,7 @@ static void ark_pci_remove(struct pci_dev *dev) > > > > pci_iounmap(dev, info->screen_base); > > pci_release_regions(dev); > > -/* pci_disable_device(dev); */ > > + pci_disable_device(dev); > > > > framebuffer_release(info); > > } > > -- Ondrej Zary

Re: [PATCH v4 1/4] video: fbdev: atyfb: only use ioremap_uc() on i386 and ia64

2023-03-08 Thread Ondrej Zary
uan He > > Cc: Helge Deller > > Cc: Thomas Zimmermann > > Cc: Christophe Leroy > > Cc: linux-fb...@vger.kernel.org > > Cc: dri-devel@lists.freedesktop.org > > Reviewed-by: Luis Chamberlain > > Is anyone using this driver these days? How often do fbdev driv

[PATCH] fbdev: i740fb: use memset_io() to clear screen

2022-04-10 Thread Ondrej Zary
sparse complains that using memset() on __iomem pointer is wrong: incorrect type in argument 1 (different address spaces) Use memset_io() to clear screen instead. Tested on real i740 cards. Signed-off-by: Ondrej Zary --- drivers/video/fbdev/i740fb.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH 1/7] video: fbdev: i740fb: Error out if 'pixclock' equals zero

2022-04-10 Thread Ondrej Zary
> > better fix (to round up the pixclock when it's invalid). > > If not, I will apply this one later. > > I'm also looking forward to a more appropriate patch for this driver! I was not able to reproduce it at first but finally found it: the monitor must be unplugged. If a valid EDID is present, fb_validate_mode() call in i740fb_check_var() will refuse zero pixclock. Haven't found any obvious way to correct zero pixclock value. Most other drivers simply return -EINVAL. > Thanks, > Zheyu Ma > -- Ondrej Zary

Re: [BUG] fbdev: i740fb: Divide error when ‘var->pixclock’ is zero

2022-04-05 Thread Ondrej Zary
> round up the invalid to a valid value, if possible. > > I don't disagree. > The main problem probably is: what is the next valid value? > This needs to be analyzed on a per-driver base and ideally tested. > Right now a division-by-zero is tiggered which is probably more worse. I still have an i740 card so I can test it. > That said, currently I'd prefer to apply the zero-checks patches over > any untested patches. It's easy to revert such checks if a better solution > becomes available. > > Thoughts? > > > Commit b36b242d4b8ea178 ("video: fbdev: asiliantfb: Error out if > > 'pixclock' equals zero") does not do that. > > Helge > -- Ondrej Zary

Re: [Nouveau] nouveau broken again on Riva TNT2 in 5.14.0-rc2

2021-07-23 Thread Ondrej Zary
It just landed in Linus' tree yesterday, please retest that or -rc3. > If it's still broken it's something else. > -Daniel Thanks, it works! -- Ondrej Zary

nouveau broken again on Riva TNT2 in 5.14.0-rc2

2021-07-22 Thread Ondrej Zary
ort_pc 8139cp parport [ 58.798016] CR2: 017c [ 58.798147] ---[ end trace 732829d39ed65de9 ]--- d02117f8efaa5fbc37437df1ae955a147a2a424a is the first bad commit -- Ondrej Zary

Re: [PATCH] drm/nouveau: fix dma_address check for CPU/GPU sync

2021-06-14 Thread Ondrej Zary
On Monday 14 June 2021 13:05:17 Christian König wrote: > AGP for example doesn't have a dma_address array. > > Signed-off-by: Christian König Fixes NULL pointer dereference in nouveau_bo_sync_for_device on AGP cards. Tested-by: Ondrej Zary > --- > drivers/gpu/drm/nouv

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-11 Thread Ondrej Zary
On Friday 11 June 2021 14:38:18 Christian König wrote: > > Am 10.06.21 um 19:59 schrieb Christian König: > > Am 10.06.21 um 19:50 schrieb Ondrej Zary: > >> [SNIP] > >>> I can't see how this is called from the nouveau code, only > >>> possibility

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-10 Thread Ondrej Zary
On Thursday 10 June 2021 08:43:06 Christian König wrote: > > Am 09.06.21 um 22:00 schrieb Ondrej Zary: > > On Wednesday 09 June 2021 11:21:05 Christian König wrote: > >> Am 09.06.21 um 09:10 schrieb Ondrej Zary: > >>> On Wednesday 09 June 2021, Christian König w

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-09 Thread Ondrej Zary
On Wednesday 09 June 2021 11:21:05 Christian König wrote: > Am 09.06.21 um 09:10 schrieb Ondrej Zary: > > On Wednesday 09 June 2021, Christian König wrote: > >> Am 09.06.21 um 08:57 schrieb Ondrej Zary: > >>> [SNIP] > >>>> Thanks for the heads up.

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-09 Thread Ondrej Zary
On Wednesday 09 June 2021, Christian König wrote: > Am 09.06.21 um 08:57 schrieb Ondrej Zary: > > [SNIP] > >> Thanks for the heads up. So the problem with my patch is already fixed, > >> isn't it? > > The NULL pointer derefe

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-08 Thread Ondrej Zary
On Wednesday 09 June 2021, Christian König wrote: > Am 08.06.21 um 23:59 schrieb Ondrej Zary: > > On Tuesday 08 June 2021 22:01:56 Ondrej Zary wrote: > >> On Tuesday 08 June 2021 20:47:42 Ondrej Zary wrote: > >>> On Monday 07 June 2021 22:58:43 Ondrej Zary wrote: &g

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-08 Thread Ondrej Zary
On Tuesday 08 June 2021 22:01:56 Ondrej Zary wrote: > On Tuesday 08 June 2021 20:47:42 Ondrej Zary wrote: > > On Monday 07 June 2021 22:58:43 Ondrej Zary wrote: > > > On Sunday 06 June 2021 23:16:03 Ondrej Zary wrote: > > > > On Saturday 05 June 2021 23:34:23 O

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-08 Thread Ondrej Zary
On Tuesday 08 June 2021 20:47:42 Ondrej Zary wrote: > On Monday 07 June 2021 22:58:43 Ondrej Zary wrote: > > On Sunday 06 June 2021 23:16:03 Ondrej Zary wrote: > > > On Saturday 05 June 2021 23:34:23 Ondrej Zary wrote: > > > > On Saturday 05 June 2021 21:43:52 Ondr

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-08 Thread Ondrej Zary
On Monday 07 June 2021 22:58:43 Ondrej Zary wrote: > On Sunday 06 June 2021 23:16:03 Ondrej Zary wrote: > > On Saturday 05 June 2021 23:34:23 Ondrej Zary wrote: > > > On Saturday 05 June 2021 21:43:52 Ondrej Zary wrote: > > > > Hello, > > > > I'm t

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-07 Thread Ondrej Zary
On Sunday 06 June 2021 23:16:03 Ondrej Zary wrote: > On Saturday 05 June 2021 23:34:23 Ondrej Zary wrote: > > On Saturday 05 June 2021 21:43:52 Ondrej Zary wrote: > > > Hello, > > > I'm testing 5.13.0-rc4 and nouveau crashes with NULL pointer dereference >

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-07 Thread Ondrej Zary
On Saturday 05 June 2021 23:34:23 Ondrej Zary wrote: > On Saturday 05 June 2021 21:43:52 Ondrej Zary wrote: > > Hello, > > I'm testing 5.13.0-rc4 and nouveau crashes with NULL pointer dereference in > > nouveau_bo_sync_for_device. > > Found various report

nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-07 Thread Ondrej Zary
01 5d f0 58 eb ae [ 21.006476] EAX: EBX: 0010 ECX: 0001 EDX: [ 21.006485] ESI: c3e90280 EDI: c185a494 EBP: c2ed7c10 ESP: c2ed7bf8 [ 21.006494] DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0068 EFLAGS: 00210206 [ 21.006503] CR0: 80050033 CR2: CR3: 02ecb000 CR4: 0690 -- Ondrej Zary

Re: nouveau broken on Riva TNT2 in 5.13.0-rc4: NULL pointer dereference in nouveau_bo_sync_for_device

2021-06-07 Thread Ondrej Zary
On Saturday 05 June 2021 21:43:52 Ondrej Zary wrote: > Hello, > I'm testing 5.13.0-rc4 and nouveau crashes with NULL pointer dereference in > nouveau_bo_sync_for_device. > Found various reports like this but that was back in februaryso that should > be fixed now. So it is

Re: [Nouveau] nouveau broken on Riva TNT2 in 5.9.0-rc8: GPU not supported on big-endian

2020-10-29 Thread Ondrej Zary
On Saturday 10 October 2020 02:02:42 Karol Herbst wrote: > 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: > > > > > >

Re: [Nouveau] nouveau broken on Riva TNT2 in 5.9.0-rc8: GPU not supported on big-endian

2020-10-10 Thread Ondrej Zary
On Saturday 10 October 2020 00:23:38 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 stop

nouveau broken on Riva TNT2 in 5.9.0-rc8: GPU not supported on big-endian

2020-10-10 Thread Ondrej Zary
60x64 [ 23.571050] nouveau :01:00.0: fb0: nouveaudrmfb frame buffer device -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: nouveau: System crashes with NVIDIA GeForce 8600 GT

2019-08-19 Thread Ondrej Zary
se nVidia chips are known to fail and corrupt BIOS logo suggests that. -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] video: fbdev: remove dead igafb driver

2017-10-18 Thread Ondrej Zary
t; > It could be that I actually have this particular SPARC framebuffer in > > my hardware collection. > > Unless you have a 32-bit sparc laptop, you don't have a machine that > will use this driver. There are also some x86 PCI cards using this chip. -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Nouveau] [PATCH] [resend] nouveau: Disable AGP for SiS 761

2015-09-30 Thread Ondrej Zary
rry. Didn't notice it was merged. -- Ondrej Zary

[PATCH] [resend] nouveau: Disable AGP for SiS 761

2015-09-30 Thread Ondrej Zary
t the host bridge in nvkm_device_agp_quirks and add entry for SiS 761 with mode 0 (AGP disabled). Signed-off-by: Ondrej Zary --- drivers/gpu/drm/nouveau/nvkm/subdev/pci/agp.c |8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/pci/agp

No more new fbdev drivers, please

2015-09-25 Thread Ondrej Zary
CI_DEVICE_ID_XGI_20 PCI_VENDOR_ID_XGI, PCI_DEVICE_ID_XGI_27 PCI_VENDOR_ID_XGI, PCI_DEVICE_ID_XGI_40 PCI_VENDOR_ID_XGI, PCI_DEVICE_ID_XGI_42 Two of them are already supported by sisfb: PCI_VENDOR_ID_XGI, PCI_DEVICE_ID_XGI_20 PCI_VENDOR_ID_XGI, PCI_DEVICE_ID_XGI_40 So I think that support for the remaining two (and missing features, if any) should be added to sisfb. -- Ondrej Zary

No more new fbdev drivers, please

2015-09-24 Thread Ondrej Zary
h simple drivers doesn't even consider drm > "because I don't have a desktop gpu" which is just silly - drm has become > rather flexible. And that's essentially why writing simple drm drivers > still has a bit too much boilerplate, since no one yet bothered to add a > bit of helper support needed. Is there a simple way to convert existing fbdev drivers to DRM? Let's say I want to convert tridentfb to DRM, keeping the 2D acceleration (pan, fillrect, copyarea, imageblit) to be usable by the console (and maybe extend it to X11 using some generic 2D driver?) -- Ondrej Zary

AGP cards in PCI mode (fake slots like AGPro, AGP Express, AGI, AGX, XGP)

2015-09-15 Thread Ondrej Zary
On Monday 14 September 2015 04:31:43 Alex Deucher wrote: > On Sun, Sep 13, 2015 at 2:57 PM, Ondrej Zary > wrote: > > Hello, > > I have a PC Chips A31G board with AGPro slot and found that nouveau does > > not work properly with it. Console works but reverts to software

AGP cards in PCI mode (fake slots like AGPro, AGP Express, AGI, AGX, XGP)

2015-09-14 Thread Ondrej Zary
On Sunday 13 September 2015 21:12:25 Ilia Mirkin wrote: > On Sun, Sep 13, 2015 at 2:57 PM, Ondrej Zary > wrote: > > Hello, > > I have a PC Chips A31G board with AGPro slot and found that nouveau does > > not work properly with it. Console works but reverts to software

AGP cards in PCI mode (fake slots like AGPro, AGP Express, AGI, AGX, XGP)

2015-09-13 Thread Ondrej Zary
Hello, I have a PC Chips A31G board with AGPro slot and found that nouveau does not work properly with it. Console works but reverts to software mode, X11 hangs with mouse cursor only. The slot is physically AGP 1.5V but is wired to PCI bus as the chipset (SiS 761) does not support AGP cards. To f

[PATCH] Another card with wrong primary dac adj

2013-07-20 Thread Ondrej Zary
On Friday 19 July 2013 23:50:50 Alex Deucher wrote: > On Fri, Jul 19, 2013 at 3:08 PM, Ondrej Zary wrote: > > Hello, > > got another card with "too bright" problem: > > Sapphire Radeon VE 7000 DDR (VGA+S-Video) > > > > lspci -vnn: > > 01:00.0

[PATCH] Another card with wrong primary dac adj

2013-07-19 Thread Ondrej Zary
different types are broken this way. The "wrong" ps2_pdac_adj value that comes from BIOS on this card is 0x300. drm/radeon: Add primary dac adj quirk for Sapphire Radeon VE 7000 DDR Values from BIOS are wrong, causing too bright colors. Use default values instea

Re: [PATCH] Another card with wrong primary dac adj

2013-07-19 Thread Ondrej Zary
On Friday 19 July 2013 23:50:50 Alex Deucher wrote: > On Fri, Jul 19, 2013 at 3:08 PM, Ondrej Zary wrote: > > Hello, > > got another card with "too bright" problem: > > Sapphire Radeon VE 7000 DDR (VGA+S-Video) > > > > lspci -vnn: > > 01:00.0

[PATCH] Another card with wrong primary dac adj

2013-07-19 Thread Ondrej Zary
different types are broken this way. The "wrong" ps2_pdac_adj value that comes from BIOS on this card is 0x300. drm/radeon: Add primary dac adj quirk for Sapphire Radeon VE 7000 DDR Values from BIOS are wrong, causing too bright colors. Use default values instea

Future desktop on dumb frame buffers?

2011-03-21 Thread Ondrej Zary
2D acceleration. > This is important to me, as the various old drivers I've been hacking > on won't be accepted upstream without some sort of userspace which can > work with them. One of the big goals of KMS was a generic > userspace-facing API, like FB, but without the suck. -- Ondrej Zary

Re: Future desktop on dumb frame buffers?

2011-03-21 Thread Ondrej Zary
2D acceleration. > This is important to me, as the various old drivers I've been hacking > on won't be accepted upstream without some sort of userspace which can > work with them. One of the big goals of KMS was a generic > userspace-facing API, like FB, but without the suck. -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Regression, post-2.6.34] Hibernation broken on machines with radeon/KMS and r300

2010-06-16 Thread Ondrej Zary
o, it doesn't. I try to hibernate, everything works to the point where > > the screen goes off and the box hangs (solid). Normally, it would turn > > the screen back on and continue with saving the image. > > > > But, since that happens with the patch above applied, I think it doesn't > > really pass the suspend phase (IOW, it probably hangs somewhere in the > > radeon's suspend routine). > > I've just verified that in fact hibernation works on HP nx6325 with > 2.6.35-rc3, but it takes about 55 sec. to suspend the graphics adapter in > the "freeze" phase. Surprisingly enough, during suspend to RAM it works > normally (as well as in the "poweroff" phase of hibernation). It takes 2 minutes on RV530: https://bugzilla.redhat.com/show_bug.cgi?id=586522 -- Ondrej Zary

Re: [Regression, post-2.6.34] Hibernation broken on machines with radeon/KMS and r300

2010-06-15 Thread Ondrej Zary
o, it doesn't. I try to hibernate, everything works to the point where > > the screen goes off and the box hangs (solid). Normally, it would turn > > the screen back on and continue with saving the image. > > > > But, since that happens with the patch above applied, I think it doesn't > > really pass the suspend phase (IOW, it probably hangs somewhere in the > > radeon's suspend routine). > > I've just verified that in fact hibernation works on HP nx6325 with > 2.6.35-rc3, but it takes about 55 sec. to suspend the graphics adapter in > the "freeze" phase. Surprisingly enough, during suspend to RAM it works > normally (as well as in the "poweroff" phase of hibernation). It takes 2 minutes on RV530: https://bugzilla.redhat.com/show_bug.cgi?id=586522 -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-06 Thread Ondrej Zary
On Sunday 06 June 2010 11:04:44 Dave Airlie wrote: > On Sun, Jun 6, 2010 at 6:28 AM, Ondrej Zary wrote: > > On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > >> On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary wrote: > >> > Hello, > >> > I

Re: regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-06 Thread Ondrej Zary
On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary wrote: > > Hello, > > I'm testing 2.6.35-rc1 kernel on Asus P4P800-VM (i865G chipset). After > > loading i915 module, the screen goes blank and the kernel hangs > &

Re: regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-06 Thread Ondrej Zary
On Sunday 06 June 2010 11:04:44 Dave Airlie wrote: > On Sun, Jun 6, 2010 at 6:28 AM, Ondrej Zary wrote: > > On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > >> On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary wrote: > >> > Hello, > >> > I

regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-05 Thread Ondrej Zary
On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary rainbow-software.org> wrote: > > Hello, > > I'm testing 2.6.35-rc1 kernel on Asus P4P800-VM (i865G chipset). After > > loading i915 module, the screen go

regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-05 Thread Ondrej Zary
On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary rainbow-software.org> wrote: > > Hello, > > I'm testing 2.6.35-rc1 kernel on Asus P4P800-VM (i865G chipset). After > > loading i915 module, the screen go

Re: regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-05 Thread Ondrej Zary
On Saturday 05 June 2010 02:23:27 Eric Anholt wrote: > On Fri, 4 Jun 2010 22:01:28 +0200, Ondrej Zary > wrote: > > Hello, > > I'm testing 2.6.35-rc1 kernel on Asus P4P800-VM (i865G chipset). After > > loading i915 module, the screen goes blank and the kernel ha

regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-04 Thread Ondrej Zary
Is this a known regression? -- Ondrej Zary

regression: 2.6.35-rc1 hangs on i865G with KMS

2010-06-04 Thread Ondrej Zary
Is this a known regression? -- Ondrej Zary ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel