Hi Rabin,
The patch looks fine to me -- forwarding to the trace.c maintainer Jie Fei
directly for his ACK.
Thanks,
  Dave
  
----- Original Message -----
 From: Rabin Vincent <rabinv(a)axis.com>
 
 This max buffer is not available in all configurations of the kernel so
 we shouldn't attempt to use the unitialized max_tr_ring_buffer variable.
 A similar check existed previously before the recent rework of the trace
 extension to support multiple buffers.
 ---
  extensions/trace.c | 3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/extensions/trace.c b/extensions/trace.c
 index 51df98d..22039a9 100644
 --- a/extensions/trace.c
 +++ b/extensions/trace.c
 @@ -487,6 +487,9 @@ static int ftrace_init_trace(struct trace_instance *ti,
 ulong instance_addr)
  			ti->pages) < 0)
  		goto out_fail;
  
 +	if (!ti->max_tr_ring_buffer)
 +		return 0;
 +
  	ti->max_tr_buffers = calloc(sizeof(*ti->max_tr_buffers), nr_cpu_ids);
  	if (ti->max_tr_buffers == NULL)
  		goto out_fail;
 --
 2.1.4
 
 --
 Crash-utility mailing list
 Crash-utility(a)redhat.com
 
https://www.redhat.com/mailman/listinfo/crash-utility