diff mbox series

[1/1] doc: LTP C And Shell Test API Comparison

Message ID 20221004120653.23237-1-pvorel@suse.cz
State Accepted
Headers show
Series [1/1] doc: LTP C And Shell Test API Comparison | expand

Commit Message

Petr Vorel Oct. 4, 2022, 12:06 p.m. UTC
Deliberately put ndash (–) instead of TODO on missing features,
because not everything needs to be implemented.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Do you consider this useful?
If yes, should there be links to the doc section?

Kind regards,
Petr

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

Comments

Richard Palethorpe Oct. 18, 2022, 8:59 a.m. UTC | #1
Hello,

Petr Vorel <pvorel@suse.cz> writes:

> Deliberately put ndash (–) instead of TODO on missing features,
> because not everything needs to be implemented.
>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>

Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com>

> ---
> Do you consider this useful?
> If yes, should there be links to the doc section?

Yes, I think so.

>
> Kind regards,
> Petr
>
>  doc/test-writing-guidelines.txt | 64 +++++++++++++++++++++++++++++++++
>  1 file changed, 64 insertions(+)
>
> diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
> index 420ae9c43..7043c3902 100644
> --- a/doc/test-writing-guidelines.txt
> +++ b/doc/test-writing-guidelines.txt
> @@ -344,3 +344,67 @@ of the stable kernel ABI the associated test must be moved out of staging.
>  
>  This is primarily to help test kernel RCs by avoiding the need to download
>  separate LTP patchsets.
> +
> +8 LTP C And Shell Test API Comparison
> +-------------------------------------
> +
> +Comparison of
> +https://github.com/linux-test-project/ltp/wiki/C-Test-API[C Test API] and
> +https://github.com/linux-test-project/ltp/wiki/Shell-Test-API[Shell Test API].
> +
> +[options="header"]
> +|================================================================================
> +|  C API ('struct tst_test' members) | shell API ('$TST_*' environment variables)
> +| '.all_filesystems' | 'TST_ALL_FILESYSTEMS'
> +| '.bufs' | –
> +| '.caps' | –
> +| '.child_needs_reinit' | not applicable
> +| '.cleanup' | 'TST_CLEANUP'
> +| '.dev_extra_opts' | 'TST_DEV_EXTRA_OPTS'
> +| '.dev_fs_opts' | 'TST_DEV_FS_OPTS'
> +| '.dev_fs_type' | 'TST_FS_TYPE'
> +| '.dev_min_size' | not applicable
> +| '.format_device' | 'TST_FORMAT_DEVICE'
> +| '.max_runtime' | –
> +| '.min_cpus' | not applicable
> +| '.min_kver' | 'TST_MIN_KVER'
> +| '.min_mem_avail' | not applicable
> +| '.mnt_flags' | 'TST_MNT_PARAMS'
> +| '.mntpoint', '.mnt_data' | 'TST_MNTPOINT'
> +| '.mount_device' | 'TST_MOUNT_DEVICE'
> +| '.needs_cgroup_ctrls' | –
> +| '.needs_checkpoints' | 'NEEDS_CHECKPOINTS'
> +| '.needs_cmds' | 'TST_NEEDS_CMDS'
> +| '.needs_devfs' | –
> +| '.needs_device' | 'TST_NEEDS_DEVICE'
> +| '.needs_drivers' | 'TST_NEEDS_DRIVERS'
> +| '.needs_kconfigs' | 'TST_NEEDS_KCONFIGS'
> +| '.needs_overlay' |
> +| '.needs_rofs' | –
> +| '.needs_root' | 'TST_NEEDS_ROOT'
> +| '.needs_tmpdir' | 'TST_NEEDS_TMPDIR'
> +| '.options' | 'TST_PARSE_ARGS', 'TST_OPTS'
> +| '.resource_files' | –
> +| '.restore_wallclock' | not applicable
> +| '.sample' | –
> +| '.save_restore' | –
> +| '.scall' | not applicable
> +| '.setup' | 'TST_SETUP'
> +| '.skip_filesystems' | 'TST_SKIP_FILESYSTEMS'
> +| '.skip_in_compat' | –
> +| '.skip_in_lockdown' | –
> +| '.supported_archs' | not applicable
> +| '.tags' | –
> +| '.taint_check' | –
> +| '.tcnt' | 'TST_CNT'
> +| '.tconf_msg' | not applicable
> +| '.test', '.test_all' | 'TST_TESTFUNC'
> +| '.test_variants' | –
> +| '.timeout' | 'TST_TIMEOUT'
> +| '.tst_hugepage' | not applicable
> +| .format_device | 'TST_DEVICE'
> +| not applicable | 'TST_NEEDS_KCONFIGS_IFS'
> +| not applicable | 'TST_NEEDS_MODULE'
> +| not applicable | 'TST_POS_ARGS'
> +| not applicable | 'TST_USAGE'
> +|================================================================================
> -- 
> 2.37.3
Petr Vorel Nov. 1, 2022, 1:37 p.m. UTC | #2
Hi Richie,

