* [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile
@ 2021-10-13 1:14 Yang Xu
2021-10-13 1:14 ` [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run Yang Xu
2021-10-13 4:19 ` [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Petr Vorel
0 siblings, 2 replies; 9+ messages in thread
From: Yang Xu @ 2021-10-13 1:14 UTC (permalink / raw)
To: ltp
Fixes: f8fb073708 ("kernel/irq: Add irqbalance01")
Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
---
testcases/kernel/Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git a/testcases/kernel/Makefile b/testcases/kernel/Makefile
index f3a608f38..4604f1f38 100644
--- a/testcases/kernel/Makefile
+++ b/testcases/kernel/Makefile
@@ -28,6 +28,7 @@ SUBDIRS += connectors \
input \
io \
ipc \
+ irq \
logging \
mem \
numa \
--
2.23.0
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 1:14 [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Yang Xu
@ 2021-10-13 1:14 ` Yang Xu
2021-10-13 4:24 ` Petr Vorel
2021-10-13 6:54 ` Richard Palethorpe
2021-10-13 4:19 ` [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Petr Vorel
1 sibling, 2 replies; 9+ messages in thread
From: Yang Xu @ 2021-10-13 1:14 UTC (permalink / raw)
To: ltp
Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
---
scenario_groups/default | 1 +
1 file changed, 1 insertion(+)
diff --git a/scenario_groups/default b/scenario_groups/default
index 439783dac..1dc2987d5 100644
--- a/scenario_groups/default
+++ b/scenario_groups/default
@@ -6,6 +6,7 @@ dio
io
mm
ipc
+irq
sched
math
nptl
--
2.23.0
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile
2021-10-13 1:14 [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Yang Xu
2021-10-13 1:14 ` [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run Yang Xu
@ 2021-10-13 4:19 ` Petr Vorel
1 sibling, 0 replies; 9+ messages in thread
From: Petr Vorel @ 2021-10-13 4:19 UTC (permalink / raw)
To: Yang Xu; +Cc: ltp
Hi Xu,
obviously correct, thus pushed.
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 1:14 ` [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run Yang Xu
@ 2021-10-13 4:24 ` Petr Vorel
2021-10-13 6:54 ` Richard Palethorpe
1 sibling, 0 replies; 9+ messages in thread
From: Petr Vorel @ 2021-10-13 4:24 UTC (permalink / raw)
To: Yang Xu; +Cc: Richard Palethorpe, ltp
Hi,
yes, it should go to default scenario group as well.
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Kind regards,
Petr
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 1:14 ` [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run Yang Xu
2021-10-13 4:24 ` Petr Vorel
@ 2021-10-13 6:54 ` Richard Palethorpe
2021-10-13 8:15 ` xuyang2018.jy
1 sibling, 1 reply; 9+ messages in thread
From: Richard Palethorpe @ 2021-10-13 6:54 UTC (permalink / raw)
To: Yang Xu; +Cc: ltp
Hello,
Yang Xu <xuyang2018.jy@fujitsu.com> writes:
> Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
> ---
> scenario_groups/default | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scenario_groups/default b/scenario_groups/default
> index 439783dac..1dc2987d5 100644
> --- a/scenario_groups/default
> +++ b/scenario_groups/default
> @@ -6,6 +6,7 @@ dio
> io
> mm
> ipc
> +irq
> sched
> math
> nptl
> --
> 2.23.0
I'm not sure this should go in the default group at this time. The only
test in irq (irqbalance), only works on some configurations. As
discussed in the test review, the user must figure out if they should
run it or not.
Perhaps we need a new scenario group?
--
Thank you,
Richard.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 6:54 ` Richard Palethorpe
@ 2021-10-13 8:15 ` xuyang2018.jy
2021-10-13 10:10 ` Cyril Hrubis
0 siblings, 1 reply; 9+ messages in thread
From: xuyang2018.jy @ 2021-10-13 8:15 UTC (permalink / raw)
To: rpalethorpe; +Cc: ltp
Hi Richard
> Hello,
>
> Yang Xu<xuyang2018.jy@fujitsu.com> writes:
>
>> Signed-off-by: Yang Xu<xuyang2018.jy@fujitsu.com>
>> ---
>> scenario_groups/default | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/scenario_groups/default b/scenario_groups/default
>> index 439783dac..1dc2987d5 100644
>> --- a/scenario_groups/default
>> +++ b/scenario_groups/default
>> @@ -6,6 +6,7 @@ dio
>> io
>> mm
>> ipc
>> +irq
>> sched
>> math
>> nptl
>> --
>> 2.23.0
>
> I'm not sure this should go in the default group at this time. The only
> test in irq (irqbalance), only works on some configurations. As
> discussed in the test review, the user must figure out if they should
> run it or not.
It is hard to say moving this case into default run at this time is good
or bad.
With an optimistic attitude, I want to add it into default run(add some
comment in irqbalance01.c that irqbalance01 may fail because it needs
some configuration in service or hardware)and then listen whether many
users complain about this failure.
ps: Many people still use runltp to test ltp instead of runltp-ng and
they usually only run default group. That is a important reason that I
want to add this case into default group.
>
> Perhaps we need a new scenario group?
I don't have objection. So do you want to cover what scenario(advanced
or experimental scenario group)?
Best Regards
Yang Xu
>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 8:15 ` xuyang2018.jy
@ 2021-10-13 10:10 ` Cyril Hrubis
2021-10-14 0:56 ` xuyang2018.jy
0 siblings, 1 reply; 9+ messages in thread
From: Cyril Hrubis @ 2021-10-13 10:10 UTC (permalink / raw)
To: xuyang2018.jy; +Cc: ltp
Hi!
> >> Signed-off-by: Yang Xu<xuyang2018.jy@fujitsu.com>
> >> ---
> >> scenario_groups/default | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/scenario_groups/default b/scenario_groups/default
> >> index 439783dac..1dc2987d5 100644
> >> --- a/scenario_groups/default
> >> +++ b/scenario_groups/default
> >> @@ -6,6 +6,7 @@ dio
> >> io
> >> mm
> >> ipc
> >> +irq
> >> sched
> >> math
> >> nptl
> >> --
> >> 2.23.0
> >
> > I'm not sure this should go in the default group at this time. The only
> > test in irq (irqbalance), only works on some configurations. As
> > discussed in the test review, the user must figure out if they should
> > run it or not.
>
> It is hard to say moving this case into default run at this time is good
> or bad.
>
> With an optimistic attitude, I want to add it into default run(add some
> comment in irqbalance01.c that irqbalance01 may fail because it needs
> some configuration in service or hardware)and then listen whether many
> users complain about this failure.
>
> ps: Many people still use runltp to test ltp instead of runltp-ng and
> they usually only run default group. That is a important reason that I
> want to add this case into default group.
Well there are two wrong choices.
If we add it to the default scenario people will complain that the test
fails for no good reason.
If we do not, the test will be largerly unused and probably bitrot over
the time.
However if majority here things that we should enable it by default, we
can try that and revert it if we got too many complaints.
--
Cyril Hrubis
chrubis@suse.cz
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-13 10:10 ` Cyril Hrubis
@ 2021-10-14 0:56 ` xuyang2018.jy
2021-11-24 10:33 ` Petr Vorel
0 siblings, 1 reply; 9+ messages in thread
From: xuyang2018.jy @ 2021-10-14 0:56 UTC (permalink / raw)
To: Cyril Hrubis; +Cc: ltp
Hi Cyril
> Hi!
>>>> Signed-off-by: Yang Xu<xuyang2018.jy@fujitsu.com>
>>>> ---
>>>> scenario_groups/default | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/scenario_groups/default b/scenario_groups/default
>>>> index 439783dac..1dc2987d5 100644
>>>> --- a/scenario_groups/default
>>>> +++ b/scenario_groups/default
>>>> @@ -6,6 +6,7 @@ dio
>>>> io
>>>> mm
>>>> ipc
>>>> +irq
>>>> sched
>>>> math
>>>> nptl
>>>> --
>>>> 2.23.0
>>>
>>> I'm not sure this should go in the default group at this time. The only
>>> test in irq (irqbalance), only works on some configurations. As
>>> discussed in the test review, the user must figure out if they should
>>> run it or not.
>>
>> It is hard to say moving this case into default run at this time is good
>> or bad.
>>
>> With an optimistic attitude, I want to add it into default run(add some
>> comment in irqbalance01.c that irqbalance01 may fail because it needs
>> some configuration in service or hardware)and then listen whether many
>> users complain about this failure.
>>
>> ps: Many people still use runltp to test ltp instead of runltp-ng and
>> they usually only run default group. That is a important reason that I
>> want to add this case into default group.
>
> Well there are two wrong choices.
>
> If we add it to the default scenario people will complain that the test
> fails for no good reason.
>
> If we do not, the test will be largerly unused and probably bitrot over
> the time.
>
> However if majority here things that we should enable it by default, we
> can try that and revert it if we got too many complaints.
Yes, that's what I was thinking too.
Best Regards
Yang Xu
>
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run
2021-10-14 0:56 ` xuyang2018.jy
@ 2021-11-24 10:33 ` Petr Vorel
0 siblings, 0 replies; 9+ messages in thread
From: Petr Vorel @ 2021-11-24 10:33 UTC (permalink / raw)
To: xuyang2018.jy; +Cc: ltp, Richard Palethorpe
Hi all,
> Hi Cyril
> > Hi!
> >>>> Signed-off-by: Yang Xu<xuyang2018.jy@fujitsu.com>
> >>>> ---
> >>>> scenario_groups/default | 1 +
> >>>> 1 file changed, 1 insertion(+)
> >>>> diff --git a/scenario_groups/default b/scenario_groups/default
> >>>> index 439783dac..1dc2987d5 100644
> >>>> --- a/scenario_groups/default
> >>>> +++ b/scenario_groups/default
> >>>> @@ -6,6 +6,7 @@ dio
> >>>> io
> >>>> mm
> >>>> ipc
> >>>> +irq
> >>>> sched
> >>>> math
> >>>> nptl
> >>>> --
> >>>> 2.23.0
> >>> I'm not sure this should go in the default group at this time. The only
> >>> test in irq (irqbalance), only works on some configurations. As
> >>> discussed in the test review, the user must figure out if they should
> >>> run it or not.
> >> It is hard to say moving this case into default run at this time is good
> >> or bad.
> >> With an optimistic attitude, I want to add it into default run(add some
> >> comment in irqbalance01.c that irqbalance01 may fail because it needs
> >> some configuration in service or hardware)and then listen whether many
> >> users complain about this failure.
> >> ps: Many people still use runltp to test ltp instead of runltp-ng and
> >> they usually only run default group. That is a important reason that I
> >> want to add this case into default group.
> > Well there are two wrong choices.
> > If we add it to the default scenario people will complain that the test
> > fails for no good reason.
> > If we do not, the test will be largerly unused and probably bitrot over
> > the time.
> > However if majority here things that we should enable it by default, we
> > can try that and revert it if we got too many complaints.
> Yes, that's what I was thinking too.
OK, merged, let's see if it's ok to run for majority (we can always revert it).
Kind regards,
Petr
> Best Regards
> Yang Xu
--
Mailing list info: https://lists.linux.it/listinfo/ltp
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-11-24 10:33 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-13 1:14 [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Yang Xu
2021-10-13 1:14 ` [LTP] [PATCH 2/2] scenario_groups/default: Add irq into default run Yang Xu
2021-10-13 4:24 ` Petr Vorel
2021-10-13 6:54 ` Richard Palethorpe
2021-10-13 8:15 ` xuyang2018.jy
2021-10-13 10:10 ` Cyril Hrubis
2021-10-14 0:56 ` xuyang2018.jy
2021-11-24 10:33 ` Petr Vorel
2021-10-13 4:19 ` [LTP] [PATCH 1/2] kernel/Makefile: Add irq to Makefile Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.