Project

General

Profile

Actions
Copy link

action #17092

closed

100% cpu usage in virtio_screen.pm function read_until

Added by nadvornik about 7 years ago. Updated about 7 years ago.

Status:
Resolved
Priority:
High
Assignee:
rpalethorpe
Category:
Regressions/Crashes
Target version:
Milestone 6
Start date:
2017-02-15
Due date:
% Done:

0%

Estimated time:

Description

On my machine isotovideo runs with 100% cpu usage in virtio_screen.pm function read_until.

strace shows that it cycles on this:
read(21, 0x6670db0, 2048) = -1 EAGAIN (Resource temporarily unavailable)

Adding usleep(1000) before the retry on EAGAIN (virtio_screen.pm line 197) helped, but maybe there is a better fix.

Actions
Copy link
 #1

Updated by rpalethorpe about 7 years ago

  • Priority changed from Normal to High

I guess we can set the socket to blocking and set a timer to signal a timeout or similar, I will do some research, it could be improved anyway.

Actions
Copy link
 #2

Updated by rpalethorpe about 7 years ago

  • Status changed from New to Feedback

I have made a patch which blocks using select. https://github.com/os-autoinst/os-autoinst/pull/724

Actions
Copy link
 #3

Updated by okurz about 7 years ago

  • Category set to Regressions/Crashes
  • Target version set to Milestone 6
Actions
Copy link
 #4

Updated by okurz about 7 years ago

  • Status changed from Feedback to Resolved

merged, nadvornik proved it's working for him.

Actions
Copy link

Also available in: Atom PDF