[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[Cyril Hrubis]

This adds a short paragraph that writes down previously unwritten rule
for resource cleanup.

Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
CC: Jan Stancek <jstancek@redhat.com>
---
 doc/test-writing-guidelines.txt | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
index f1912dc12..21223ba03 100644
--- a/doc/test-writing-guidelines.txt
+++ b/doc/test-writing-guidelines.txt
@@ -347,6 +347,14 @@ assigned a valid file descriptor. For most of the things you need to create
 extra flag that is set right after successful initialization though. Consider,
 for example, test setup below.
 
+We also prefer cleaning up resources that would otherwise be released on the
+program exit. There are two main reasons for this decision. Resources such as
+file descriptors and mmaped memory could block umounting a block device in
+cases where the test library has mounted a filesystem for the test temporary
+directory. Not freeing allocated memory would upset static analysis and tools
+such as valgrind and produce false-positives when checking for leaks in the
+libc and other low level libraries.
+
 [source,c]
 -------------------------------------------------------------------------------
 static int fd0, fd1, mount_flag;
-- 
2.19.2

[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[Jan Stancek]

----- Original Message -----
> This adds a short paragraph that writes down previously unwritten rule
> for resource cleanup.
> 
> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> CC: Jan Stancek <jstancek@redhat.com>
> ---
>  doc/test-writing-guidelines.txt | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/doc/test-writing-guidelines.txt
> b/doc/test-writing-guidelines.txt
> index f1912dc12..21223ba03 100644
> --- a/doc/test-writing-guidelines.txt
> +++ b/doc/test-writing-guidelines.txt
> @@ -347,6 +347,14 @@ assigned a valid file descriptor. For most of the things
> you need to create
>  extra flag that is set right after successful initialization though.
>  Consider,
>  for example, test setup below.
>  
> +We also prefer cleaning up resources that would otherwise be released on the
> +program exit. There are two main reasons for this decision. Resources such
> as
> +file descriptors and mmaped memory could block umounting a block device in
> +cases where the test library has mounted a filesystem for the test temporary
> +directory. Not freeing allocated memory would upset static analysis and
> tools
> +such as valgrind and produce false-positives when checking for leaks in the
> +libc and other low level libraries.
> +

I'm OK with the text. Do we leave the paragraph we talked about in style-guide.txt?

>  [source,c]
>  -------------------------------------------------------------------------------
>  static int fd0, fd1, mount_flag;
> --
> 2.19.2
> 
> 

[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[Petr Vorel]

Hi Cyril,

> This adds a short paragraph that writes down previously unwritten rule
> for resource cleanup.

> Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> CC: Jan Stancek <jstancek@redhat.com>
LGTM.

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

Kind regards,
Petr

[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[Cyril Hrubis]

Hi!
> I'm OK with the text. Do we leave the paragraph we talked about in style-guide.txt?

The document has to be revisited anyways, I added that to my TODO now.

-- 
Cyril Hrubis
chrubis@suse.cz

[LTP] [PATCH] [RFC] doc: Clarify what should be cleaned up at test end

[Cyril Hrubis]

Hi!
Pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz