From patchwork Wed Jan 24 07:18:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Jones X-Patchwork-Id: 13528674 Received: from out-181.mta0.migadu.com (out-181.mta0.migadu.com [91.218.175.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6F32F17752 for ; Wed, 24 Jan 2024 07:19:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706080746; cv=none; b=XxjClid3RbAsiFS2A+ZCN4mUgavqzcgUSRJZxkgK6fssabhA7KTQ7ob0YRIeuRb/0k7hzXc2n7LJW8XT9/H6kiBrQUCb13sLTzAtCEjStLSDrWSmjXz6fvalZ6JQLHlrcPMpDHRItWJqsE2bKjQhmfLeQx8hLzAwtOMBE7Wpx0I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706080746; c=relaxed/simple; bh=Nw8HzYpp81iwd1Up1q56UYdNoPXd6MK56s4C6EggVqU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-type; b=GiKLkxzYrm0N6VrmmDNU7TtE2azwoufzPTr7VOrBp3TnntDVOpQ75dOLdWEqSub0MA9A6tSEJuxO3i89svQlJG815uXJhAtPEDI/fMa+KBlFrCAtYYbmQaEm8qq2n9wpA07j19xjrmao549jI+qXCm71SZa2LwZMvw+bf+onUbs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=lD9IQu8d; arc=none smtp.client-ip=91.218.175.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="lD9IQu8d" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1706080742; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Ad85rv8uoTjpCnNqcda1d8moggW5XED13BqFM4Q8Hg8=; b=lD9IQu8d2ToRfpveL3WNbHEyJvn/kIB2dzMHFugB2i1yWgARk9laxj9UoEL7OqgQa/sUxH y4WkUdie/wVj9SAhr1ShjOn2P617928EWzxkWDfxK5TdLnc4ISW0fCzqrz9FSmPI2VTZ4b x4F9c0eM/c9h1HWEmCCd2Cxt44okWu4= From: Andrew Jones To: kvm@vger.kernel.org, kvm-riscv@lists.infradead.org, kvmarm@lists.linux.dev Cc: ajones@ventanamicro.com, anup@brainfault.org, atishp@atishpatra.org, pbonzini@redhat.com, thuth@redhat.com, alexandru.elisei@arm.com, eric.auger@redhat.com Subject: [kvm-unit-tests PATCH 17/24] riscv: Populate memregions and switch to page allocator Date: Wed, 24 Jan 2024 08:18:33 +0100 Message-ID: <20240124071815.6898-43-andrew.jones@linux.dev> In-Reply-To: <20240124071815.6898-26-andrew.jones@linux.dev> References: <20240124071815.6898-26-andrew.jones@linux.dev> Precedence: bulk X-Mailing-List: kvm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-type: text/plain X-Migadu-Flow: FLOW_OUT Get the memory regions from the DT rather than just assuming we have enough. Getting memory regions and setting their flags is also a first step to enabling the MMU. Also switch to the page allocator, which is a second step to enabling the MMU. Signed-off-by: Andrew Jones --- lib/riscv/asm/memory_areas.h | 1 + lib/riscv/setup.c | 55 ++++++++++++++++++++++++++++++++++-- riscv/Makefile | 2 ++ 3 files changed, 55 insertions(+), 3 deletions(-) create mode 100644 lib/riscv/asm/memory_areas.h diff --git a/lib/riscv/asm/memory_areas.h b/lib/riscv/asm/memory_areas.h new file mode 100644 index 000000000000..2b34e63502dd --- /dev/null +++ b/lib/riscv/asm/memory_areas.h @@ -0,0 +1 @@ +#include diff --git a/lib/riscv/setup.c b/lib/riscv/setup.c index 9ff446b5e171..848ec8e83496 100644 --- a/lib/riscv/setup.c +++ b/lib/riscv/setup.c @@ -6,22 +6,31 @@ */ #include #include +#include #include #include #include #include +#include #include #include #include #include #include +#define VA_BASE ((phys_addr_t)3 * SZ_1G) + +#define MAX_DT_MEM_REGIONS 16 +#define NR_MEM_REGIONS (MAX_DT_MEM_REGIONS + 16) + char *initrd; u32 initrd_size; struct thread_info cpus[NR_CPUS]; int nr_cpus; +static struct mem_region riscv_mem_regions[NR_MEM_REGIONS + 1]; + int hartid_to_cpu(unsigned long hartid) { int cpu; @@ -64,10 +73,50 @@ static void cpu_init(void) cpu0_calls_idle = true; } +extern unsigned long _etext; + static void mem_init(phys_addr_t freemem_start) { - //TODO - for now just assume we've got some memory available - phys_alloc_init(freemem_start, 16 * SZ_1M); + struct mem_region *freemem, *code, *data; + phys_addr_t freemem_end, base, top; + + memregions_init(riscv_mem_regions, NR_MEM_REGIONS); + memregions_add_dt_regions(MAX_DT_MEM_REGIONS); + + /* Split the region with the code into two regions; code and data */ + memregions_split((unsigned long)&_etext, &code, &data); + assert(code); + code->flags |= MR_F_CODE; + + freemem = memregions_find(freemem_start); + assert(freemem && !(freemem->flags & (MR_F_IO | MR_F_CODE))); + + freemem_end = freemem->end & PAGE_MASK; + + /* + * The assert below is mostly checking that the free memory doesn't + * start in the 3G-4G range, which is reserved for virtual addresses, + * but it also confirms that there is some free memory (the amount + * is arbitrarily selected, but should be sufficient for a unit test) + * + * TODO: Allow the VA range to shrink and move. + */ + if (freemem_end > VA_BASE) + freemem_end = VA_BASE; + assert(freemem_end - freemem_start >= SZ_1M * 16); + + /* + * TODO: Remove the need for this phys allocator dance, since, as we + * can see with the assert, we could have gone straight to the page + * allocator. + */ + phys_alloc_init(freemem_start, freemem_end - freemem_start); + phys_alloc_set_minimum_alignment(PAGE_SIZE); + phys_alloc_get_unused(&base, &top); + assert(base == freemem_start && top == freemem_end); + + page_alloc_init_area(0, freemem_start >> PAGE_SHIFT, freemem_end >> PAGE_SHIFT); + page_alloc_ops_enable(); } static void banner(void) @@ -86,7 +135,7 @@ void setup(const void *fdt, phys_addr_t freemem_start) u32 fdt_size; int ret; - assert(sizeof(long) == 8 || freemem_start < (3ul << 30)); + assert(sizeof(long) == 8 || freemem_start < VA_BASE); freemem = (void *)(unsigned long)freemem_start; /* Move the FDT to the base of free memory */ diff --git a/riscv/Makefile b/riscv/Makefile index 932f3378264c..ed1a14025ed2 100644 --- a/riscv/Makefile +++ b/riscv/Makefile @@ -22,8 +22,10 @@ $(TEST_DIR)/sieve.elf: AUXFLAGS = 0x1 cstart.o = $(TEST_DIR)/cstart.o cflatobjs += lib/alloc.o +cflatobjs += lib/alloc_page.o cflatobjs += lib/alloc_phys.o cflatobjs += lib/devicetree.o +cflatobjs += lib/memregions.o cflatobjs += lib/on-cpus.o cflatobjs += lib/riscv/bitops.o cflatobjs += lib/riscv/io.o