[Fuego] [PATCH v2] Benchmark.cyclictest: add a warning for rt

Tim.Bird at sony.com Tim.Bird at sony.com
Fri Nov 9 01:25:17 UTC 2018



> -----Original Message-----
> From: Daniel Sangorrin 
> Hi Tim,
> 
> > -----Original Message-----
> > From: Tim Bird
> >
> > > -----Original Message-----
> > > From: Li Xiaoming on  Thursday, November 01, 2018 8:35 PM
> > >
> > > details: add a warning if CONFIG_RT_GROUP_SCHED enabled in kernel.
> > >
> > > Signed-off-by: Li Xiaoming <lixm.fnst at cn.fujitsu.com>
> > > ---
> > >  engine/tests/Benchmark.cyclictest/fuego_test.sh | 8 ++++++++
> > >  1 file changed, 8 insertions(+)
> > >
> > > diff --git a/engine/tests/Benchmark.cyclictest/fuego_test.sh
> > > b/engine/tests/Benchmark.cyclictest/fuego_test.sh
> > > index 319f361..d233b87 100755
> > > --- a/engine/tests/Benchmark.cyclictest/fuego_test.sh
> > > +++ b/engine/tests/Benchmark.cyclictest/fuego_test.sh
> > > @@ -4,6 +4,14 @@ NEED_ROOT=1
> > >
> > >  function test_pre_check {
> > >      assert_define BENCHMARK_CYCLICTEST_PARAMS
> > > +
> > > +    if check_kconfig "CONFIG_RT_GROUP_SCHED=y"; then
> > > +        echo "WARNING: CONFIG_RT_GROUP_SCHED enabled in your
> kernel.
> > > Please check the RT"
> > > +        echo "settings as following if this test failed with 'Unable to change
> > > scheduling policy'."
> > > +        echo "- If the user does not need the RT groups functionality,
> disable
> > > CONFIG_RT_GROUP_SCHED and compile the kernel again, or 'sysctl -w
> > > kernel.sched_rt_runtime_us=-1'"
> > > +        echo "- If the user does want to use RT groups functionality, put the
> > > sshd daemon into a cgroup with assigned runtime"
> >
> > This last bullet point seems to be missing a word.  I don't understand it.
> >
> > Can you please describe more what it means to "put the ssh daemon into a
> cgroup
> > with
> > assigned runtime"?
> 
> Sorry about that. How about using this wording?
> 
> "Assign the ssh daemon service a realtime budget by specifying the
> cpu.rt_runtime_us attribute."

I changed the wording to something close to this.

Thanks,
 -- Tim



More information about the Fuego mailing list