* [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt [not found] <1461238529-12810-1-git-send-email-mark.rutland@arm.com> @ 2016-04-21 11:35 ` Mark Rutland 2016-04-21 16:48 ` Will Deacon 2016-04-21 11:35 ` [PATCHv2 3/5] arm/efi: " Mark Rutland 1 sibling, 1 reply; 4+ messages in thread From: Mark Rutland @ 2016-04-21 11:35 UTC (permalink / raw) To: linux-arm-kernel Now there's a common template for {__,}efi_call_virt, remove the duplicate logic from the arm64 efi code. Signed-off-by: Mark Rutland <mark.rutland@arm.com> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Matt Fleming <matt@codeblueprint.co.uk> Cc: Will Deacon <will.deacon@arm.com> Cc: linux-arm-kernel at lists.infradead.org Cc: linux-efi at vger.kernel.org Cc: linux-kernel at vger.kernel.org --- arch/arm64/include/asm/efi.h | 21 +++++++-------------- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/arch/arm64/include/asm/efi.h b/arch/arm64/include/asm/efi.h index 8e88a69..f4f71224 100644 --- a/arch/arm64/include/asm/efi.h +++ b/arch/arm64/include/asm/efi.h @@ -14,28 +14,21 @@ extern void efi_init(void); int efi_create_mapping(struct mm_struct *mm, efi_memory_desc_t *md); -#define efi_call_virt(f, ...) \ +#define arch_efi_call_virt_setup() \ ({ \ - efi_##f##_t *__f; \ - efi_status_t __s; \ - \ kernel_neon_begin(); \ efi_virtmap_load(); \ - __f = efi.systab->runtime->f; \ - __s = __f(__VA_ARGS__); \ - efi_virtmap_unload(); \ - kernel_neon_end(); \ - __s; \ }) -#define __efi_call_virt(f, ...) \ +#define arch_efi_call_virt(f, args...) \ ({ \ efi_##f##_t *__f; \ - \ - kernel_neon_begin(); \ - efi_virtmap_load(); \ __f = efi.systab->runtime->f; \ - __f(__VA_ARGS__); \ + __f(args); \ +}) + +#define arch_efi_call_virt_teardown() \ +({ \ efi_virtmap_unload(); \ kernel_neon_end(); \ }) -- 1.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt 2016-04-21 11:35 ` [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt Mark Rutland @ 2016-04-21 16:48 ` Will Deacon 2016-04-21 16:58 ` Mark Rutland 0 siblings, 1 reply; 4+ messages in thread From: Will Deacon @ 2016-04-21 16:48 UTC (permalink / raw) To: linux-arm-kernel On Thu, Apr 21, 2016 at 12:35:26PM +0100, Mark Rutland wrote: > Now there's a common template for {__,}efi_call_virt, remove the > duplicate logic from the arm64 efi code. > > Signed-off-by: Mark Rutland <mark.rutland@arm.com> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Leif Lindholm <leif.lindholm@linaro.org> > Cc: Matt Fleming <matt@codeblueprint.co.uk> > Cc: Will Deacon <will.deacon@arm.com> > Cc: linux-arm-kernel at lists.infradead.org > Cc: linux-efi at vger.kernel.org > Cc: linux-kernel at vger.kernel.org > --- > arch/arm64/include/asm/efi.h | 21 +++++++-------------- > 1 file changed, 7 insertions(+), 14 deletions(-) > > diff --git a/arch/arm64/include/asm/efi.h b/arch/arm64/include/asm/efi.h > index 8e88a69..f4f71224 100644 > --- a/arch/arm64/include/asm/efi.h > +++ b/arch/arm64/include/asm/efi.h > @@ -14,28 +14,21 @@ extern void efi_init(void); > > int efi_create_mapping(struct mm_struct *mm, efi_memory_desc_t *md); > > -#define efi_call_virt(f, ...) \ > +#define arch_efi_call_virt_setup() \ > ({ \ > - efi_##f##_t *__f; \ > - efi_status_t __s; \ > - \ > kernel_neon_begin(); \ > efi_virtmap_load(); \ > - __f = efi.systab->runtime->f; \ > - __s = __f(__VA_ARGS__); \ > - efi_virtmap_unload(); \ > - kernel_neon_end(); \ > - __s; \ > }) > > -#define __efi_call_virt(f, ...) \ > +#define arch_efi_call_virt(f, args...) \ > ({ \ > efi_##f##_t *__f; \ > - \ > - kernel_neon_begin(); \ > - efi_virtmap_load(); \ > __f = efi.systab->runtime->f; \ > - __f(__VA_ARGS__); \ > + __f(args); \ Any reason to change this to a named argument? This patch is hard enough to review as it is, given the way the diff has been generated! Either way: Acked-by: Will Deacon <will.deacon@arm.com> Will ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt 2016-04-21 16:48 ` Will Deacon @ 2016-04-21 16:58 ` Mark Rutland 0 siblings, 0 replies; 4+ messages in thread From: Mark Rutland @ 2016-04-21 16:58 UTC (permalink / raw) To: linux-arm-kernel On Thu, Apr 21, 2016 at 05:48:40PM +0100, Will Deacon wrote: > On Thu, Apr 21, 2016 at 12:35:26PM +0100, Mark Rutland wrote: > > Now there's a common template for {__,}efi_call_virt, remove the > > duplicate logic from the arm64 efi code. > > > > Signed-off-by: Mark Rutland <mark.rutland@arm.com> > > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> > > Cc: Catalin Marinas <catalin.marinas@arm.com> > > Cc: Leif Lindholm <leif.lindholm@linaro.org> > > Cc: Matt Fleming <matt@codeblueprint.co.uk> > > Cc: Will Deacon <will.deacon@arm.com> > > Cc: linux-arm-kernel at lists.infradead.org > > Cc: linux-efi at vger.kernel.org > > Cc: linux-kernel at vger.kernel.org > > --- > > arch/arm64/include/asm/efi.h | 21 +++++++-------------- > > 1 file changed, 7 insertions(+), 14 deletions(-) > > > > diff --git a/arch/arm64/include/asm/efi.h b/arch/arm64/include/asm/efi.h > > index 8e88a69..f4f71224 100644 > > --- a/arch/arm64/include/asm/efi.h > > +++ b/arch/arm64/include/asm/efi.h > > @@ -14,28 +14,21 @@ extern void efi_init(void); > > > > int efi_create_mapping(struct mm_struct *mm, efi_memory_desc_t *md); > > > > -#define efi_call_virt(f, ...) \ > > +#define arch_efi_call_virt_setup() \ > > ({ \ > > - efi_##f##_t *__f; \ > > - efi_status_t __s; \ > > - \ > > kernel_neon_begin(); \ > > efi_virtmap_load(); \ > > - __f = efi.systab->runtime->f; \ > > - __s = __f(__VA_ARGS__); \ > > - efi_virtmap_unload(); \ > > - kernel_neon_end(); \ > > - __s; \ > > }) > > > > -#define __efi_call_virt(f, ...) \ > > +#define arch_efi_call_virt(f, args...) \ > > ({ \ > > efi_##f##_t *__f; \ > > - \ > > - kernel_neon_begin(); \ > > - efi_virtmap_load(); \ > > __f = efi.systab->runtime->f; \ > > - __f(__VA_ARGS__); \ > > + __f(args); \ > > Any reason to change this to a named argument? This patch is hard enough > to review as it is, given the way the diff has been generated! That was to enforce consistency across arm/arm64/x86 for this. It seemed nicer to make them all use a named argument than it did to make them all use __VA_ARGS__, either of which was nice than each of them doing something slightly different. Sorry for the pain that evidently caused! > Either way: > > Acked-by: Will Deacon <will.deacon@arm.com> Cheers! Mark. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCHv2 3/5] arm/efi: move to generic {__,}efi_call_virt [not found] <1461238529-12810-1-git-send-email-mark.rutland@arm.com> 2016-04-21 11:35 ` [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt Mark Rutland @ 2016-04-21 11:35 ` Mark Rutland 1 sibling, 0 replies; 4+ messages in thread From: Mark Rutland @ 2016-04-21 11:35 UTC (permalink / raw) To: linux-arm-kernel Now there's a common template for {__,}efi_call_virt, remove the duplicate logic from the arm efi code. Signed-off-by: Mark Rutland <mark.rutland@arm.com> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Matt Fleming <matt@codeblueprint.co.uk> Cc: Russell King <linux@arm.linux.org.uk> Cc: linux-arm-kernel at lists.infradead.org Cc: linux-efi at vger.kernel.org Cc: linux-kernel at vger.kernel.org --- arch/arm/include/asm/efi.h | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/arch/arm/include/asm/efi.h b/arch/arm/include/asm/efi.h index e0eea72..e1461fc 100644 --- a/arch/arm/include/asm/efi.h +++ b/arch/arm/include/asm/efi.h @@ -23,26 +23,14 @@ void efi_init(void); int efi_create_mapping(struct mm_struct *mm, efi_memory_desc_t *md); -#define efi_call_virt(f, ...) \ -({ \ - efi_##f##_t *__f; \ - efi_status_t __s; \ - \ - efi_virtmap_load(); \ - __f = efi.systab->runtime->f; \ - __s = __f(__VA_ARGS__); \ - efi_virtmap_unload(); \ - __s; \ -}) +#define arch_efi_call_virt_setup() efi_virtmap_load() +#define arch_efi_call_virt_teardown() efi_virtmap_unload() -#define __efi_call_virt(f, ...) \ +#define arch_efi_call_virt(f, args...) \ ({ \ efi_##f##_t *__f; \ - \ - efi_virtmap_load(); \ __f = efi.systab->runtime->f; \ - __f(__VA_ARGS__); \ - efi_virtmap_unload(); \ + __f(args); \ }) static inline void efi_set_pgd(struct mm_struct *mm) -- 1.9.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-04-21 16:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1461238529-12810-1-git-send-email-mark.rutland@arm.com>
2016-04-21 11:35 ` [PATCHv2 2/5] arm64/efi: move to generic {__,}efi_call_virt Mark Rutland
2016-04-21 16:48 ` Will Deacon
2016-04-21 16:58 ` Mark Rutland
2016-04-21 11:35 ` [PATCHv2 3/5] arm/efi: " Mark Rutland
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).