Re: tty update speed regression (was: 2.6.14-rc2-mm1)

[Nish Aravamudan <nish.aravamudan@gmail.com>]

On 9/22/05, Nishanth Aravamudan <nacc@us.ibm.com> wrote:
> On 23.09.2005 [01:49:26 +0400], Alexey Dobriyan wrote:
> > On Thu, Sep 22, 2005 at 11:50:29PM +0400, Alexey Dobriyan wrote:
> > > I see regression in tty update speed with ADOM (ncurses based
> > > roguelike) [1].
> > >
> > > Messages at the top ("goblin hits you") are printed slowly. An eye can
> > > notice letter after letter printing.
> > >
> > > 2.6.14-rc2 is OK.
> > >
> > > I'll try to revert tty-layer-buffering-revamp*.patch pieces and see if
> > > it'll change something.
> > >
> > > [1] http://adom.de/adom/download/linux/adom-111-elf.tar.gz (binary only)
> >
> > Scratch TTY revamp, the sucker is
> > fix-sys_poll-large-timeout-handling.patch
> >
> > HZ=250 here.
>
> Alexey,
>
> Thanks for the report. I will take a look on my Thinkpad with HZ=250
> under -mm2. I have some ideas for debugging it if I see the same
> problem.

I did not see any tty refresh problems on my TP with HZ=250 under
2.6.14-rc2-mm1 (excuse the typo in my previous response) under the
adom binary you sent me. I even played two games just to make sure ;)

Is there any chance you can do an strace of the process while it is
slow to redraw your screen? Just to verify how poll() is being called
[if my patch is the problem, then poll() must be being used somewhat
differently than I expected -- e.g. a dependency on the broken
behavior]. The only thing I can think of right now is that I made
timeout_jiffies unsigned, when schedule_timeout() will treat it as
signed, but I'm not sure if that is the problem.

We may want to contact the adom author eventually to figure out how
poll() is being used in the Linux port, if strace is unable to help
further.

Thanks,
Nish
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/