summaryrefslogtreecommitdiff
path: root/src/stdio/fgetc.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-07-30 08:02:14 -0400
committerRich Felker <dalias@aerifal.cx>2011-07-30 08:02:14 -0400
commitdba68bf98fc708cea4c478278c889fc7ad802b00 (patch)
treeb90a55aaa51c72475cdb69efca7b9ddd37e50b2e /src/stdio/fgetc.c
parent7683fceedebd98dda19b1a379b92406b64ce7f92 (diff)
downloadmusl-dba68bf98fc708cea4c478278c889fc7ad802b00.tar.gz
musl-dba68bf98fc708cea4c478278c889fc7ad802b00.tar.bz2
musl-dba68bf98fc708cea4c478278c889fc7ad802b00.tar.xz
musl-dba68bf98fc708cea4c478278c889fc7ad802b00.zip
add proper fuxed-based locking for stdio
previously, stdio used spinlocks, which would be unacceptable if we ever add support for thread priorities, and which yielded pathologically bad performance if an application attempted to use flockfile on a key file as a major/primary locking mechanism. i had held off on making this change for fear that it would hurt performance in the non-threaded case, but actually support for recursive locking had already inflicted that cost. by having the internal locking functions store a flag indicating whether they need to perform unlocking, rather than using the actual recursive lock counter, i was able to combine the conditionals at unlock time, eliminating any additional cost, and also avoid a nasty corner case where a huge number of calls to ftrylockfile could cause deadlock later at the point of internal locking. this commit also fixes some issues with usage of pthread_self conflicting with __attribute__((const)) which resulted in crashes with some compiler versions/optimizations, mainly in flockfile prior to pthread_create.
Diffstat (limited to 'src/stdio/fgetc.c')
-rw-r--r--src/stdio/fgetc.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/stdio/fgetc.c b/src/stdio/fgetc.c
index da638684..4d8aca37 100644
--- a/src/stdio/fgetc.c
+++ b/src/stdio/fgetc.c
@@ -3,9 +3,10 @@
int fgetc(FILE *f)
{
int c;
- FLOCK(f);
+ if (f->lock < 0 || !__lockfile(f))
+ return getc_unlocked(f);
c = getc_unlocked(f);
- FUNLOCK(f);
+ __unlockfile(f);
return c;
}