IronPort-SDR: EK6lzJ7oPYIdhuTQLQ960XwrV6alckdYezmvdXMO3Fpaap/Okb5OkNmsjhnUztt6OeLCLc6psZ
 ymMYrRjpS/wtvrMTVNa4XkTu36j7fKNP6qv/slX43idwX8p24wjJVVmUgCVZsQ9uGBdaegwQ8q
 1Xyv2n8pwYQYqLKMvLmw//zGn9Q6nMIkTETTDy63KNF8dPixl4uM3L6PaCtogRm374kkw1mEG7
 yZLQrTvLT1z9F5dU4Spfn4esJL/YM3kJkV8QooNSEMfWE4Db/5N5WZBsdWCZmHpU+OSZuJRBZK
 At8=
IronPort-SDR: NsEDAFCd4ZSTj3Wj+EakJfW6g54gNTbZ7pUBC5kxqH3zsNJ6NbOmgzPfKBcdfVyIMsirPrBM1n
 qYthxSzZ30XnZr3HB/mLtMQA/NWnxM75f0eIv59/YRTZLjZDw/4Dm8lqMEKc86Ji7ZSNPo2Omp
 b8KE+p5Ce7dooEk5e14OIPxjHRRFm2+6iHrejhx/4jNE4h+GSakB5Gfh87OBTMMpLaivbCVOD8
 WMov4zT6Y+bf6TtJbnNv0jhLPeiub89dEbRA7X8DBFq1o5ioFa38dJ9i+f4B/r/XEzMTGwZs6W
 /Bo=
WDCIronportException: Internal
Message-ID: <3719a4f0-1a5a-4661-a6d7-5d4f67d7b7e8@opensource.wdc.com>
Date:   Wed, 1 Mar 2023 07:49:05 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH 0/2] riscv: support ELF format binaries in nommu mode
Content-Language: en-US
To:     Palmer Dabbelt <palmer@dabbelt.com>, gerg@kernel.org,
        Niklas Cassel <Niklas.Cassel@wdc.com>
Cc:     linux-riscv@lists.infradead.org, linux-mm@kvack.org,
        linux-kernel@vger.kernel.org
References: <mhng-20faeb04-8ed4-4759-8f13-aef3d2446d15@palmer-ri-x1c9a>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <mhng-20faeb04-8ed4-4759-8f13-aef3d2446d15@palmer-ri-x1c9a>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 3/1/23 06:52, Palmer Dabbelt wrote:
> On Tue, 28 Feb 2023 05:51:24 PST (-0800), gerg@kernel.org wrote:
>> The following changes add the ability to run ELF format binaries when
>> running RISC-V in nommu mode. That support is actually part of the
>> ELF-FDPIC loader, so these changes are all about making that work on
>> RISC-V.
>>
>> The first issue to deal with is making the ELF-FDPIC loader capable of
>> handling 64-bit ELF files. As coded right now it only supports 32-bit
>> ELF files.
>>
>> Secondly some changes are required to enable and compile the ELF-FDPIC
>> loader on RISC-V and to pass the ELF-FDPIC mapping addresses through to
>> user space when execing the new program.
>>
>> These changes have not been used to run actual ELF-FDPIC binaries.
>> It is used to load and run normal ELF - compiled -pie format. Though the
>> underlying changes are expected to work with full ELF-FDPIC binaries if
>> or when that is supported on RISC-V in gcc.
>>
>> To avoid needing changes to the C-library (tested with uClibc-ng
>> currently) there is a simple runtime dynamic loader (interpreter)
>> available to do the final relocations, https://github.com/gregungerer/uldso.
>> The nice thing about doing it this way is that the same program
>> binary can also be loaded with the usual ELF loader in MMU linux.
>>
>> The motivation here is to provide an easy to use alternative to the
>> flat format binaries normally used for RISC-V nommu based systems.
>>
>> Signed-off-by: Greg Ungerer <gerg@kernel.org>
>> ---
>>
>>  arch/riscv/include/asm/elf.h         |   11 +++++++++-
>>  arch/riscv/include/asm/mmu.h         |    4 +++
>>  arch/riscv/include/uapi/asm/ptrace.h |    5 ++++
>>  fs/Kconfig.binfmt                    |    2 -
>>  fs/binfmt_elf_fdpic.c                |   38 +++++++++++++++++------------------
>>  include/linux/elf-fdpic.h            |   14 +++++++++++-
>>  include/uapi/linux/elf-fdpic.h       |   15 +++++++++++++
>>  7 files changed, 67 insertions(+), 22 deletions(-)
> 
> Adding Damien, as IIRC he's had some hacked up userspace bits for the 
> K210.  I'm yet to get anything running, but it'd be great if we get this 
> to a point where I can actually boot test this on QEMU (I'm just doing 
> builds now).

+Niklas

Niklas, didn't you add a nommu qemu build for buildroot ? Can't find the config
though...

> 
> Given that it's the second week of the merge window and this is a bunch 
> of new uABI it seems best to hold off until the next cycle.  I poked 
> around and don't see anything wrong, but I'll try and take a more 
> detailed look after the merge window.

Does any riscv compiler support nommu fdpic now ? When doing the work on the
k210, there was no support at all, hence the statically linked user binaries
used (no shared text for libraries).


-- 
Damien Le Moal
Western Digital Research