* [lttng-dev] [PATCH lttng-modules 1/2] Add x86-64 override for accept4 syscall
@ 2016-08-29 20:02 Jérémie Galarneau
2016-08-29 20:02 ` [lttng-dev] [PATCH lttng-modules 2/2] Cleanup: reuse code in accept() and accept4() instrumentation Jérémie Galarneau
0 siblings, 1 reply; 3+ messages in thread
From: Jérémie Galarneau @ 2016-08-29 20:02 UTC (permalink / raw)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 3257 bytes --]
This patch adds an instrumentation override for the accept4() syscall
which is almost identical to accept(), except for an additional
"flags" parameter.
A follow-up patch refactors both overrides to minimize code
duplication as is done for the select/pselect6 overrides.
Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
---
.../x86-64-syscalls-3.10.0-rc7_pointers_override.h | 57 ++++++++++++++++++++++
1 file changed, 57 insertions(+)
diff --git a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
index 40fa930..6c59790 100644
--- a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
+++ b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
@@ -108,6 +108,63 @@ SC_LTTNG_TRACEPOINT_EVENT_CODE(accept,
TP_code_post()
)
+#define OVERRIDE_64_accept4
+SC_LTTNG_TRACEPOINT_EVENT_CODE(accept4,
+ TP_PROTO(sc_exit(long ret,) int fd, struct sockaddr * upeer_sockaddr, int * upeer_addrlen, int flags),
+ TP_ARGS(sc_exit(ret,) fd, upeer_sockaddr, upeer_addrlen, flags),
+ TP_locvar(
+ __typeof__(upeer_sockaddr->sa_family) sa_family;
+ uint16_t sport;
+ uint32_t v4addr;
+ uint16_t v6addr[8];
+ int v4addr_len, v6addr_len;
+ int uaddr_len;
+ ),
+ TP_code_pre(
+ sc_inout(
+ memset(tp_locvar, 0, sizeof(*tp_locvar));
+ (void) get_user(tp_locvar->uaddr_len, upeer_addrlen);
+ )
+ sc_out(
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr))
+ goto skip_code;
+ (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family);
+ switch (tp_locvar->sa_family) {
+ case AF_INET:
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in))
+ goto skip_code;
+ (void) get_user(tp_locvar->sport, &((struct sockaddr_in *) upeer_sockaddr)->sin_port);
+ (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *) upeer_sockaddr)->sin_addr.s_addr);
+ tp_locvar->v4addr_len = 4;
+ break;
+ case AF_INET6:
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6))
+ goto skip_code;
+ (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_port);
+ if (copy_from_user(tp_locvar->v6addr,
+ &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8,
+ sizeof(tp_locvar->v6addr)))
+ memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr));
+ tp_locvar->v6addr_len = 8;
+ break;
+ }
+ skip_code:
+ )
+ ),
+ TP_FIELDS(
+ sc_exit(ctf_integer(long, ret, ret))
+ sc_in(ctf_integer(int, fd, fd))
+ sc_in(ctf_integer_hex(struct sockaddr *, upeer_sockaddr, upeer_sockaddr))
+ sc_inout(ctf_integer(int, upeer_addrlen, tp_locvar->uaddr_len))
+ sc_in(ctf_integer_hex(int, flags, flags))
+ sc_out(ctf_integer(int, family, tp_locvar->sa_family))
+ sc_out(ctf_integer_network(uint16_t, sport, tp_locvar->sport))
+ sc_out(ctf_sequence_network(uint8_t, v4addr, &tp_locvar->v4addr, unsigned int, tp_locvar->v4addr_len))
+ sc_out(ctf_sequence_network(uint16_t, v6addr, &tp_locvar->v6addr, unsigned int, tp_locvar->v6addr_len))
+ ),
+ TP_code_post()
+)
+
#define OVERRIDE_64_pipe
SC_LTTNG_TRACEPOINT_EVENT(pipe,
TP_PROTO(sc_exit(long ret,) int * fildes),
--
2.9.3
^ permalink raw reply [flat|nested] 3+ messages in thread
* [lttng-dev] [PATCH lttng-modules 2/2] Cleanup: reuse code in accept() and accept4() instrumentation
2016-08-29 20:02 [lttng-dev] [PATCH lttng-modules 1/2] Add x86-64 override for accept4 syscall Jérémie Galarneau
@ 2016-08-29 20:02 ` Jérémie Galarneau
2016-08-29 20:28 ` Mathieu Desnoyers
0 siblings, 1 reply; 3+ messages in thread
From: Jérémie Galarneau @ 2016-08-29 20:02 UTC (permalink / raw)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 5324 bytes --]
Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
---
.../x86-64-syscalls-3.10.0-rc7_pointers_override.h | 106 ++++++++-------------
1 file changed, 42 insertions(+), 64 deletions(-)
diff --git a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
index 6c59790..9d94aee 100644
--- a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
+++ b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
@@ -52,48 +52,54 @@ SC_LTTNG_TRACEPOINT_EVENT_CODE(connect,
TP_code_post()
)
+#define LTTNG_SYSCALL_ACCEPT_locvar \
+ __typeof__(upeer_sockaddr->sa_family) sa_family;\
+ uint16_t sport; \
+ uint32_t v4addr; \
+ uint16_t v6addr[8]; \
+ int v4addr_len, v6addr_len; \
+ int uaddr_len;
+
+#define LTTNG_SYSCALL_ACCEPT_code_pre \
+ sc_inout( \
+ memset(tp_locvar, 0, sizeof(*tp_locvar)); \
+ (void) get_user(tp_locvar->uaddr_len, upeer_addrlen); \
+ ) \
+ sc_out( \
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr)) \
+ goto skip_code; \
+ (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family); \
+ switch (tp_locvar->sa_family) { \
+ case AF_INET: \
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in)) \
+ goto skip_code; \
+ (void) get_user(tp_locvar->sport, &((struct sockaddr_in *) upeer_sockaddr)->sin_port); \
+ (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *) upeer_sockaddr)->sin_addr.s_addr); \
+ tp_locvar->v4addr_len = 4; \
+ break; \
+ case AF_INET6: \
+ if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6)) \
+ goto skip_code; \
+ (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_port); \
+ if (copy_from_user(tp_locvar->v6addr, \
+ &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8, \
+ sizeof(tp_locvar->v6addr))) \
+ memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr)); \
+ tp_locvar->v6addr_len = 8; \
+ break; \
+ } \
+ skip_code: \
+ )
+
#define OVERRIDE_64_accept
SC_LTTNG_TRACEPOINT_EVENT_CODE(accept,
TP_PROTO(sc_exit(long ret,) int fd, struct sockaddr * upeer_sockaddr, int * upeer_addrlen),
TP_ARGS(sc_exit(ret,) fd, upeer_sockaddr, upeer_addrlen),
TP_locvar(
- __typeof__(upeer_sockaddr->sa_family) sa_family;
- uint16_t sport;
- uint32_t v4addr;
- uint16_t v6addr[8];
- int v4addr_len, v6addr_len;
- int uaddr_len;
+ LTTNG_SYSCALL_ACCEPT_locvar
),
TP_code_pre(
- sc_inout(
- memset(tp_locvar, 0, sizeof(*tp_locvar));
- (void) get_user(tp_locvar->uaddr_len, upeer_addrlen);
- )
- sc_out(
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr))
- goto skip_code;
- (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family);
- switch (tp_locvar->sa_family) {
- case AF_INET:
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in))
- goto skip_code;
- (void) get_user(tp_locvar->sport, &((struct sockaddr_in *) upeer_sockaddr)->sin_port);
- (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *) upeer_sockaddr)->sin_addr.s_addr);
- tp_locvar->v4addr_len = 4;
- break;
- case AF_INET6:
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6))
- goto skip_code;
- (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_port);
- if (copy_from_user(tp_locvar->v6addr,
- &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8,
- sizeof(tp_locvar->v6addr)))
- memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr));
- tp_locvar->v6addr_len = 8;
- break;
- }
- skip_code:
- )
+ LTTNG_SYSCALL_ACCEPT_code_pre
),
TP_FIELDS(
sc_exit(ctf_integer(long, ret, ret))
@@ -121,35 +127,7 @@ SC_LTTNG_TRACEPOINT_EVENT_CODE(accept4,
int uaddr_len;
),
TP_code_pre(
- sc_inout(
- memset(tp_locvar, 0, sizeof(*tp_locvar));
- (void) get_user(tp_locvar->uaddr_len, upeer_addrlen);
- )
- sc_out(
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr))
- goto skip_code;
- (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family);
- switch (tp_locvar->sa_family) {
- case AF_INET:
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in))
- goto skip_code;
- (void) get_user(tp_locvar->sport, &((struct sockaddr_in *) upeer_sockaddr)->sin_port);
- (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *) upeer_sockaddr)->sin_addr.s_addr);
- tp_locvar->v4addr_len = 4;
- break;
- case AF_INET6:
- if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6))
- goto skip_code;
- (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_port);
- if (copy_from_user(tp_locvar->v6addr,
- &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8,
- sizeof(tp_locvar->v6addr)))
- memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr));
- tp_locvar->v6addr_len = 8;
- break;
- }
- skip_code:
- )
+ LTTNG_SYSCALL_ACCEPT_code_pre
),
TP_FIELDS(
sc_exit(ctf_integer(long, ret, ret))
--
2.9.3
^ permalink raw reply [flat|nested] 3+ messages in thread
* [lttng-dev] [PATCH lttng-modules 2/2] Cleanup: reuse code in accept() and accept4() instrumentation
2016-08-29 20:02 ` [lttng-dev] [PATCH lttng-modules 2/2] Cleanup: reuse code in accept() and accept4() instrumentation Jérémie Galarneau
@ 2016-08-29 20:28 ` Mathieu Desnoyers
0 siblings, 0 replies; 3+ messages in thread
From: Mathieu Desnoyers @ 2016-08-29 20:28 UTC (permalink / raw)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 5887 bytes --]
Merged patch 1 and 2 into master. Fixup added: using LTTNG_SYSCALL_ACCEPT_locvar
in the accept4 code too.
Thanks,
Mathieu
----- On Aug 29, 2016, at 4:02 PM, Jeremie Galarneau jeremie.galarneau at efficios.com wrote:
> Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
> ---
> .../x86-64-syscalls-3.10.0-rc7_pointers_override.h | 106 ++++++++-------------
> 1 file changed, 42 insertions(+), 64 deletions(-)
>
> diff --git
> a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
> b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
> index 6c59790..9d94aee 100644
> ---
> a/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
> +++
> b/instrumentation/syscalls/headers/x86-64-syscalls-3.10.0-rc7_pointers_override.h
> @@ -52,48 +52,54 @@ SC_LTTNG_TRACEPOINT_EVENT_CODE(connect,
> TP_code_post()
> )
>
> +#define LTTNG_SYSCALL_ACCEPT_locvar \
> + __typeof__(upeer_sockaddr->sa_family) sa_family;\
> + uint16_t sport; \
> + uint32_t v4addr; \
> + uint16_t v6addr[8]; \
> + int v4addr_len, v6addr_len; \
> + int uaddr_len;
> +
> +#define LTTNG_SYSCALL_ACCEPT_code_pre \
> + sc_inout( \
> + memset(tp_locvar, 0, sizeof(*tp_locvar)); \
> + (void) get_user(tp_locvar->uaddr_len, upeer_addrlen); \
> + ) \
> + sc_out( \
> + if (tp_locvar->uaddr_len < sizeof(struct sockaddr)) \
> + goto skip_code; \
> + (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family); \
> + switch (tp_locvar->sa_family) { \
> + case AF_INET: \
> + if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in)) \
> + goto skip_code; \
> + (void) get_user(tp_locvar->sport, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_port); \
> + (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_addr.s_addr); \
> + tp_locvar->v4addr_len = 4; \
> + break; \
> + case AF_INET6: \
> + if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6)) \
> + goto skip_code; \
> + (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *)
> upeer_sockaddr)->sin6_port); \
> + if (copy_from_user(tp_locvar->v6addr, \
> + &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8, \
> + sizeof(tp_locvar->v6addr))) \
> + memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr)); \
> + tp_locvar->v6addr_len = 8; \
> + break; \
> + } \
> + skip_code: \
> + )
> +
> #define OVERRIDE_64_accept
> SC_LTTNG_TRACEPOINT_EVENT_CODE(accept,
> TP_PROTO(sc_exit(long ret,) int fd, struct sockaddr * upeer_sockaddr, int *
> upeer_addrlen),
> TP_ARGS(sc_exit(ret,) fd, upeer_sockaddr, upeer_addrlen),
> TP_locvar(
> - __typeof__(upeer_sockaddr->sa_family) sa_family;
> - uint16_t sport;
> - uint32_t v4addr;
> - uint16_t v6addr[8];
> - int v4addr_len, v6addr_len;
> - int uaddr_len;
> + LTTNG_SYSCALL_ACCEPT_locvar
> ),
> TP_code_pre(
> - sc_inout(
> - memset(tp_locvar, 0, sizeof(*tp_locvar));
> - (void) get_user(tp_locvar->uaddr_len, upeer_addrlen);
> - )
> - sc_out(
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr))
> - goto skip_code;
> - (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family);
> - switch (tp_locvar->sa_family) {
> - case AF_INET:
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in))
> - goto skip_code;
> - (void) get_user(tp_locvar->sport, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_port);
> - (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_addr.s_addr);
> - tp_locvar->v4addr_len = 4;
> - break;
> - case AF_INET6:
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6))
> - goto skip_code;
> - (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *)
> upeer_sockaddr)->sin6_port);
> - if (copy_from_user(tp_locvar->v6addr,
> - &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8,
> - sizeof(tp_locvar->v6addr)))
> - memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr));
> - tp_locvar->v6addr_len = 8;
> - break;
> - }
> - skip_code:
> - )
> + LTTNG_SYSCALL_ACCEPT_code_pre
> ),
> TP_FIELDS(
> sc_exit(ctf_integer(long, ret, ret))
> @@ -121,35 +127,7 @@ SC_LTTNG_TRACEPOINT_EVENT_CODE(accept4,
> int uaddr_len;
> ),
> TP_code_pre(
> - sc_inout(
> - memset(tp_locvar, 0, sizeof(*tp_locvar));
> - (void) get_user(tp_locvar->uaddr_len, upeer_addrlen);
> - )
> - sc_out(
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr))
> - goto skip_code;
> - (void) get_user(tp_locvar->sa_family, &upeer_sockaddr->sa_family);
> - switch (tp_locvar->sa_family) {
> - case AF_INET:
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in))
> - goto skip_code;
> - (void) get_user(tp_locvar->sport, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_port);
> - (void) get_user(tp_locvar->v4addr, &((struct sockaddr_in *)
> upeer_sockaddr)->sin_addr.s_addr);
> - tp_locvar->v4addr_len = 4;
> - break;
> - case AF_INET6:
> - if (tp_locvar->uaddr_len < sizeof(struct sockaddr_in6))
> - goto skip_code;
> - (void) get_user(tp_locvar->sport, &((struct sockaddr_in6 *)
> upeer_sockaddr)->sin6_port);
> - if (copy_from_user(tp_locvar->v6addr,
> - &((struct sockaddr_in6 *) upeer_sockaddr)->sin6_addr.in6_u.u6_addr8,
> - sizeof(tp_locvar->v6addr)))
> - memset(tp_locvar->v6addr, 0, sizeof(tp_locvar->v6addr));
> - tp_locvar->v6addr_len = 8;
> - break;
> - }
> - skip_code:
> - )
> + LTTNG_SYSCALL_ACCEPT_code_pre
> ),
> TP_FIELDS(
> sc_exit(ctf_integer(long, ret, ret))
> --
> 2.9.3
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-29 20:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-29 20:02 [lttng-dev] [PATCH lttng-modules 1/2] Add x86-64 override for accept4 syscall Jérémie Galarneau
2016-08-29 20:02 ` [lttng-dev] [PATCH lttng-modules 2/2] Cleanup: reuse code in accept() and accept4() instrumentation Jérémie Galarneau
2016-08-29 20:28 ` Mathieu Desnoyers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox