Re: [PATCH 1/4] xen: build infrastructure for generating hypercall depending symbols

[Juergen Gross]

On 12/12/2014 11:48 PM, Boris Ostrovsky wrote:
> On 12/11/2014 01:04 PM, Juergen Gross wrote:
> > diff --git a/scripts/xen-hypercalls.sh b/scripts/xen-hypercalls.sh
> > new file mode 100644
> > index 0000000..e6447b7
> > --- /dev/null
> > +++ b/scripts/xen-hypercalls.sh
> > @@ -0,0 +1,11 @@
> > +#!/bin/sh
> > +out="$1"
> > +shift
> > +in="$@"
> > +
> > +for i in $in; do
> > +    eval $CPP $LINUXINCLUDE -dD -imacros "$i" -x c /dev/null
> > +done | \
> > +awk '$1 == "#define" && $2 ~ /__HYPERVISOR_[a-z][a-z_0-9]*/ { v[$3] =
> > $2 }
> > +    END {for (i in v) if (!(v[i] in v))
> > +        print "HYPERCALL("substr(v[i], 14)")"}' | sort -u >$out
>
> Why do you 'sort -u'? Do you expect multiple definitions of the same
> hypercall?

Paranoia related to the use of wildcards for files scanned:

$(srctree)/include/xen/interface/xen*.h


Juergen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/