[lkml]   [2012]   [Jul]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 00/13] UAPI header file split

I've not checked whether any of the below are fixed in the adjustments
that you made in the last 20 hours (though it looks like at least some
of them are not), but a little scripting to check the content of the
split files showed that while most of them were okay, in the cases
below, some comment text was being discarded by the scripts.

./include/linux/irqnr.h ./include/uapi/linux/irqnr.h
./arch/frv/include/asm/types.h ./arch/frv/include/uapi/asm/types.h
./arch/cris/include/asm/types.h ./arch/cris/include/uapi/asm/types.h
./arch/arm/include/asm/types.h ./arch/arm/include/uapi/asm/types.h
./arch/sh/include/asm/types.h ./arch/sh/include/uapi/asm/types.h
./arch/ia64/include/asm/kvm_para.h ./arch/ia64/include/uapi/asm/kvm_para.h
./arch/mn10300/include/asm/types.h ./arch/mn10300/include/uapi/asm/types.h
./arch/m68k/include/asm/types.h ./arch/m68k/include/uapi/asm/types.h
./arch/avr32/include/asm/types.h ./arch/avr32/include/uapi/asm/types.h
./arch/m32r/include/asm/types.h ./arch/m32r/include/uapi/asm/types.h

I found these using the script below, which tries to check the
integrity of the transformations you are making with your scripts.
Other than the files above, the results looked good.




# Set these two variables as appropriate

ORIG=$HOME/linux-pre-uapi # Tree before UAPI split
UDIR=$HOME/uapi/linux-headers # Tree following UAPI split

cd $UDIR
for uapi in $(find . -name '*.h' | grep uapi); do
kapi=$(echo $uapi | sed 's%uapi/%%')
if test -e $kapi; then

# Create "canonicalized versions" of input and output headers, by
# stripping out all lines starting with #, and sort to remove
# duplicated lines

cat $uapi $r_kapi | grep -v '^#' | sort -u > /tmp/res.$$
cat $ORIG/$kapi | grep -v '^#' | sort -u > /tmp/orig.$$

# Are there lines in one version of the "canonical"
# files, but not the other?

comm -3 /tmp/res.$$ /tmp/orig.$$ > /tmp/cdiff.$$

if test $(cat /tmp/cdiff.$$ | wc -l) -ne 0; then
echo "========" $r_kapi $uapi
cat /tmp/cdiff.$$

Michael Kerrisk Linux man-pages maintainer;
Author of "The Linux Programming Interface",

 \ /
  Last update: 2012-07-26 16:01    [W:0.082 / U:1.932 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site