* [PATCH v2] x86/efi: Clean up a minor mistake in code comment
@ 2017-03-24 9:17 ` Baoquan He
0 siblings, 0 replies; 6+ messages in thread
From: Baoquan He @ 2017-03-24 9:17 UTC (permalink / raw)
To: ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A,
matt-mF/unelCI9GS6iBeEJttW/XRex20P6io
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-efi-u79uwXL29TY76Z2rM5mHXA, Baoquan He
EFI allocate runtime services regions from EFI_VA_START, -4G, down
to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
Clean it up to avoid confusion.
Signed-off-by: Baoquan He <bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
v1->v2:
Update patch log to corret a description mistake Ard pointed out.
arch/x86/platform/efi/efi_64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/platform/efi/efi_64.c b/arch/x86/platform/efi/efi_64.c
index a4695da..6cbf9e0 100644
--- a/arch/x86/platform/efi/efi_64.c
+++ b/arch/x86/platform/efi/efi_64.c
@@ -47,7 +47,7 @@
#include <asm/pgalloc.h>
/*
- * We allocate runtime services regions bottom-up, starting from -4G, i.e.
+ * We allocate runtime services regions top-down, starting from -4G, i.e.
* 0xffff_ffff_0000_0000 and limit EFI VA mapping space to 64G.
*/
static u64 efi_va = EFI_VA_START;
--
2.5.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH v2] x86/efi: Clean up a minor mistake in code comment
@ 2017-03-24 9:17 ` Baoquan He
0 siblings, 0 replies; 6+ messages in thread
From: Baoquan He @ 2017-03-24 9:17 UTC (permalink / raw)
To: ard.biesheuvel, matt; +Cc: linux-kernel, linux-efi, Baoquan He
EFI allocate runtime services regions from EFI_VA_START, -4G, down
to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
Clean it up to avoid confusion.
Signed-off-by: Baoquan He <bhe@redhat.com>
---
v1->v2:
Update patch log to corret a description mistake Ard pointed out.
arch/x86/platform/efi/efi_64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/platform/efi/efi_64.c b/arch/x86/platform/efi/efi_64.c
index a4695da..6cbf9e0 100644
--- a/arch/x86/platform/efi/efi_64.c
+++ b/arch/x86/platform/efi/efi_64.c
@@ -47,7 +47,7 @@
#include <asm/pgalloc.h>
/*
- * We allocate runtime services regions bottom-up, starting from -4G, i.e.
+ * We allocate runtime services regions top-down, starting from -4G, i.e.
* 0xffff_ffff_0000_0000 and limit EFI VA mapping space to 64G.
*/
static u64 efi_va = EFI_VA_START;
--
2.5.5
^ permalink raw reply related [flat|nested] 6+ messages in thread[parent not found: <1490347040-28720-1-git-send-email-bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v2] x86/efi: Clean up a minor mistake in code comment
2017-03-24 9:17 ` Baoquan He
@ 2017-03-24 9:43 ` Ard Biesheuvel
-1 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-03-24 9:43 UTC (permalink / raw)
To: Baoquan He
Cc: Matt Fleming,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
On 24 March 2017 at 09:17, Baoquan He <bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> EFI allocate runtime services regions from EFI_VA_START, -4G, down
> to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
> commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
>
> Clean it up to avoid confusion.
>
> Signed-off-by: Baoquan He <bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> ---
> v1->v2:
> Update patch log to corret a description mistake Ard pointed out.
>
Queued in efi/next for v4.12
Thanks,
Ard.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] x86/efi: Clean up a minor mistake in code comment
@ 2017-03-24 9:43 ` Ard Biesheuvel
0 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2017-03-24 9:43 UTC (permalink / raw)
To: Baoquan He
Cc: Matt Fleming, linux-kernel@vger.kernel.org,
linux-efi@vger.kernel.org
On 24 March 2017 at 09:17, Baoquan He <bhe@redhat.com> wrote:
> EFI allocate runtime services regions from EFI_VA_START, -4G, down
> to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
> commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
>
> Clean it up to avoid confusion.
>
> Signed-off-by: Baoquan He <bhe@redhat.com>
> ---
> v1->v2:
> Update patch log to corret a description mistake Ard pointed out.
>
Queued in efi/next for v4.12
Thanks,
Ard.
^ permalink raw reply [flat|nested] 6+ messages in thread[parent not found: <CAKv+Gu_ojQ48eTqqd3m9SxqqLygVPFGAUWJHxjznr8t6WYbygQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH v2] x86/efi: Clean up a minor mistake in code comment
2017-03-24 9:43 ` Ard Biesheuvel
@ 2017-03-24 9:50 ` Baoquan He
-1 siblings, 0 replies; 6+ messages in thread
From: Baoquan He @ 2017-03-24 9:50 UTC (permalink / raw)
To: Ard Biesheuvel
Cc: Matt Fleming,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
On 03/24/17 at 09:43am, Ard Biesheuvel wrote:
> On 24 March 2017 at 09:17, Baoquan He <bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> > EFI allocate runtime services regions from EFI_VA_START, -4G, down
> > to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
> > commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
> >
> > Clean it up to avoid confusion.
> >
> > Signed-off-by: Baoquan He <bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> > ---
> > v1->v2:
> > Update patch log to corret a description mistake Ard pointed out.
> >
>
> Queued in efi/next for v4.12
Thanks, Ard!
Baoquan
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] x86/efi: Clean up a minor mistake in code comment
@ 2017-03-24 9:50 ` Baoquan He
0 siblings, 0 replies; 6+ messages in thread
From: Baoquan He @ 2017-03-24 9:50 UTC (permalink / raw)
To: Ard Biesheuvel
Cc: Matt Fleming, linux-kernel@vger.kernel.org,
linux-efi@vger.kernel.org
On 03/24/17 at 09:43am, Ard Biesheuvel wrote:
> On 24 March 2017 at 09:17, Baoquan He <bhe@redhat.com> wrote:
> > EFI allocate runtime services regions from EFI_VA_START, -4G, down
> > to -68G, EFI_VA_END, 64G altogether. The mechanism was introduced in
> > commit d2f7cbe7b26a7 ("x86/efi: Runtime services virtual mapping").
> >
> > Clean it up to avoid confusion.
> >
> > Signed-off-by: Baoquan He <bhe@redhat.com>
> > ---
> > v1->v2:
> > Update patch log to corret a description mistake Ard pointed out.
> >
>
> Queued in efi/next for v4.12
Thanks, Ard!
Baoquan
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-24 10:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-24 9:17 [PATCH v2] x86/efi: Clean up a minor mistake in code comment Baoquan He
2017-03-24 9:17 ` Baoquan He
[not found] ` <1490347040-28720-1-git-send-email-bhe-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-03-24 9:43 ` Ard Biesheuvel
2017-03-24 9:43 ` Ard Biesheuvel
[not found] ` <CAKv+Gu_ojQ48eTqqd3m9SxqqLygVPFGAUWJHxjznr8t6WYbygQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-24 9:50 ` Baoquan He
2017-03-24 9:50 ` Baoquan He
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.