linux-kernel - Re: [patches] Re: [PATCH] dt-bindings: Add a RISC-V SBI firmware node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <mhng-dadf76d9-0823-4fc8-9177-1772629573b6@palmer-si-x1c4>
Date:   Tue, 21 Nov 2017 09:37:02 -0800 (PST)
From:   Palmer Dabbelt <palmer@...ive.com>
To:     robh@...nel.org, j.neuschaefer@....net
CC:     mark.rutland@....com, devicetree@...r.kernel.org,
        patches@...ups.riscv.org, linux-kernel@...r.kernel.org
Subject:     Re: [patches] Re: [PATCH] dt-bindings: Add a RISC-V SBI firmware node

On Mon, 20 Nov 2017 13:45:20 PST (-0800), robh@...nel.org wrote:
> On Mon, Nov 20, 2017 at 11:50:00AM -0800, Palmer Dabbelt wrote:
>> The RISC-V privileged ISA mandates the presence of an SBI, but there's
>> no reason not to put it in the device tree.  This would allow us to
>> possibly remove the SBI later.
>
> If it is mandatory, then it should not be in DT. DT is not a driver
> instantiation mechanism.
>
> If your ISA can vary, I'd recommend you make that discoverable. DT is
> for what we failed to make discoverable.

OK, that makes sense.  Since this the presence of an SBI is mandated by the ISA 
there's no way to discover it (just like there's no way to discover a load 
instruction).  For extensibility reasons you can, of course, determine which 
SBI calls are implemented, but this mechanism assumes you have something 
running to catch the SBI calls on the other end (SBI calls are just system 
calls from the Linux).  I think the original goal here was to avoid an SBI 
entirely, which isn't something the ISA is designed for.

This isn't really a big deal to me, as I'm only interested in RISC-V systems, 
but there's been some pushback on the concept of an SBI so it seemed like a 
simple way to allow people to build non-SBI (and there for not really RISC-V) 
systems.  One option that wouldn't require a device tree node would be to have 
Linux boot in machine mode (where the SBI implementation lives on systems 
without a hypervisor, if you've got a hypervisor then I assume the SBI isn't a 
problem) and then provide its own SBI implementation.  This wouldn't impose any 
additional burden: if there's no SBI then Linux will need to start in machine 
mode because that's where you need to be in order to do the things the SBI 
implementation needs to go.  This will be awkward to implement, but having a 
device tree entry won't help with any of that.

I think the right thing to do here is just not define a device tree entry.