Discussion:
[pcp] PCP Updates: kenj: Conditional context locking in libpcp
Dave Brolley
2017-07-19 15:45:39 UTC
Permalink
Changes committed to git://git.pcp.io/pcp master

Ken McDonell (1):
libpcp + apps: move conditional context locking inside libpcp

src/libpcp/src/logutil.c | 119
+++++++++++++++++++++++++++++++---------
src/pmdumplog/pmdumplog.c | 5 -
src/pmlogcheck/pass3.c | 4 -
src/pmlogrewrite/pmlogrewrite.c | 5 -
4 files changed, 95 insertions(+), 38 deletions(-)

Details ...

commit a35c3b1f2c35037be6731e755be32566c25b7ff6
Author: Ken McDonell <***@internode.on.net>
Date: Wed Jul 19 20:52:21 2017 +1000

libpcp + apps: move conditional context locking inside libpcp

__pmLogFetch() and __pmLogRead_ctx() will now lock and unlock the
current context if it is not already locked ... this is only possible
now that context locks are not recursive.

This allowed us to remove the PM_LOCK()/PM_UNLOCK() operations above
the PMAPI for pmlogdump, pmlogcheck and pmlogrewrite.

All of this as per Dave's suggestion in a recent review.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links:

You receive all messages sent to this group.

View/Reply Online (#16240): https://groups.io/g/pcp/message/16240
View All Messages In Topic (1): https://groups.io/g/pcp/topic/5584612
Mute This Topic: https://groups.io/mt/5584612/174549
New Topic: https://groups.io/g/pcp/post
-=-=-
pcp mailing list
***@groups.io
https://groups.io/g/pcp/messages
-=-=-
Change Your Subscription: https://groups.io/g/pcp/editsub/174549
Group Home: https://groups.io/g/pcp
Contact Group Owner: pcp+***@groups.io
Terms of Service: https://groups.io/static/tos
Unsubscribe: https://groups.io/g/pcp/leave/354206/1187008340/xyzzy
-=-=-=-=-=-=-=-=-=-=-=-

Loading...