diff mbox series

[v2,3/4] xen: arm: Moved domain_page.c to MMU specific

Message ID 20240808120936.3299937-4-ayan.kumar.halder@amd.com (mailing list archive)
State Superseded
Headers show
Series xen: arm: Split MMU code in preparation for MPU work (part 2) | expand

Commit Message

Ayan Kumar Halder Aug. 8, 2024, 12:09 p.m. UTC
domain_page.c has been moved to mmu/domain_page.c. The reason
being this file implements MMU specific logic for mapping,
unmapping, etc domain page.

Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@amd.com>
---
 xen/arch/arm/mmu/Makefile            | 1 +
 xen/arch/arm/{ => mmu}/domain_page.c | 0
 2 files changed, 1 insertion(+)
 rename xen/arch/arm/{ => mmu}/domain_page.c (100%)

Comments

Jan Beulich Aug. 8, 2024, 12:51 p.m. UTC | #1
On 08.08.2024 14:09, Ayan Kumar Halder wrote:
> domain_page.c has been moved to mmu/domain_page.c. The reason
> being this file implements MMU specific logic for mapping,
> unmapping, etc domain page.
> 
> Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@amd.com>
> ---
>  xen/arch/arm/mmu/Makefile            | 1 +
>  xen/arch/arm/{ => mmu}/domain_page.c | 0
>  2 files changed, 1 insertion(+)
>  rename xen/arch/arm/{ => mmu}/domain_page.c (100%)
> 
> diff --git a/xen/arch/arm/mmu/Makefile b/xen/arch/arm/mmu/Makefile
> index 67475fcd80..2cb44b857d 100644
> --- a/xen/arch/arm/mmu/Makefile
> +++ b/xen/arch/arm/mmu/Makefile
> @@ -2,3 +2,4 @@ obj-y += p2m.o
>  obj-y += pt.o
>  obj-y += setup.o
>  obj-y += smpboot.o
> +obj-$(CONFIG_ARCH_MAP_DOMAIN_PAGE) += domain_page.o

Don't you then also need to remove the respective line from arm/Makefile?

Jan
Ayan Kumar Halder Aug. 8, 2024, 4:01 p.m. UTC | #2
Hi Jan,

On 08/08/2024 13:51, Jan Beulich wrote:
> On 08.08.2024 14:09, Ayan Kumar Halder wrote:
>> domain_page.c has been moved to mmu/domain_page.c. The reason
>> being this file implements MMU specific logic for mapping,
>> unmapping, etc domain page.
>>
>> Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@amd.com>
>> ---
>>   xen/arch/arm/mmu/Makefile            | 1 +
>>   xen/arch/arm/{ => mmu}/domain_page.c | 0
>>   2 files changed, 1 insertion(+)
>>   rename xen/arch/arm/{ => mmu}/domain_page.c (100%)
>>
>> diff --git a/xen/arch/arm/mmu/Makefile b/xen/arch/arm/mmu/Makefile
>> index 67475fcd80..2cb44b857d 100644
>> --- a/xen/arch/arm/mmu/Makefile
>> +++ b/xen/arch/arm/mmu/Makefile
>> @@ -2,3 +2,4 @@ obj-y += p2m.o
>>   obj-y += pt.o
>>   obj-y += setup.o
>>   obj-y += smpboot.o
>> +obj-$(CONFIG_ARCH_MAP_DOMAIN_PAGE) += domain_page.o
> Don't you then also need to remove the respective line from arm/Makefile?

Yes you are correct. I missed it.

- Ayan

>
> Jan
diff mbox series

Patch

diff --git a/xen/arch/arm/mmu/Makefile b/xen/arch/arm/mmu/Makefile
index 67475fcd80..2cb44b857d 100644
--- a/xen/arch/arm/mmu/Makefile
+++ b/xen/arch/arm/mmu/Makefile
@@ -2,3 +2,4 @@  obj-y += p2m.o
 obj-y += pt.o
 obj-y += setup.o
 obj-y += smpboot.o
+obj-$(CONFIG_ARCH_MAP_DOMAIN_PAGE) += domain_page.o
diff --git a/xen/arch/arm/domain_page.c b/xen/arch/arm/mmu/domain_page.c
similarity index 100%
rename from xen/arch/arm/domain_page.c
rename to xen/arch/arm/mmu/domain_page.c