lkml.org 
[lkml]   [2012]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: KGTP (Linux Kernel debugger and tracer) 20120424 release(doc update)[1/3]code
Date
Hui Zhu <teawater@gmail.com> writes:

Could you please split it down further. a 9kLOC patch is pretty daunting.

From a quick glance over some focus on simplication would be good: no
own allocators (kernel has enough), no own types (that would remove
asm), remove optional facilities like the private ring buffer. Some
obsolete code like using semaphores vs mutexes. The variable list looks
overcomplicated, just use an array? Generally too many macros I would
say, try using inlines or removing them.

Do you have any rarely used optional features that could be left out a
version 1? If yes do that. I'm sure with some effort the 9kLOC could be
much less. A patch half the size would be much easier to merge.

There are some things that checkpatch will likely warn about.

It would be good if you could describe the use case a bit better:
this is for the kernel only or also user applications?

-Andi

--
ak@linux.intel.com -- Speaking for myself only


\
 
 \ /
  Last update: 2012-04-25 11:53    [W:0.033 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site