From: Matt Fleming <matt@codeblueprint.co.uk>
To: Alex Thorlton <athorlton@sgi.com>
Cc: linux-kernel@vger.kernel.org, Russ Anderson <rja@sgi.com>,
Dimitri Sivanich <sivanich@sgi.com>,
Russell King <linux@armlinux.org.uk>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Mark Rutland <mark.rutland@arm.com>,
Roy Franz <roy.franz@linaro.org>,
linux-arm-kernel@lists.infradead.org, linux-efi@vger.kernel.org,
x86@kernel.org
Subject: Re: [PATCHv2 0/3] x86/UV, x86/efi: Re-factor efi_call_virt for general use
Date: Thu, 23 Jun 2016 13:44:33 +0100 [thread overview]
Message-ID: <20160623124433.GE8415@codeblueprint.co.uk> (raw)
In-Reply-To: <1466017455-236934-1-git-send-email-athorlton@sgi.com>
On Wed, 15 Jun, at 02:04:12PM, Alex Thorlton wrote:
> Hey guys,
>
> This patchset creates a general purpose version of the efi_call_virt
> macro that does not assume that the function pointer being passed in is
> inside of efi.systab->runtime. It also fixes up uv_bios_call to use the
> new functionality, and does a bit of cleanup in the efi_thunk macro.
>
> Quick breakdown of the patches:
>
> Patch 1) Move necessary macros to locations where we can access them.
> Remove hard-coded efi.systab reference from efi_call_virt.
> Rename/create new macros as needed.
> Patch 2) Simple change to allow UV code to utilize the new
> functionality. Included a detailed explanation of how we got
> here.
> Patch 3) Replace a few bits of the efi_thunk macro with the
> arch_efi_call_setup/teardown macros.
>
> The first two have been tested on simulators and hardware, but the third
> has only been compile-tested. I don't have any hardware to test that
> on.
>
> Updates for v2:
>
> - Fix up arm and arm64 versions of arch_efi_call_virt. I missed these
> on my first pass
> - Add some more detail to the commit message for the uv_bios_call fix.
> - Change the third patch to use the arch_efi_call_setup/teardown macros
> inside of the efi_thunk macro, instead of replacing efi_thunk entirely
> for the CONFIG_EFI_MIXED case.
Whole series looks good to me Alex, and everything works fine on my
mixed-mode config. Applied for v4.8, thanks!
WARNING: multiple messages have this Message-ID (diff)
From: Matt Fleming <matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
To: Alex Thorlton <athorlton-sJ/iWh9BUns@public.gmane.org>
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Russ Anderson <rja-sJ/iWh9BUns@public.gmane.org>,
Dimitri Sivanich <sivanich-sJ/iWh9BUns@public.gmane.org>,
Russell King <linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>,
Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>,
Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>,
Thomas Gleixner <tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>,
Ingo Molnar <mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
"H. Peter Anvin" <hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org>,
Ard Biesheuvel
<ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Roy Franz <roy.franz-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Subject: Re: [PATCHv2 0/3] x86/UV, x86/efi: Re-factor efi_call_virt for general use
Date: Thu, 23 Jun 2016 13:44:33 +0100 [thread overview]
Message-ID: <20160623124433.GE8415@codeblueprint.co.uk> (raw)
In-Reply-To: <1466017455-236934-1-git-send-email-athorlton-sJ/iWh9BUns@public.gmane.org>
On Wed, 15 Jun, at 02:04:12PM, Alex Thorlton wrote:
> Hey guys,
>
> This patchset creates a general purpose version of the efi_call_virt
> macro that does not assume that the function pointer being passed in is
> inside of efi.systab->runtime. It also fixes up uv_bios_call to use the
> new functionality, and does a bit of cleanup in the efi_thunk macro.
>
> Quick breakdown of the patches:
>
> Patch 1) Move necessary macros to locations where we can access them.
> Remove hard-coded efi.systab reference from efi_call_virt.
> Rename/create new macros as needed.
> Patch 2) Simple change to allow UV code to utilize the new
> functionality. Included a detailed explanation of how we got
> here.
> Patch 3) Replace a few bits of the efi_thunk macro with the
> arch_efi_call_setup/teardown macros.
>
> The first two have been tested on simulators and hardware, but the third
> has only been compile-tested. I don't have any hardware to test that
> on.
>
> Updates for v2:
>
> - Fix up arm and arm64 versions of arch_efi_call_virt. I missed these
> on my first pass
> - Add some more detail to the commit message for the uv_bios_call fix.
> - Change the third patch to use the arch_efi_call_setup/teardown macros
> inside of the efi_thunk macro, instead of replacing efi_thunk entirely
> for the CONFIG_EFI_MIXED case.
Whole series looks good to me Alex, and everything works fine on my
mixed-mode config. Applied for v4.8, thanks!
WARNING: multiple messages have this Message-ID (diff)
From: matt@codeblueprint.co.uk (Matt Fleming)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2 0/3] x86/UV, x86/efi: Re-factor efi_call_virt for general use
Date: Thu, 23 Jun 2016 13:44:33 +0100 [thread overview]
Message-ID: <20160623124433.GE8415@codeblueprint.co.uk> (raw)
In-Reply-To: <1466017455-236934-1-git-send-email-athorlton@sgi.com>
On Wed, 15 Jun, at 02:04:12PM, Alex Thorlton wrote:
> Hey guys,
>
> This patchset creates a general purpose version of the efi_call_virt
> macro that does not assume that the function pointer being passed in is
> inside of efi.systab->runtime. It also fixes up uv_bios_call to use the
> new functionality, and does a bit of cleanup in the efi_thunk macro.
>
> Quick breakdown of the patches:
>
> Patch 1) Move necessary macros to locations where we can access them.
> Remove hard-coded efi.systab reference from efi_call_virt.
> Rename/create new macros as needed.
> Patch 2) Simple change to allow UV code to utilize the new
> functionality. Included a detailed explanation of how we got
> here.
> Patch 3) Replace a few bits of the efi_thunk macro with the
> arch_efi_call_setup/teardown macros.
>
> The first two have been tested on simulators and hardware, but the third
> has only been compile-tested. I don't have any hardware to test that
> on.
>
> Updates for v2:
>
> - Fix up arm and arm64 versions of arch_efi_call_virt. I missed these
> on my first pass
> - Add some more detail to the commit message for the uv_bios_call fix.
> - Change the third patch to use the arch_efi_call_setup/teardown macros
> inside of the efi_thunk macro, instead of replacing efi_thunk entirely
> for the CONFIG_EFI_MIXED case.
Whole series looks good to me Alex, and everything works fine on my
mixed-mode config. Applied for v4.8, thanks!
next prev parent reply other threads:[~2016-06-23 12:44 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-15 19:04 [PATCHv2 0/3] x86/UV, x86/efi: Re-factor efi_call_virt for general use Alex Thorlton
2016-06-15 19:04 ` Alex Thorlton
2016-06-15 19:04 ` [PATCH 1/3] Convert efi_call_virt to efi_call_virt_pointer Alex Thorlton
2016-06-15 19:04 ` Alex Thorlton
2016-06-15 19:04 ` [PATCH 2/3] Update uv_bios_call to use efi_call_virt_pointer Alex Thorlton
2016-06-15 19:04 ` Alex Thorlton
2016-06-15 19:04 ` [PATCH 3/3] Update efi_thunk to use the the arch_efi_call_virt* macros Alex Thorlton
2016-06-15 19:04 ` Alex Thorlton
2016-06-15 19:04 ` Alex Thorlton
2016-06-23 12:44 ` Matt Fleming [this message]
2016-06-23 12:44 ` [PATCHv2 0/3] x86/UV, x86/efi: Re-factor efi_call_virt for general use Matt Fleming
2016-06-23 12:44 ` Matt Fleming
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160623124433.GE8415@codeblueprint.co.uk \
--to=matt@codeblueprint.co.uk \
--cc=ard.biesheuvel@linaro.org \
--cc=athorlton@sgi.com \
--cc=catalin.marinas@arm.com \
--cc=hpa@zytor.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=mingo@redhat.com \
--cc=rja@sgi.com \
--cc=roy.franz@linaro.org \
--cc=sivanich@sgi.com \
--cc=tglx@linutronix.de \
--cc=will.deacon@arm.com \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.