On 18.11.2020 18:46, Julien Grall wrote: > On 23/10/2020 11:18, Jan Beulich wrote: >> Build this code into an archive, which results in not linking it into >> x86 final binaries. This saves about 1.5k of dead code. >> >> While moving the source file, take the opportunity and drop the >> pointless EXPORT_SYMBOL(). > > Given this code is borrowed from Linux, I don't think we want to remove > them. This is to make easier to re-sync.
That's one view on it. My view is that we should get rid of EXPORT_SYMBOL altogether (and it is a good opportunity to do so here). Not the least because otherwise for future cloning of Linux code we may then need to introduce further variants of this construct for no real purpose. >> --- a/xen/common/rbtree.c >> +++ b/xen/lib/rbtree.c >> @@ -25,7 +25,7 @@ >> #include <xen/rbtree.h> >> >> /* >> - * red-black trees properties: http://en.wikipedia.org/wiki/Rbtree >> + * red-black trees properties: http://en.wikipedia.org/wiki/Rbtree > > This looks like a spurious change. Not really, no - while not visible here anymore, it's correcting an instance of trailing whitespace. Following your request on an earlier patch, I've also added half a sentence to the description here to mention this entirely benign change. Jan
