public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [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).