All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ernst Herzberg <list-lkml@net4u.de>
To: linux-kernel@vger.kernel.org
Subject: 2.6.16-rc3 macromedia flash regression...
Date: Fri, 17 Feb 2006 05:08:52 +0100	[thread overview]
Message-ID: <200602170508.52712.list-lkml@net4u.de> (raw)


.... or not regession, that's the question.

2.6.16-rc2 works without problems.

With -rc3 a .swf that opens a ip connection back to the server takes ages to 
load. strace shows that the player hangs for long times in select().

Digging through the changelog brings up

commit 643a654540579b0dcc7a206a4a7475276a41aff0
Author: Andrew Morton <akpm@osdl.org>
Date:   Sat Feb 11 17:55:52 2006 -0800

    [PATCH] select: fix returned timeval


Reverting the patch partial like

--- lx-2.6.16-rc3/fs/select.c   2006-02-17 04:35:10.000000000 +0100
+++ lx-2.6.16-rc3.old/fs/select.c       2006-02-17 04:44:00.000000000 +0100
@@ -404,8 +404,6 @@ asmlinkage long sys_select(int n, fd_set
                        goto sticky;
                rtv.tv_usec = jiffies_to_usecs(do_div((*(u64*)&timeout), HZ));
                rtv.tv_sec = timeout;
-               if (timeval_compare(&rtv, &tv) < 0)
-                       rtv = tv;
                if (copy_to_user(tvp, &rtv, sizeof(rtv))) {
 sticky:
                        /*
@@ -471,8 +469,6 @@ asmlinkage long sys_pselect7(int n, fd_s
                rts.tv_nsec = jiffies_to_usecs(do_div((*(u64*)&timeout), HZ)) 
*
                                                1000;
                rts.tv_sec = timeout;
-               if (timespec_compare(&rts, &ts) < 0)
-                       rts = ts;
                if (copy_to_user(tsp, &rts, sizeof(rts))) {
 sticky:
                        /*

----------------------

fixed that, the player works again in normal speed.

The player is the newest from macromedia (7.0.61.0, gentoo ebuild), binary 
only of course:( Reading the patch and the documentation of select() i think 
the player is buggy... but works until 2.6.16-rc3

So what is your opinion? I think, macromedia should fix the player... 

Thanks 

<earny/>

             reply	other threads:[~2006-02-17  4:08 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-17  4:08 Ernst Herzberg [this message]
2006-02-17  7:10 ` 2.6.16-rc3 macromedia flash regression Andrew Morton
2006-02-17 19:37   ` Ernst Herzberg
2006-02-17  7:23 ` Andrew Morton
2006-02-17 19:38   ` Ernst Herzberg
2006-02-17 21:16     ` Andrew Morton
2006-02-17 22:07       ` Ernst Herzberg
2006-02-17 22:13         ` Andrew Morton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200602170508.52712.list-lkml@net4u.de \
    --to=list-lkml@net4u.de \
    --cc=earny@net4u.de \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.