* Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx> wrote: > +#define efi_call_virt_pointer(p, f, args...) \ > +({ \ > + efi_status_t __s; \ > + unsigned long flags; \ > + \ > + arch_efi_call_virt_setup(); \ > + \ > + local_save_flags(flags); \ > + __s = arch_efi_call_virt(p, f, args); \ > + efi_call_virt_check_flags(flags, __stringify(f)); \ > + \ > + arch_efi_call_virt_teardown(); \ > + \ > + __s; \ > +}) > + > +#define __efi_call_virt_pointer(p, f, args...) \ > +({ \ > + unsigned long flags; \ > + \ > + arch_efi_call_virt_setup(); \ > + \ > + local_save_flags(flags); \ > + arch_efi_call_virt(p, f, args); \ > + efi_call_virt_check_flags(flags, __stringify(f)); \ > + \ > + arch_efi_call_virt_teardown(); \ > +}) Note that while at it I renamed 'flags' to '__flags' because 'flags' is a commonly used variable name and the 'efi_status_t __s' variable was macro-prefixed already. Any objections? Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html