projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Docs: document why a negative socket may be passed
[lttng-tools.git]
/
src
/
bin
/
lttng-sessiond
/
ust-app.c
diff --git
a/src/bin/lttng-sessiond/ust-app.c
b/src/bin/lttng-sessiond/ust-app.c
index 2c1b845adf56bc35ef2ec1875599e2c2113904ae..23f29c13ca414a0a02903952879701b73dda397b 100644
(file)
--- a/
src/bin/lttng-sessiond/ust-app.c
+++ b/
src/bin/lttng-sessiond/ust-app.c
@@
-489,6
+489,11
@@
void delete_ust_app_channel(int sock, struct ust_app_channel *ua_chan,
ust_registry_channel_del_free(registry, ua_chan->key,
sock >= 0);
}
ust_registry_channel_del_free(registry, ua_chan->key,
sock >= 0);
}
+ /*
+ * A negative socket can be used by the caller when
+ * cleaning-up a ua_chan in an error path. Skip the
+ * accounting in this case.
+ */
if (sock >= 0) {
save_per_pid_lost_discarded_counters(ua_chan);
}
if (sock >= 0) {
save_per_pid_lost_discarded_counters(ua_chan);
}
@@
-6066,15
+6071,17
@@
int ust_app_snapshot_record(struct ltt_ust_session *usess,
nb_packets_per_stream,
trace_archive_id);
if (ret < 0) {
nb_packets_per_stream,
trace_archive_id);
if (ret < 0) {
+ if (ret == -LTTNG_ERR_CHAN_NOT_FOUND) {
+ continue;
+ }
goto error;
}
}
registry = get_session_registry(ua_sess);
if (!registry) {
goto error;
}
}
registry = get_session_registry(ua_sess);
if (!registry) {
- DBG("Application session is being torn down. Abort snapshot record.");
- ret = -1;
- goto error;
+ DBG("Application session is being torn down. Skip application.");
+ continue;
}
ret = consumer_snapshot_channel(socket,
registry->metadata_key, output,
}
ret = consumer_snapshot_channel(socket,
registry->metadata_key, output,
@@
-6082,6
+6089,9
@@
int ust_app_snapshot_record(struct ltt_ust_session *usess,
pathname, wait, 0,
trace_archive_id);
if (ret < 0) {
pathname, wait, 0,
trace_archive_id);
if (ret < 0) {
+ if (ret == -LTTNG_ERR_CHAN_NOT_FOUND) {
+ continue;
+ }
goto error;
}
}
goto error;
}
}
@@
-6331,11
+6341,12
@@
int ust_app_regenerate_statedump_all(struct ltt_ust_session *usess)
/*
* Rotate all the channels of a session.
*
/*
* Rotate all the channels of a session.
*
- * Return
0 on success or else a negative valu
e.
+ * Return
LTTNG_OK on success or else an LTTng error cod
e.
*/
*/
-
int
ust_app_rotate_session(struct ltt_session *session)
+
enum lttng_error_code
ust_app_rotate_session(struct ltt_session *session)
{
{
- int ret = 0;
+ int ret;
+ enum lttng_error_code cmd_ret = LTTNG_OK;
struct lttng_ht_iter iter;
struct ust_app *app;
struct ltt_ust_session *usess = session->ust_session;
struct lttng_ht_iter iter;
struct ust_app *app;
struct ltt_ust_session *usess = session->ust_session;
@@
-6358,7
+6369,7
@@
int ust_app_rotate_session(struct ltt_session *session)
socket = consumer_find_socket_by_bitness(reg->bits_per_long,
usess->consumer);
if (!socket) {
socket = consumer_find_socket_by_bitness(reg->bits_per_long,
usess->consumer);
if (!socket) {
-
ret = -EINVAL
;
+
cmd_ret = LTTNG_ERR_INVALID
;
goto error;
}
goto error;
}
@@
-6367,6
+6378,7
@@
int ust_app_rotate_session(struct ltt_session *session)
reg->uid, reg->bits_per_long);
if (ret < 0 || ret == sizeof(pathname)) {
PERROR("Failed to format rotation path");
reg->uid, reg->bits_per_long);
if (ret < 0 || ret == sizeof(pathname)) {
PERROR("Failed to format rotation path");
+ cmd_ret = LTTNG_ERR_INVALID;
goto error;
}
goto error;
}
@@
-6380,6
+6392,7
@@
int ust_app_rotate_session(struct ltt_session *session)
/* is_metadata_channel */ false,
session->current_archive_id);
if (ret < 0) {
/* is_metadata_channel */ false,
session->current_archive_id);
if (ret < 0) {
+ cmd_ret = LTTNG_ERR_ROTATION_FAIL_CONSUMER;
goto error;
}
}
goto error;
}
}
@@
-6393,6
+6406,7
@@
int ust_app_rotate_session(struct ltt_session *session)
/* is_metadata_channel */ true,
session->current_archive_id);
if (ret < 0) {
/* is_metadata_channel */ true,
session->current_archive_id);
if (ret < 0) {
+ cmd_ret = LTTNG_ERR_ROTATION_FAIL_CONSUMER;
goto error;
}
}
goto error;
}
}
@@
-6417,6
+6431,7
@@
int ust_app_rotate_session(struct ltt_session *session)
ua_sess->path);
if (ret < 0 || ret == sizeof(pathname)) {
PERROR("Failed to format rotation path");
ua_sess->path);
if (ret < 0 || ret == sizeof(pathname)) {
PERROR("Failed to format rotation path");
+ cmd_ret = LTTNG_ERR_INVALID;
goto error;
}
goto error;
}
@@
-6424,15
+6439,14
@@
int ust_app_rotate_session(struct ltt_session *session)
socket = consumer_find_socket_by_bitness(app->bits_per_long,
usess->consumer);
if (!socket) {
socket = consumer_find_socket_by_bitness(app->bits_per_long,
usess->consumer);
if (!socket) {
-
ret = -EINVAL
;
+
cmd_ret = LTTNG_ERR_INVALID
;
goto error;
}
registry = get_session_registry(ua_sess);
if (!registry) {
goto error;
}
registry = get_session_registry(ua_sess);
if (!registry) {
- DBG("Application session is being torn down. Abort session rotation.");
- ret = -1;
- goto error;
+ DBG("Application session is being torn down. Skip application.");
+ continue;
}
}
@@
-6445,6
+6459,10
@@
int ust_app_rotate_session(struct ltt_session *session)
/* is_metadata_channel */ false,
session->current_archive_id);
if (ret < 0) {
/* is_metadata_channel */ false,
session->current_archive_id);
if (ret < 0) {
+ /* Per-PID buffer and application going away. */
+ if (ret == -LTTNG_ERR_CHAN_NOT_FOUND)
+ continue;
+ cmd_ret = LTTNG_ERR_ROTATION_FAIL_CONSUMER;
goto error;
}
}
goto error;
}
}
@@
-6457,6
+6475,10
@@
int ust_app_rotate_session(struct ltt_session *session)
/* is_metadata_channel */ true,
session->current_archive_id);
if (ret < 0) {
/* is_metadata_channel */ true,
session->current_archive_id);
if (ret < 0) {
+ /* Per-PID buffer and application going away. */
+ if (ret == -LTTNG_ERR_CHAN_NOT_FOUND)
+ continue;
+ cmd_ret = LTTNG_ERR_ROTATION_FAIL_CONSUMER;
goto error;
}
}
goto error;
}
}
@@
-6467,9
+6489,9
@@
int ust_app_rotate_session(struct ltt_session *session)
break;
}
break;
}
- ret = LTTNG_OK;
+
cmd_
ret = LTTNG_OK;
error:
rcu_read_unlock();
error:
rcu_read_unlock();
- return ret;
+ return
cmd_
ret;
}
}
This page took
0.025694 seconds
and
4
git commands to generate.