Bug 1877 - neomagic regression in 6.8.1: very slow scrolling with xterm under X11
Summary: neomagic regression in 6.8.1: very slow scrolling with xterm under X11
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/Neomagic (show other bugs)
Version: 6.8.1
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Egbert Eich
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-22 00:05 UTC by Igor Bukanov
Modified: 2004-12-15 21:39 UTC (History)
4 users (show)

See Also:
i915 platform:
i915 features:


Attachments
undo 6.7.0 (9.37 KB, patch)
2004-11-23 12:55 UTC, Valery Inozemtsev
no flags Details | Splinter Review
xorg-x11-6.8.0-alt-neomagic (1.07 KB, patch)
2004-11-29 03:08 UTC, Valery Inozemtsev
no flags Details | Splinter Review
Fix (17.55 KB, patch)
2004-12-06 07:23 UTC, Egbert Eich
no flags Details | Splinter Review
[FIXED_X11R68x] Fix (update) (17.60 KB, patch)
2004-12-06 07:41 UTC, Egbert Eich
roland.mainz: 6.8-branch+
Details | Splinter Review

Description Igor Bukanov 2004-11-22 00:05:56 UTC
This was originally reported at
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=139175 :

xterm in Fedora Core 3 scrolls very slowly and it seems the only reason is
changes in neomagic driver as replacing neomagic_drv.o with the version from
Xorg 6.7.0 restored xtrem speed.

My video hardware on Sony Vaio PCG-Z505R notebook with 366 MHz Pentiun
II CPU:

NeoMagic MagicMedia 256AV (laptop/notebook)
Comment 1 Dag Nygren 2004-11-23 11:00:43 UTC
I can confirm this on a Dell Latitude Cpi Neomagic installation. 
It is not only the xterm scrolling, but all text scrolling applications. 
For example my exmh is very badly affected. 
Comment 2 Valery Inozemtsev 2004-11-23 12:50:28 UTC
reproduce on IBM TP570
Comment 3 Valery Inozemtsev 2004-11-23 12:55:29 UTC
Created attachment 1362 [details] [review]
undo 6.7.0
Comment 4 Alex Deucher 2004-11-24 20:16:31 UTC
this patch seems a little drastic.  the changes to neo_video.c for instance
shouldn't have any affect on your chip.  If possible, could you track down what
exact change causes the problem?  it shouldn't be too hard since the changes are
pretty minimal.
Comment 5 Egbert Eich 2004-11-25 07:13:16 UTC
The patch is definitely not the solution. I'm the maintainer of this driver and
I'm going to look at this as soon as I'm organized again.
Comment 6 Valery Inozemtsev 2004-11-29 03:08:42 UTC
Created attachment 1414 [details] [review]
xorg-x11-6.8.0-alt-neomagic
Comment 7 Valery Inozemtsev 2004-11-29 03:11:15 UTC
this problem is fixed by this patch
Comment 8 Egbert Eich 2004-11-30 06:41:35 UTC
Your patch backs out the XAA wrapper that is necessary for the pseudocolor
emulation. I've discovered some problems with the wrapper today which I will
address ASAP.
Comment 9 Egbert Eich 2004-12-06 07:23:23 UTC
Created attachment 1480 [details] [review]
Fix

This patch should fix the issues with the neomaic driver.
It also contains two fixes that are yet missing from the tree.
A large part of this fix backs out some #if 0'ed stuff.
Comment 10 Egbert Eich 2004-12-06 07:41:03 UTC
Created attachment 1481 [details] [review]
[FIXED_X11R68x] Fix (update)

This overrides the previous patch which was created from a slightly wrong
version.
Comment 11 Egbert Eich 2004-12-06 07:54:53 UTC
This has been committed to head.
Please reopen when you still see problems.
Comment 12 Igor Bukanov 2004-12-06 13:29:02 UTC
Is it possible to get a compiled driver for testing? 
Comment 13 Jim Gettys 2004-12-08 08:29:34 UTC
Keith, can you review this patch please, in particular the composite change, for
the sake of the maintenance release?
Comment 14 Roland Mainz 2004-12-11 15:29:03 UTC
Keith:
ping!
Comment 15 Keith Packard 2004-12-13 11:38:41 UTC
The change to fbwindow.c shouldn't be necessary; we always build with COMPOSITE
support.  That said, I think the COMPOSITE code in that case is probably correct
even without COMPOSITE defined.

As for the rest of the changes, I don't have any idea what xaaWrapper is for.
Comment 16 Roland Mainz 2004-12-16 16:16:16 UTC
Comment on attachment 1481 [details] [review]
[FIXED_X11R68x] Fix (update)

Approval for X11R6.8.x stable branch granted in the 2004-12-13
release-wranglers phone call.
Comment 17 Roland Mainz 2004-12-16 16:38:59 UTC
Comment on attachment 1481 [details] [review]
[FIXED_X11R68x] Fix (update)

Patch checked-in into X11R6.8.x stable branch:

/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.365.2.102; previous revision: 1.365.2.101
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
/cvs/xorg/xc/programs/Xserver/fb/fbwindow.c,v  <--  fbwindow.c
new revision: 1.4.2.1; previous revision: 1.4
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
/cvs/xorg/xc/programs/Xserver/hw/xfree86/xaa/xaaWrapper.c,v  <--  xaaWrapper.c
new revision: 1.1.2.1; previous revision: 1.1
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
/cvs/xorg/xc/programs/Xserver/mi/micmap.c,v  <--  micmap.c
new revision: 1.3.4.1; previous revision: 1.3
cvs commit: Using deprecated info format strings.  Convert your scripts to use
the new argument format and remove '1's from your info file format strings.
Mailing the commit message to xorg-commit@lists.freedesktop.org...


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.