All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH 1/1] doc/user-guide: LTP_RUNTIME_MUL is only for C API
@ 2022-09-12 10:19 Petr Vorel
  2022-09-12 13:37 ` Cyril Hrubis
  0 siblings, 1 reply; 3+ messages in thread
From: Petr Vorel @ 2022-09-12 10:19 UTC (permalink / raw)
  To: ltp; +Cc: Richard Palethorpe

Not yet implemented in shell API.
+ fix indent.

Fixes: 0f72d519a ("Introduce a concept of max runtime")

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 doc/user-guide.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/user-guide.txt b/doc/user-guide.txt
index 0dc0c958e..fdd74ca41 100644
--- a/doc/user-guide.txt
+++ b/doc/user-guide.txt
@@ -27,8 +27,8 @@ For running LTP network tests see `testcases/network/README.md`.
                           Variable is also used in shell tests, but ceiled to int.
 | 'LTP_RUNTIME_MUL'     | Multiplies maximal test iteration runtime. Tests
                           that run for more than a second or two are capped on
-			  runtime. You can scale the default runtime both up
-			  and down with this multiplier.
+                          runtime. You can scale the default runtime both up
+                          and down with this multiplier. NOTE: only for C API.
 | 'LTP_VIRT_OVERRIDE'   | Overrides virtual machine detection in the test
                           library. Setting it to empty string tell the library
                           that system is not a virtual machine. Other possible
-- 
2.37.3


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [LTP] [PATCH 1/1] doc/user-guide: LTP_RUNTIME_MUL is only for C API
  2022-09-12 10:19 [LTP] [PATCH 1/1] doc/user-guide: LTP_RUNTIME_MUL is only for C API Petr Vorel
@ 2022-09-12 13:37 ` Cyril Hrubis
  2022-09-12 20:11   ` Petr Vorel
  0 siblings, 1 reply; 3+ messages in thread
From: Cyril Hrubis @ 2022-09-12 13:37 UTC (permalink / raw)
  To: Petr Vorel; +Cc: Richard Palethorpe, ltp

Hi!
> Not yet implemented in shell API.
> + fix indent.
> 
> Fixes: 0f72d519a ("Introduce a concept of max runtime")

Should we rather then add "Not yet implemented" instead of "only for C
API" to the actual documentation too?

> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  doc/user-guide.txt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/doc/user-guide.txt b/doc/user-guide.txt
> index 0dc0c958e..fdd74ca41 100644
> --- a/doc/user-guide.txt
> +++ b/doc/user-guide.txt
> @@ -27,8 +27,8 @@ For running LTP network tests see `testcases/network/README.md`.
>                            Variable is also used in shell tests, but ceiled to int.
>  | 'LTP_RUNTIME_MUL'     | Multiplies maximal test iteration runtime. Tests
>                            that run for more than a second or two are capped on
> -			  runtime. You can scale the default runtime both up
> -			  and down with this multiplier.
> +                          runtime. You can scale the default runtime both up
> +                          and down with this multiplier. NOTE: only for C API.
>  | 'LTP_VIRT_OVERRIDE'   | Overrides virtual machine detection in the test
>                            library. Setting it to empty string tell the library
>                            that system is not a virtual machine. Other possible
> -- 
> 2.37.3
> 

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [LTP] [PATCH 1/1] doc/user-guide: LTP_RUNTIME_MUL is only for C API
  2022-09-12 13:37 ` Cyril Hrubis
@ 2022-09-12 20:11   ` Petr Vorel
  0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2022-09-12 20:11 UTC (permalink / raw)
  To: Cyril Hrubis; +Cc: Richard Palethorpe, ltp

> Hi!
> > Not yet implemented in shell API.
> > + fix indent.

> > Fixes: 0f72d519a ("Introduce a concept of max runtime")

> Should we rather then add "Not yet implemented" instead of "only for C
> API" to the actual documentation too?
Sure, no problem.

Kind regards,
Petr

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>
> > ---
> >  doc/user-guide.txt | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)

> > diff --git a/doc/user-guide.txt b/doc/user-guide.txt
> > index 0dc0c958e..fdd74ca41 100644
> > --- a/doc/user-guide.txt
> > +++ b/doc/user-guide.txt
> > @@ -27,8 +27,8 @@ For running LTP network tests see `testcases/network/README.md`.
> >                            Variable is also used in shell tests, but ceiled to int.
> >  | 'LTP_RUNTIME_MUL'     | Multiplies maximal test iteration runtime. Tests
> >                            that run for more than a second or two are capped on
> > -			  runtime. You can scale the default runtime both up
> > -			  and down with this multiplier.
> > +                          runtime. You can scale the default runtime both up
> > +                          and down with this multiplier. NOTE: only for C API.
> >  | 'LTP_VIRT_OVERRIDE'   | Overrides virtual machine detection in the test
> >                            library. Setting it to empty string tell the library
> >                            that system is not a virtual machine. Other possible
> > -- 
> > 2.37.3

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-09-12 20:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-12 10:19 [LTP] [PATCH 1/1] doc/user-guide: LTP_RUNTIME_MUL is only for C API Petr Vorel
2022-09-12 13:37 ` Cyril Hrubis
2022-09-12 20:11   ` 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.