FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Crash Utility

 
 
LinkBack Thread Tools
 
Old 06-19-2012, 01:07 PM
Dave Anderson
 
Default Update trace extension for per-cpu buffer size

----- Original Message -----
> On 06/18/2012 11:15 PM, Dave Anderson wrote:
> >
> > ----- Original Message -----
> >> Since Linux v3.4 (specifically, commit 438ced1720b584000 "ring-buffer:
> >> Add per_cpu ring buffer control files"), the trace buffer size is now
> >> per-cpu. The patch below updates the trace extension to handle this.
> >>
> >> Rabin
>
>
> It looks very good to me, but for future possible convenience, it would
> be better if the buffers[i].nr_pages is also set when !per_cpu_buffer_sizes.
>
> ACK-ed.
>
> Thanks,
> Lai

OK thanks -- I have updated Rabin's patch with your additional suggestion.
The updated patch for crash-6.0.8 is attached.

Thanks,
Dave

>
> >> @@ -362,6 +374,10 @@ static int ftrace_init_buffers(struct
> >> ring_buffer_per_cpu *buffers,
> >> buffer_read_value(reader_page);
> >> buffer_read_value(overrun);
> >> buffer_read_value(entries);
> >> + if (per_cpu_buffer_sizes) {
> >> + buffer_read_value(nr_pages);
> >> + pages = buffers[i].nr_pages;
> >> + }
>
> else {
> buffers[i].nr_pages = pages;
> }
>
>
>
> >> #undef buffer_read_value
> >>
> >> if (ftrace_init_pages(buffers + i, pages) < 0)
>
>
>
> >> --
> >> 1.7.9.5
> >>
--
Crash-utility mailing list
Crash-utility@redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility
 

Thread Tools




All times are GMT. The time now is 06:35 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org