this patch pretty prints the hypercall section for $xenperf -f previously, xenperf -f simply listed counts for all hyper calls. I added a hypercall index to this so it will be easier to tell which hypercall was called how many times. this is an architecture independent patch. Signed-off-by: Ken Hironaka <hironaka.ken@soft.fujitsu.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Hollis Blanchard
2006-Aug-18 16:28 UTC
Re: [Xen-devel] [PATCH][RFC]xenperf hypercall pretty print
On Fri, 2006-08-18 at 18:48 +0900, Ken Hironaka wrote:> > +const char* hypercall_name_table[64]> +{ > + "set_trap_table", //0 > + "mmu_update", //1 > + "set_gdt", //2... Rather than have all those numbers in the comments, it would be better to use the C99 array initialization syntax: const char* hypercall_name_table[64] = { [__HYPERVISOR_set_trap_table] = "set_trap_table", [__HYPERVISOR_mmu_update] = "mmu_update", ... };> @@ -117,10 +185,17 @@ int main(int argc, char *argv[]) > sum += val[j]; > printf ("T=%10u ", (unsigned int)sum); > > - if ( full || (pcd[i].nr_vals <= 4) ) > + if ( full || (pcd[i].nr_vals <= 4) ){ > + if( strcmp(pcd[i].name, "hypercalls") == 0 ){ > + for( j = 0; j < pcd[i].nr_vals; j++ ) > + printf("%s\t", > hypercall_name_table[j]); > + printf("\n"); > + } > for ( j = 0; j < pcd[i].nr_vals; j++ ) > printf(" %10u", (unsigned int)val[j]); > > + } > + > printf("\n"); > val += pcd[i].nr_vals; > }Looks like the whitespace got out of hand here. Please double-check. Also, it looks like this file has had some tabs creep into it. You should probably send a tab-removal patch first, then this patch on top of that. -- Hollis Blanchard IBM Linux Technology Center _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Mark Williamson
2006-Aug-18 16:32 UTC
Re: [Xen-devel] [PATCH][RFC]xenperf hypercall pretty print
> On Fri, 2006-08-18 at 18:48 +0900, Ken Hironaka wrote: > > +const char* hypercall_name_table[64]> > +{ > > + "set_trap_table", //0 > > + "mmu_update", //1 > > + "set_gdt", //2 > > ... > > Rather than have all those numbers in the comments, it would be better > to use the C99 array initialization syntax: > > const char* hypercall_name_table[64] = { > [__HYPERVISOR_set_trap_table] = "set_trap_table", > [__HYPERVISOR_mmu_update] = "mmu_update", > ... > };Agreed! Also, just thought I''d be pedantic and mention we generally seek to avoid C++ style comments. We seem to have standardised on /* */ comments. Sorry, Ken, I know it''s geeky but I saw those slashes and couldn''t resist! Maybe it''ll be useful for future patches. Cheers, Mark> > @@ -117,10 +185,17 @@ int main(int argc, char *argv[]) > > sum += val[j]; > > printf ("T=%10u ", (unsigned int)sum); > > > > - if ( full || (pcd[i].nr_vals <= 4) ) > > + if ( full || (pcd[i].nr_vals <= 4) ){ > > + if( strcmp(pcd[i].name, "hypercalls") == 0 ){ > > + for( j = 0; j < pcd[i].nr_vals; j++ ) > > + printf("%s\t", > > hypercall_name_table[j]); > > + printf("\n"); > > + } > > for ( j = 0; j < pcd[i].nr_vals; j++ ) > > printf(" %10u", (unsigned int)val[j]); > > > > + } > > + > > printf("\n"); > > val += pcd[i].nr_vals; > > } > > Looks like the whitespace got out of hand here. Please double-check. > > Also, it looks like this file has had some tabs creep into it. You > should probably send a tab-removal patch first, then this patch on top > of that.-- Dave: Just a question. What use is a unicyle with no seat? And no pedals! Mark: To answer a question with a question: What use is a skateboard? Dave: Skateboards have wheels. Mark: My wheel has a wheel! _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel