* [PATCH] manual/sysinfo.texi: Fix addmntent's annotation as MT-Safe
@ 2014-11-14 8:28 Ma Shimiao
2014-11-14 16:57 ` Alexandre Oliva
0 siblings, 1 reply; 3+ messages in thread
From: Ma Shimiao @ 2014-11-14 8:28 UTC (permalink / raw)
To: libc-alpha; +Cc: aoliva, Ma Shimiao
According to the description of race in manual, function addmntent should be
marked as MT-Safe race:stream.
---
manual/sysinfo.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/manual/sysinfo.texi b/manual/sysinfo.texi
index 1c9f51b..e6c44d6 100644
--- a/manual/sysinfo.texi
+++ b/manual/sysinfo.texi
@@ -790,7 +790,7 @@ end of file reached,
@comment mntent.h
@comment BSD
@deftypefun int addmntent (FILE *@var{stream}, const struct mntent *@var{mnt})
-@safety{@prelim{}@mtunsafe{@mtasurace{:stream} @mtslocale{}}@asunsafe{@asucorrupt{}}@acunsafe{@acucorrupt{}}}
+@safety{@prelim{}@mtsafe{@mtsrace{:stream} @mtslocale{}}@asunsafe{@asucorrupt{}}@acunsafe{@acucorrupt{}}}
@c addmntent @mtasurace:stream @mtslocale @asucorrupt @acucorrupt
@c fseek dup @asucorrupt @acucorrupt [no @aculock]
@c encode_name ok
--
1.8.3.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] manual/sysinfo.texi: Fix addmntent's annotation as MT-Safe
2014-11-14 8:28 [PATCH] manual/sysinfo.texi: Fix addmntent's annotation as MT-Safe Ma Shimiao
@ 2014-11-14 16:57 ` Alexandre Oliva
2014-11-21 5:02 ` Alexandre Oliva
0 siblings, 1 reply; 3+ messages in thread
From: Alexandre Oliva @ 2014-11-14 16:57 UTC (permalink / raw)
To: Ma Shimiao; +Cc: libc-alpha
On Nov 14, 2014, Ma Shimiao <mashimiao.fnst@cn.fujitsu.com> wrote:
> According to the description of race in manual, function addmntent should be
> marked as MT-Safe race:stream.
Thanks for catching this. We need a ChangeLog entry to go with the
patch, something like:
2014-11-14 Ma Shimiao <mashimiao.fnst@cn.fujitsu.com>
* manual/sysinfo.texi (addmntent): It is actually MT-Safe,
because the potential race is on the user-supplied stream.
If nobody objects till Monday, I'll check this in on Tuesday along with
the ChangeLog entry above.
Thanks!
> -@safety{@prelim{}@mtunsafe{@mtasurace{:stream} @mtslocale{}}@asunsafe{@asucorrupt{}}@acunsafe{@acucorrupt{}}}
> +@safety{@prelim{}@mtsafe{@mtsrace{:stream} @mtslocale{}}@asunsafe{@asucorrupt{}}@acunsafe{@acucorrupt{}}}
--
Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/ FSF Latin America board member
Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] manual/sysinfo.texi: Fix addmntent's annotation as MT-Safe
2014-11-14 16:57 ` Alexandre Oliva
@ 2014-11-21 5:02 ` Alexandre Oliva
0 siblings, 0 replies; 3+ messages in thread
From: Alexandre Oliva @ 2014-11-21 5:02 UTC (permalink / raw)
To: Ma Shimiao; +Cc: libc-alpha
On Nov 14, 2014, Alexandre Oliva <aoliva@redhat.com> wrote:
> On Nov 14, 2014, Ma Shimiao <mashimiao.fnst@cn.fujitsu.com> wrote:
>> According to the description of race in manual, function addmntent should be
>> marked as MT-Safe race:stream.
> Thanks for catching this. We need a ChangeLog entry to go with the
> patch, something like:
> 2014-11-14 Ma Shimiao <mashimiao.fnst@cn.fujitsu.com>
> * manual/sysinfo.texi (addmntent): It is actually MT-Safe,
> because the potential race is on the user-supplied stream.
> If nobody objects till Monday, I'll check this in on Tuesday along with
> the ChangeLog entry above.
I've just checked it in. Thanks again,
--
Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/ FSF Latin America board member
Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-21 5:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-14 8:28 [PATCH] manual/sysinfo.texi: Fix addmntent's annotation as MT-Safe Ma Shimiao
2014-11-14 16:57 ` Alexandre Oliva
2014-11-21 5:02 ` Alexandre Oliva
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).