diff mbox series

[1/1] doc: Document .min_cpus

Message ID 20201207145639.8424-1-pvorel@suse.cz
State Accepted
Headers show
Series [1/1] doc: Document .min_cpus | expand

Commit Message

Petr Vorel Dec. 7, 2020, 2:56 p.m. UTC
Recently added (in fa32a727d).

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Feel free to document it more.

Kind regards,
Petr

 doc/test-writing-guidelines.txt | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

Li Wang Dec. 8, 2020, 2:44 a.m. UTC | #1
Hi Petr,

I didn't write down this since I thought it's easy to understand the
.min_cpus.
(sounds like an excuse to mask my lazy :)
But it's nice to have, thanks for writing this!

Just a minor comment below.

On Mon, Dec 7, 2020 at 10:56 PM Petr Vorel <pvorel@suse.cz> wrote:

> Recently added (in fa32a727d).
>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
>
Acked-by: Li Wang <liwang@redhat.com>


> ---
> Feel free to document it more.
>
> Kind regards,
> Petr
>
>  doc/test-writing-guidelines.txt | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/doc/test-writing-guidelines.txt
> b/doc/test-writing-guidelines.txt
> index 3c2ab7166..dcb5d9d43 100644
> --- a/doc/test-writing-guidelines.txt
> +++ b/doc/test-writing-guidelines.txt
> @@ -2134,8 +2134,14 @@ struct tst_test test = {
>         .test_all = run,
>         ...
>  };
> -
>
>  -------------------------------------------------------------------------------
> +
> +2.2.37 Require minimum numbers of CPU for a testcase
> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> +
> +Some tests require more than single CPU. It can be defined with
>

single --> specific number of



> +`.min_cpus = N`.
> +
>  2.3 Writing a testcase in shell
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> --
> 2.29.2
>
>
Petr Vorel Dec. 8, 2020, 6:44 a.m. UTC | #2
Hi Li,

> Hi Petr,

> I didn't write down this since I thought it's easy to understand the
> .min_cpus.
> (sounds like an excuse to mask my lazy :)
I'm lazy to grep library sources, I look into docs :).

> But it's nice to have, thanks for writing this!

> Just a minor comment below.

> On Mon, Dec 7, 2020 at 10:56 PM Petr Vorel <pvorel@suse.cz> wrote:

> > Recently added (in fa32a727d).

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>

> Acked-by: Li Wang <liwang@redhat.com>
Thanks!

...
> > +2.2.37 Require minimum numbers of CPU for a testcase
> > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > +
> > +Some tests require more than single CPU. It can be defined with


> single --> specific number of

+1

Kind regards,
Petr
Li Wang Dec. 9, 2020, 6:24 a.m. UTC | #3
On Tue, Dec 8, 2020 at 2:45 PM Petr Vorel <pvorel@suse.cz> wrote:

> [...]
>
> > single --> specific number of
>
> +1
>

I help to modified and pushed, thanks Petr.
Petr Vorel Dec. 9, 2020, 7:15 a.m. UTC | #4
> > > single --> specific number of

> > +1

> I help to modified and pushed, thanks Petr.
Thanks, Li!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
index 3c2ab7166..dcb5d9d43 100644
--- a/doc/test-writing-guidelines.txt
+++ b/doc/test-writing-guidelines.txt
@@ -2134,8 +2134,14 @@  struct tst_test test = {
 	.test_all = run,
 	...
 };
-
 -------------------------------------------------------------------------------
+
+2.2.37 Require minimum numbers of CPU for a testcase
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+Some tests require more than single CPU. It can be defined with
+`.min_cpus = N`.
+
 2.3 Writing a testcase in shell
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~