bugGNU Screen - Bugs: bug #57697, 4.99.0: CPU usage always 100%...

 
 

bug #57697: 4.99.0: CPU usage always 100% after the commit 8becc4d (Convert select() to poll() in sched.c)

Submitter:  Koichi Murase <akinomyoga>
Submitted:  Thu 30 Jan 2020 02:46:45 AM UTC
   
 
Category:  Crash/Freeze/Infloop Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Release:  4.99.0
Fixed Release:  None Planned Release:  5.0.0
Work Required:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 04 Feb 2020 12:01:01 AM UTC, comment #3: 

Thank you. I tried the master branch and confirmed that the problem is fixed.

Koichi Murase <akinomyoga>
Mon 03 Feb 2020 10:47:00 PM UTC, comment #2: 

Fix is present in master branch.

Amadeusz Sławiński <amade>
Group administrator
Thu 30 Jan 2020 11:13:58 PM UTC, comment #1: 

Oh... interesing, if I run:
./screen -c /dev/null
it reproduces.
I must have something in my config which stops it from happening.

Amadeusz Sławiński <amade>
Group administrator
Thu 30 Jan 2020 02:46:45 AM UTC, original submission:  

Hello, I'm trying the master branch (screen-4.99.0) for 24-bit color
support, but I noticed that the CPU usage of the main screen process
is always 100%.  I confirmed this with two Linux hosts:

  $ uname -a
  Linux song123 5.3.0-26-generic #28~18.04.1-Ubuntu SMP Wed Dec 18 16:40:14 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
  $ uname -a
  Linux chatoyancy 5.1.20-300.fc30.x86_64 #1 SMP Fri Jul 26 15:03:11 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

I checked recent commits in the git repository and found that this
started to happen after the following commit:

  --------------------------------------------------------------------
  commit 8becc4d29a12688b39312639f56e1aeeb76f96e2 (HEAD)
  Author: Amadeusz Sławiński <amade@asmblr.net>
  Date:   Sun Nov 3 00:31:59 2019 +0100

      Convert select() to poll() in sched.c

      select() limits number of file descriptors that can be used by screen.
      Migrate to poll() to avoid this limitation.

      As can be seen in case of scheduler it requires quite some changes, care
      must be taken to count poll() events properly.

      Bug: 55697

      Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
  --------------------------------------------------------------------

Koichi Murase <akinomyoga>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by amade (Posted a comment)
  • -email is unavailable- added by akinomyoga (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-02-03 amade StatusConfirmed Fixed
        Open/ClosedOpen Closed
        Planned ReleaseNone 5.0.0
    2020-01-30 amade StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code