Re: [Xen-devel] [PATCH] tools/foreign: Avoid using alignment directives when not appropriate

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 08/03/16 09:54, Jan Beulich wrote:
>>>> On 07.03.16 at 19:28, <andrew.cooper3@xxxxxxxxxx> wrote:
>> --- a/tools/include/xen-foreign/Makefile
>> +++ b/tools/include/xen-foreign/Makefile
>> @@ -35,6 +35,8 @@ x86_32.h: mkheader.py structs.py 
>> $(ROOT)/arch-x86/xen-x86_32.h $(ROOT)/arch-x86/
>>  
>>  x86_64.h: mkheader.py structs.py $(ROOT)/arch-x86/xen-x86_64.h 
>> $(ROOT)/arch-x86/xen.h $(ROOT)/xen.h
>>      $(PYTHON) $< $* $@ $(filter %.h,$^)
>> +    #Avoid mixing an alignment directive with a uint64_t cast or sizeof 
>> expression
>> +    sed 's/(__align8__ uint64_t)/(uint64_t)/g' -i $@
> A two step rule like this should make use of a temporary file, to
> avoid breakage when the build process gets interrupted between
> the two steps.
>
> And then - is it perhaps worth to generalize the pattern in one or
> more of a couple of possible ways? Considering int64_t uses
> would perhaps be the most relevant one (even if not needed
> right away). But of course this could get as generic as
>
> s/(__align[0-9]*__ \([a-z0-9_]*\))/(\1)/g
>
> without - afaict (based on your commit description) - breaking
> anything.

Both of these would want to be + rather than * to ensure some content.

While generic is usually better, in this case I think it is better to
stick with the most specific fix, to reduce the risk of accidentally
clobbering a real __align__.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel