New issue
Advanced search Search tips

Issue 667822 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

should we run sshd at higher priority on DUTs and devservers?

Project Member Reported by semenzato@chromium.org, Nov 22 2016

Issue description

We see occasional test failures in the lab from ssh errors, some of which may be due to timeouts.  I am pretty sure that most of these timeouts are NOT due to poor scheduling of sshd.  But on the other hand, sshd is always working on behalf of some other process, and rarely doing long operations on its own (I am not sure about some of the key verifications, but I suspect that they are dominated by network latency).  Thus I don't see any contraindications to giving sshd some negative nice (i.e. some mean), and it may have benefits.
 

Comment 1 by vapier@chromium.org, Nov 22 2016

where does it end ?  if the DUT is choking, then ssh establishing the connection won't help with spawning of the shell ?  you'll still time out then ?

making sshd run all the time with higher priority should be easy to do, and it's not like it'd hurt if sshd was idle.
Right.  It doesn't hurt, but likely doesn't help either.

The special case that made me think of this is the errors we seem to get when we run

ssh <DUT> "touch <some-file>; reboot"

It *seems* that in a number of cases the ssh command fails.

In some experiment, I can make ssh fail (i.e. hang) with 

ssh <DUT> "reboot; sleep <T>"

with T = 0.1s.  It works with T = 0.05s.  So it's tight, and I am wondering if there are other situations in which we don't sufficiently prioritize sshd.

However:

1. It's not clear nice would help because there are other operations, like flushing dirty buffers to disk, which also compete

2. There are other ways of dealing this particular case.

So maybe it's not a good idea.  I enjoy using the bug tracker for my streams of consciousness.

Status: WontFix (was: Untriaged)
Wouldn't help.

Sign in to add a comment