public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] syscalls/getpriority01: exclude default priority check for PRIO_USER.
Date: Tue, 22 Nov 2016 10:37:42 +0100	[thread overview]
Message-ID: <20161122093742.GD7048@rei.lan> (raw)
In-Reply-To: <58341129.4000003@cn.fujitsu.com>

Hi!
> >> The current user priority is various when testing,
> >> we should not check it as default value.
> > 
> > What system does this happen on? It should probably be mentioned in the
> > commit message.
> 
> It happens on Fedora20, RHEL5.11GA, RHEL6.8GA, RNEL7.3GA, testing returns
> -20 and -11 for PRIO_USER as root and nobody respectively.

Hmm, are you sure that this is not a bug? Since -20 is the highest
priority for a process, it does not make much sense to run all root
processes like that.

Also the nice value gets inherited via fork, so it may be that if you
are connecting to the machine via ssh the ssh process gets priority so
that the machine stays responsive even under load, which is inherited by
the shell and then inherited by the test. But even then the -20 sounds
a bit too much.

-- 
Cyril Hrubis
chrubis@suse.cz

  reply	other threads:[~2016-11-22  9:37 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-17  6:02 [LTP] [PATCH] syscalls/getpriority01: exclude default priority check for PRIO_USER Guangwen Feng
2016-11-22  8:56 ` Cyril Hrubis
2016-11-22  9:34   ` Guangwen Feng
2016-11-22  9:37     ` Cyril Hrubis [this message]
2016-11-23  6:26       ` Guangwen Feng
2016-11-23 15:04         ` Cyril Hrubis
2016-11-24  7:02           ` Guangwen Feng

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=20161122093742.GD7048@rei.lan \
    --to=chrubis@suse.cz \
    --cc=ltp@lists.linux.it \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox