[lttng-dev] [LTTNG-TOOLS PATCH] Fix: reset out_fd_offset when we rotate the trace file

[David Goulet]

Merged!

Backported to 2.2 as well.

Julien Desfossez:
> This value is only used for the sync_file_range, but it has to be set
> to 0 when we start to write in a new trace file, otherwise the values
> passed to this call are bogus.
> 
> Applied to 2.3, but could probably be backported to 2.2.
> 
> Signed-off-by: Julien Desfossez <jdesfossez at efficios.com>
> ---
>  src/common/consumer.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/common/consumer.c b/src/common/consumer.c
> index 221c348..59207da 100644
> --- a/src/common/consumer.c
> +++ b/src/common/consumer.c
> @@ -1431,6 +1431,8 @@ ssize_t lttng_consumer_on_read_subbuffer_mmap(
>  			outfd = stream->out_fd = ret;
>  			/* Reset current size because we just perform a rotation. */
>  			stream->tracefile_size_current = 0;
> +			stream->out_fd_offset = 0;
> +			orig_offset = 0;
>  		}
>  		stream->tracefile_size_current += len;
>  	}
> @@ -1614,6 +1616,8 @@ ssize_t lttng_consumer_on_read_subbuffer_splice(
>  			outfd = stream->out_fd = ret;
>  			/* Reset current size because we just perform a rotation. */
>  			stream->tracefile_size_current = 0;
> +			stream->out_fd_offset = 0;
> +			orig_offset = 0;
>  		}
>  		stream->tracefile_size_current += len;
>  	}