thanks for your review, merged.

Kind regards,
Petr
diff mbox series

Patch

diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
index 420ae9c43..7043c3902 100644
--- a/doc/test-writing-guidelines.txt
+++ b/doc/test-writing-guidelines.txt
@@ -344,3 +344,67 @@  of the stable kernel ABI the associated test must be moved out of staging.
 
 This is primarily to help test kernel RCs by avoiding the need to download
 separate LTP patchsets.
+
+8 LTP C And Shell Test API Comparison
+-------------------------------------
+
+Comparison of
+https://github.com/linux-test-project/ltp/wiki/C-Test-API[C Test API] and
+https://github.com/linux-test-project/ltp/wiki/Shell-Test-API[Shell Test API].
+
+[options="header"]
+|================================================================================
+|  C API ('struct tst_test' members) | shell API ('$TST_*' environment variables)
+| '.all_filesystems' | 'TST_ALL_FILESYSTEMS'
+| '.bufs' | –
+| '.caps' | –
+| '.child_needs_reinit' | not applicable
+| '.cleanup' | 'TST_CLEANUP'
+| '.dev_extra_opts' | 'TST_DEV_EXTRA_OPTS'
+| '.dev_fs_opts' | 'TST_DEV_FS_OPTS'
+| '.dev_fs_type' | 'TST_FS_TYPE'
+| '.dev_min_size' | not applicable
+| '.format_device' | 'TST_FORMAT_DEVICE'
+| '.max_runtime' | –
+| '.min_cpus' | not applicable
+| '.min_kver' | 'TST_MIN_KVER'
+| '.min_mem_avail' | not applicable
+| '.mnt_flags' | 'TST_MNT_PARAMS'
+| '.mntpoint', '.mnt_data' | 'TST_MNTPOINT'
+| '.mount_device' | 'TST_MOUNT_DEVICE'
+| '.needs_cgroup_ctrls' | –
+| '.needs_checkpoints' | 'NEEDS_CHECKPOINTS'
+| '.needs_cmds' | 'TST_NEEDS_CMDS'
+| '.needs_devfs' | –
+| '.needs_device' | 'TST_NEEDS_DEVICE'
+| '.needs_drivers' | 'TST_NEEDS_DRIVERS'
+| '.needs_kconfigs' | 'TST_NEEDS_KCONFIGS'
+| '.needs_overlay' |
+| '.needs_rofs' | –
+| '.needs_root' | 'TST_NEEDS_ROOT'
+| '.needs_tmpdir' | 'TST_NEEDS_TMPDIR'
+| '.options' | 'TST_PARSE_ARGS', 'TST_OPTS'
+| '.resource_files' | –
+| '.restore_wallclock' | not applicable
+| '.sample' | –
+| '.save_restore' | –
+| '.scall' | not applicable
+| '.setup' | 'TST_SETUP'
+| '.skip_filesystems' | 'TST_SKIP_FILESYSTEMS'
+| '.skip_in_compat' | –
+| '.skip_in_lockdown' | –
+| '.supported_archs' | not applicable
+| '.tags' | –
+| '.taint_check' | –
+| '.tcnt' | 'TST_CNT'
+| '.tconf_msg' | not applicable
+| '.test', '.test_all' | 'TST_TESTFUNC'
+| '.test_variants' | –
+| '.timeout' | 'TST_TIMEOUT'
+| '.tst_hugepage' | not applicable
+| .format_device | 'TST_DEVICE'
+| not applicable | 'TST_NEEDS_KCONFIGS_IFS'
+| not applicable | 'TST_NEEDS_MODULE'
+| not applicable | 'TST_POS_ARGS'
+| not applicable | 'TST_USAGE'
+|================================================================================