2022-11-10 20:41:37

by Sean Christopherson

[permalink] [raw]
Subject: [PATCH v2 3/5] x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names

Rename the CPU entry area variables in kasan_init() to shorten their
names, a future fix will reference the beginning of the per-CPU portion
of the CPU entry area, and shadow_cpu_entry_per_cpu_begin is a bit much.

No functional change intended.

Signed-off-by: Sean Christopherson <[email protected]>
---
arch/x86/mm/kasan_init_64.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/x86/mm/kasan_init_64.c b/arch/x86/mm/kasan_init_64.c
index d1416926ad52..ad7872ae10ed 100644
--- a/arch/x86/mm/kasan_init_64.c
+++ b/arch/x86/mm/kasan_init_64.c
@@ -331,7 +331,7 @@ void __init kasan_populate_shadow_for_vaddr(void *va, size_t size, int nid)
void __init kasan_init(void)
{
int i;
- void *shadow_cpu_entry_begin, *shadow_cpu_entry_end;
+ void *shadow_cea_begin, *shadow_cea_end;

memcpy(early_top_pgt, init_top_pgt, sizeof(early_top_pgt));

@@ -372,16 +372,16 @@ void __init kasan_init(void)
map_range(&pfn_mapped[i]);
}

- shadow_cpu_entry_begin = (void *)CPU_ENTRY_AREA_BASE;
- shadow_cpu_entry_begin = kasan_mem_to_shadow(shadow_cpu_entry_begin);
- shadow_cpu_entry_begin = (void *)round_down(
- (unsigned long)shadow_cpu_entry_begin, PAGE_SIZE);
+ shadow_cea_begin = (void *)CPU_ENTRY_AREA_BASE;
+ shadow_cea_begin = kasan_mem_to_shadow(shadow_cea_begin);
+ shadow_cea_begin = (void *)round_down(
+ (unsigned long)shadow_cea_begin, PAGE_SIZE);

- shadow_cpu_entry_end = (void *)(CPU_ENTRY_AREA_BASE +
+ shadow_cea_end = (void *)(CPU_ENTRY_AREA_BASE +
CPU_ENTRY_AREA_MAP_SIZE);
- shadow_cpu_entry_end = kasan_mem_to_shadow(shadow_cpu_entry_end);
- shadow_cpu_entry_end = (void *)round_up(
- (unsigned long)shadow_cpu_entry_end, PAGE_SIZE);
+ shadow_cea_end = kasan_mem_to_shadow(shadow_cea_end);
+ shadow_cea_end = (void *)round_up(
+ (unsigned long)shadow_cea_end, PAGE_SIZE);

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)PAGE_OFFSET + MAXMEM),
@@ -403,9 +403,9 @@ void __init kasan_init(void)

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)VMALLOC_END + 1),
- shadow_cpu_entry_begin);
+ shadow_cea_begin);

- kasan_populate_early_shadow(shadow_cpu_entry_end,
+ kasan_populate_early_shadow(shadow_cea_end,
kasan_mem_to_shadow((void *)__START_KERNEL_map));

kasan_populate_shadow((unsigned long)kasan_mem_to_shadow(_stext),
--
2.38.1.431.g37b22c650d-goog



2022-11-14 14:37:43

by Andrey Ryabinin

[permalink] [raw]
Subject: Re: [PATCH v2 3/5] x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names



On 11/10/22 23:35, Sean Christopherson wrote:
> Rename the CPU entry area variables in kasan_init() to shorten their
> names, a future fix will reference the beginning of the per-CPU portion
> of the CPU entry area, and shadow_cpu_entry_per_cpu_begin is a bit much.
>
> No functional change intended.
>
> Signed-off-by: Sean Christopherson <[email protected]>
> ---
> arch/x86/mm/kasan_init_64.c | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>

Reviewed-by: Andrey Ryabinin <[email protected]>

2022-11-15 22:37:30

by tip-bot2 for Jacob Pan

[permalink] [raw]
Subject: [tip: x86/mm] x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names

The following commit has been merged into the x86/mm branch of tip:

Commit-ID: e93cc3aa893e74ea391e06aba09cc4bf523c12c8
Gitweb: https://git.kernel.org/tip/e93cc3aa893e74ea391e06aba09cc4bf523c12c8
Author: Sean Christopherson <[email protected]>
AuthorDate: Thu, 10 Nov 2022 20:35:02
Committer: Peter Zijlstra <[email protected]>
CommitterDate: Tue, 15 Nov 2022 22:29:59 +01:00

x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names

Rename the CPU entry area variables in kasan_init() to shorten their
names, a future fix will reference the beginning of the per-CPU portion
of the CPU entry area, and shadow_cpu_entry_per_cpu_begin is a bit much.

No functional change intended.

Signed-off-by: Sean Christopherson <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Reviewed-by: Andrey Ryabinin <[email protected]>
Link: https://lkml.kernel.org/r/[email protected]
---
arch/x86/mm/kasan_init_64.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/x86/mm/kasan_init_64.c b/arch/x86/mm/kasan_init_64.c
index d141692..ad7872a 100644
--- a/arch/x86/mm/kasan_init_64.c
+++ b/arch/x86/mm/kasan_init_64.c
@@ -331,7 +331,7 @@ void __init kasan_populate_shadow_for_vaddr(void *va, size_t size, int nid)
void __init kasan_init(void)
{
int i;
- void *shadow_cpu_entry_begin, *shadow_cpu_entry_end;
+ void *shadow_cea_begin, *shadow_cea_end;

memcpy(early_top_pgt, init_top_pgt, sizeof(early_top_pgt));

@@ -372,16 +372,16 @@ void __init kasan_init(void)
map_range(&pfn_mapped[i]);
}

- shadow_cpu_entry_begin = (void *)CPU_ENTRY_AREA_BASE;
- shadow_cpu_entry_begin = kasan_mem_to_shadow(shadow_cpu_entry_begin);
- shadow_cpu_entry_begin = (void *)round_down(
- (unsigned long)shadow_cpu_entry_begin, PAGE_SIZE);
+ shadow_cea_begin = (void *)CPU_ENTRY_AREA_BASE;
+ shadow_cea_begin = kasan_mem_to_shadow(shadow_cea_begin);
+ shadow_cea_begin = (void *)round_down(
+ (unsigned long)shadow_cea_begin, PAGE_SIZE);

- shadow_cpu_entry_end = (void *)(CPU_ENTRY_AREA_BASE +
+ shadow_cea_end = (void *)(CPU_ENTRY_AREA_BASE +
CPU_ENTRY_AREA_MAP_SIZE);
- shadow_cpu_entry_end = kasan_mem_to_shadow(shadow_cpu_entry_end);
- shadow_cpu_entry_end = (void *)round_up(
- (unsigned long)shadow_cpu_entry_end, PAGE_SIZE);
+ shadow_cea_end = kasan_mem_to_shadow(shadow_cea_end);
+ shadow_cea_end = (void *)round_up(
+ (unsigned long)shadow_cea_end, PAGE_SIZE);

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)PAGE_OFFSET + MAXMEM),
@@ -403,9 +403,9 @@ void __init kasan_init(void)

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)VMALLOC_END + 1),
- shadow_cpu_entry_begin);
+ shadow_cea_begin);

