lkml.org 
[lkml]   [2015]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] perf tools: Use postorder rbtree iteration when removing symbols

* Alex Snast <asnast@gmail.com> wrote:

> What's the benefit of having that diverge check script as on every commit you'll
> either add the new stuff to tools/include/linux/rbtree.h or add an exception to
> that script as in rb_link_node_rcu case.

The benefit is that things do not diverge - diff or md5sum is fast enough.

I don't think exceptions should be added, we should find ways to make the files
100% identical.

> And are you suggesting to check for divergence for everything under tools/
> include/linux recursively?

No, on a case by case basis, for bigger 'facility files' we copied from the kernel
source.

rbtree.h and rbtree.c certainly applies: for example we've got interval rbtree
additions in -tip that would be nice to merge upstream:

tools/kvm/include/kvm/rbtree-interval.h
tools/kvm/util/rbtree-interval.c

... but which have bitrotten meanwhile, making the copy out of sync and harder to
merge. Making sure the files are properly upstreamed during build time makes sure
things don't get out of sync.

Thanks,

Ingo


\
 
 \ /
  Last update: 2015-09-21 09:21    [W:0.040 / U:1.496 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site