summaryrefslogtreecommitdiffstats
path: root/sys/security/mac
diff options
context:
space:
mode:
authormarkj <markj@FreeBSD.org>2015-12-16 23:39:27 +0000
committermarkj <markj@FreeBSD.org>2015-12-16 23:39:27 +0000
commitfa1b8e9a4f8f45f1d5675a9f3cb1fd6c504ccbc3 (patch)
tree76bad87288933c1e1da6401e046cbcc61703d1fd /sys/security/mac
parentf64eb0a1eff489953daa256595df542b0f8aeffd (diff)
downloadFreeBSD-src-fa1b8e9a4f8f45f1d5675a9f3cb1fd6c504ccbc3.zip
FreeBSD-src-fa1b8e9a4f8f45f1d5675a9f3cb1fd6c504ccbc3.tar.gz
Fix style issues around existing SDT probes.
- Use SDT_PROBE<N>() instead of SDT_PROBE(). This has no functional effect at the moment, but will be needed for some future changes. - Don't hardcode the module component of the probe identifier. This is set automatically by the SDT framework. MFC after: 1 week
Diffstat (limited to 'sys/security/mac')
-rw-r--r--sys/security/mac/mac_framework.c12
-rw-r--r--sys/security/mac/mac_internal.h32
2 files changed, 22 insertions, 22 deletions
diff --git a/sys/security/mac/mac_framework.c b/sys/security/mac/mac_framework.c
index 0f0a2ff..d76b280 100644
--- a/sys/security/mac/mac_framework.c
+++ b/sys/security/mac/mac_framework.c
@@ -93,11 +93,11 @@ __FBSDID("$FreeBSD$");
SDT_PROVIDER_DEFINE(mac);
SDT_PROVIDER_DEFINE(mac_framework);
-SDT_PROBE_DEFINE2(mac, kernel, policy, modevent, "int",
+SDT_PROBE_DEFINE2(mac, , policy, modevent, "int",
"struct mac_policy_conf *");
-SDT_PROBE_DEFINE1(mac, kernel, policy, register,
+SDT_PROBE_DEFINE1(mac, , policy, register,
"struct mac_policy_conf *");
-SDT_PROBE_DEFINE1(mac, kernel, policy, unregister,
+SDT_PROBE_DEFINE1(mac, , policy, unregister,
"struct mac_policy_conf *");
/*
@@ -444,7 +444,7 @@ mac_policy_register(struct mac_policy_conf *mpc)
(*(mpc->mpc_ops->mpo_init))(mpc);
mac_policy_update();
- SDT_PROBE(mac, kernel, policy, register, mpc, 0, 0, 0, 0);
+ SDT_PROBE1(mac, , policy, register, mpc);
printf("Security policy loaded: %s (%s)\n", mpc->mpc_fullname,
mpc->mpc_name);
@@ -491,7 +491,7 @@ mac_policy_unregister(struct mac_policy_conf *mpc)
mac_policy_update();
mac_policy_xunlock();
- SDT_PROBE(mac, kernel, policy, unregister, mpc, 0, 0, 0, 0);
+ SDT_PROBE1(mac, , policy, unregister, mpc);
printf("Security policy unload: %s (%s)\n", mpc->mpc_fullname,
mpc->mpc_name);
@@ -517,7 +517,7 @@ mac_policy_modevent(module_t mod, int type, void *data)
}
#endif
- SDT_PROBE(mac, kernel, policy, modevent, type, mpc, 0, 0, 0);
+ SDT_PROBE2(mac, , policy, modevent, type, mpc);
switch (type) {
case MOD_LOAD:
if (mpc->mpc_loadtime_flags & MPC_LOADTIME_FLAG_NOTLATE &&
diff --git a/sys/security/mac/mac_internal.h b/sys/security/mac/mac_internal.h
index ceb1e87..2ecffc6 100644
--- a/sys/security/mac/mac_internal.h
+++ b/sys/security/mac/mac_internal.h
@@ -74,35 +74,35 @@ SDT_PROVIDER_DECLARE(mac); /* MAC Framework-level events. */
SDT_PROVIDER_DECLARE(mac_framework); /* Entry points to MAC. */
#define MAC_CHECK_PROBE_DEFINE4(name, arg0, arg1, arg2, arg3) \
- SDT_PROBE_DEFINE5(mac_framework, kernel, name, mac__check__err, \
+ SDT_PROBE_DEFINE5(mac_framework, , name, mac__check__err, \
"int", arg0, arg1, arg2, arg3); \
- SDT_PROBE_DEFINE5(mac_framework, kernel, name, mac__check__ok, \
+ SDT_PROBE_DEFINE5(mac_framework, , name, mac__check__ok, \
"int", arg0, arg1, arg2, arg3);
#define MAC_CHECK_PROBE_DEFINE3(name, arg0, arg1, arg2) \
- SDT_PROBE_DEFINE4(mac_framework, kernel, name, mac__check__err, \
+ SDT_PROBE_DEFINE4(mac_framework, , name, mac__check__err, \
"int", arg0, arg1, arg2); \
- SDT_PROBE_DEFINE4(mac_framework, kernel, name, mac__check__ok, \
+ SDT_PROBE_DEFINE4(mac_framework, , name, mac__check__ok, \
"int", arg0, arg1, arg2);
#define MAC_CHECK_PROBE_DEFINE2(name, arg0, arg1) \
- SDT_PROBE_DEFINE3(mac_framework, kernel, name, mac__check__err, \
+ SDT_PROBE_DEFINE3(mac_framework, , name, mac__check__err, \
"int", arg0, arg1); \
- SDT_PROBE_DEFINE3(mac_framework, kernel, name, mac__check__ok, \
+ SDT_PROBE_DEFINE3(mac_framework, , name, mac__check__ok, \
"int", arg0, arg1);
#define MAC_CHECK_PROBE_DEFINE1(name, arg0) \
- SDT_PROBE_DEFINE2(mac_framework, kernel, name, mac__check__err, \
+ SDT_PROBE_DEFINE2(mac_framework, , name, mac__check__err, \
"int", arg0); \
- SDT_PROBE_DEFINE2(mac_framework, kernel, name, mac__check__ok, \
+ SDT_PROBE_DEFINE2(mac_framework, , name, mac__check__ok, \
"int", arg0);
#define MAC_CHECK_PROBE4(name, error, arg0, arg1, arg2, arg3) do { \
if (error) { \
- SDT_PROBE(mac_framework, kernel, name, mac__check__err, \
+ SDT_PROBE5(mac_framework, , name, mac__check__err, \
error, arg0, arg1, arg2, arg3); \
} else { \
- SDT_PROBE(mac_framework, kernel, name, mac__check__ok, \
+ SDT_PROBE5(mac_framework, , name, mac__check__ok, \
0, arg0, arg1, arg2, arg3); \
} \
} while (0)
@@ -116,18 +116,18 @@ SDT_PROVIDER_DECLARE(mac_framework); /* Entry points to MAC. */
#endif
#define MAC_GRANT_PROBE_DEFINE2(name, arg0, arg1) \
- SDT_PROBE_DEFINE3(mac_framework, kernel, name, mac__grant__err, \
+ SDT_PROBE_DEFINE3(mac_framework, , name, mac__grant__err, \
"int", arg0, arg1); \
- SDT_PROBE_DEFINE3(mac_framework, kernel, name, mac__grant__ok, \
+ SDT_PROBE_DEFINE3(mac_framework, , name, mac__grant__ok, \
"int", arg0, arg1);
#define MAC_GRANT_PROBE2(name, error, arg0, arg1) do { \
if (error) { \
- SDT_PROBE(mac_framework, kernel, name, mac__grant__err, \
- error, arg0, arg1, 0, 0); \
+ SDT_PROBE3(mac_framework, , name, mac__grant__err, \
+ error, arg0, arg1); \
} else { \
- SDT_PROBE(mac_framework, kernel, name, mac__grant__ok, \
- error, arg0, arg1, 0, 0); \
+ SDT_PROBE3(mac_framework, , name, mac__grant__ok, \
+ error, arg0, arg1); \
} \
} while (0)
OpenPOWER on IntegriCloud