- kasan_populate_early_shadow(shadow_cpu_entry_end,
+ kasan_populate_early_shadow(shadow_cea_end,
kasan_mem_to_shadow((void *)__START_KERNEL_map));

kasan_populate_shadow((unsigned long)kasan_mem_to_shadow(_stext),

2022-12-17 19:09:09

by tip-bot2 for Jacob Pan

[permalink] [raw]
Subject: [tip: x86/mm] x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names

The following commit has been merged into the x86/mm branch of tip:

Commit-ID: 7077d2ccb94dafd00b29cc2d601c9f6891648f5b
Gitweb: https://git.kernel.org/tip/7077d2ccb94dafd00b29cc2d601c9f6891648f5b
Author: Sean Christopherson <[email protected]>
AuthorDate: Thu, 10 Nov 2022 20:35:02
Committer: Dave Hansen <[email protected]>
CommitterDate: Thu, 15 Dec 2022 10:37:28 -08:00

x86/kasan: Rename local CPU_ENTRY_AREA variables to shorten names

Rename the CPU entry area variables in kasan_init() to shorten their
names, a future fix will reference the beginning of the per-CPU portion
of the CPU entry area, and shadow_cpu_entry_per_cpu_begin is a bit much.

No functional change intended.

Signed-off-by: Sean Christopherson <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Reviewed-by: Andrey Ryabinin <[email protected]>
Link: https://lkml.kernel.org/r/[email protected]
---
arch/x86/mm/kasan_init_64.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/x86/mm/kasan_init_64.c b/arch/x86/mm/kasan_init_64.c
index d141692..ad7872a 100644
--- a/arch/x86/mm/kasan_init_64.c
+++ b/arch/x86/mm/kasan_init_64.c
@@ -331,7 +331,7 @@ void __init kasan_populate_shadow_for_vaddr(void *va, size_t size, int nid)
void __init kasan_init(void)
{
int i;
- void *shadow_cpu_entry_begin, *shadow_cpu_entry_end;
+ void *shadow_cea_begin, *shadow_cea_end;

memcpy(early_top_pgt, init_top_pgt, sizeof(early_top_pgt));

@@ -372,16 +372,16 @@ void __init kasan_init(void)
map_range(&pfn_mapped[i]);
}

- shadow_cpu_entry_begin = (void *)CPU_ENTRY_AREA_BASE;
- shadow_cpu_entry_begin = kasan_mem_to_shadow(shadow_cpu_entry_begin);
- shadow_cpu_entry_begin = (void *)round_down(
- (unsigned long)shadow_cpu_entry_begin, PAGE_SIZE);
+ shadow_cea_begin = (void *)CPU_ENTRY_AREA_BASE;
+ shadow_cea_begin = kasan_mem_to_shadow(shadow_cea_begin);
+ shadow_cea_begin = (void *)round_down(
+ (unsigned long)shadow_cea_begin, PAGE_SIZE);

- shadow_cpu_entry_end = (void *)(CPU_ENTRY_AREA_BASE +
+ shadow_cea_end = (void *)(CPU_ENTRY_AREA_BASE +
CPU_ENTRY_AREA_MAP_SIZE);
- shadow_cpu_entry_end = kasan_mem_to_shadow(shadow_cpu_entry_end);
- shadow_cpu_entry_end = (void *)round_up(
- (unsigned long)shadow_cpu_entry_end, PAGE_SIZE);
+ shadow_cea_end = kasan_mem_to_shadow(shadow_cea_end);
+ shadow_cea_end = (void *)round_up(
+ (unsigned long)shadow_cea_end, PAGE_SIZE);

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)PAGE_OFFSET + MAXMEM),
@@ -403,9 +403,9 @@ void __init kasan_init(void)

kasan_populate_early_shadow(
kasan_mem_to_shadow((void *)VMALLOC_END + 1),
- shadow_cpu_entry_begin);
+ shadow_cea_begin);

- kasan_populate_early_shadow(shadow_cpu_entry_end,
+ kasan_populate_early_shadow(shadow_cea_end,
kasan_mem_to_shadow((void *)__START_KERNEL_map));

kasan_populate_shadow((unsigned long)kasan_mem_to_shadow(_stext),