Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Shea Levy <shea@shealevy.com>
To:     Zong Li <zong@andestech.com>, palmer@sifive.com, albert@sifive.com,
        linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
        zong@andestech.com, zongbox@gmail.com
Cc:     greentime@andestech.com
Subject: Re: [PATCH 00/11] RISC-V: Resolve the issue of loadable module on 64-bit
In-Reply-To: <cover.1520928420.git.zong@andestech.com>
References: <cover.1520928420.git.zong@andestech.com>
Date:   Tue, 13 Mar 2018 17:26:01 -0400
Message-ID: <87vadzzmza.fsf@xps13.shealevy.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
        micalg=pgp-sha256; protocol="application/pgp-signature"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

--=-=-=
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

Hello!

You may be interested in my recent patchset [1], which has known issues
but addresses the same problems yours does. It differs in the approach
taken here in that, rather than supporting GOT/PLT handling which we
can't really take advantage of anyway, we simply build non-PIC modules
instead [2]. Additionally, I see your patchset has the same concern mine
does, which is that ignoring ALIGN relaxations is not actually an option
[3]. The approach I plan to take is outlined by Palmer at [4].

As of now I hope to get back to my patchset this weekend, but if you're
close to a complete implementation by then maybe I can avoid duplicating
the work ;)

Thanks,
Shea

[1]: http://lists.infradead.org/pipermail/linux-riscv/2018-February/000081.=
html
[2]: http://lists.infradead.org/pipermail/linux-riscv/2018-February/000080.=
html
[3]: http://lists.infradead.org/pipermail/linux-riscv/2018-February/000105.=
html
[4]: http://lists.infradead.org/pipermail/linux-riscv/2018-March/000147.html
Zong Li <zong@andestech.com> writes:

> These patches resolve the some issues of loadable module.
>   - symbol out of ranges
>   - unknown relocation types
>
> The reference of external variable and function symbols
> cannot exceed 32-bit offset ranges in kernel module.
> The module only can work on the 32-bit OS or the 64-bit
> OS with sv32 virtual addressing.
>
> These patches will generate the .got, .got.plt and
> .plt sections during loading module, let it can refer
> to the symbol which locate more than 32-bit offset.
> These sections depend on the relocation types:
>  - R_RISCV_GOT_HI20
>  - R_RISCV_CALL_PLT
>
> These patches also support more relocation types
>  - R_RISCV_CALL
>  - R_RISCV_HI20
>  - R_RISCV_LO12_I
>  - R_RISCV_LO12_S
>  - R_RISCV_RVC_BRANCH
>  - R_RISCV_RVC_JUMP
>  - R_RISCV_ALIGN
>  - R_RISCV_ADD32
>  - R_RISCV_SUB32
>
> Zong Li (11):
>   RISC-V: Add sections of PLT and GOT for kernel module
>   RISC-V: Add section of GOT.PLT for kernel module
>   RISC-V: Support GOT_HI20/CALL_PLT relocation type in kernel module
>   RISC-V: Support CALL relocation type in kernel module
>   RISC-V: Support HI20/LO12_I/LO12_S relocation type in kernel module
>   RISC-V: Support RVC_BRANCH/JUMP relocation type in kernel modulewq
>   RISC-V: Support ALIGN relocation type in kernel module
>   RISC-V: Support ADD32 relocation type in kernel module
>   RISC-V: Support SUB32 relocation type in kernel module
>   RISC-V: Enable module support in defconfig
>   RISC-V: Add definition of relocation types
>
>  arch/riscv/Kconfig                  |   5 ++
>  arch/riscv/Makefile                 |   3 +
>  arch/riscv/configs/defconfig        |   2 +
>  arch/riscv/include/asm/module.h     | 112 +++++++++++++++++++++++
>  arch/riscv/include/uapi/asm/elf.h   |  24 +++++
>  arch/riscv/kernel/Makefile          |   1 +
>  arch/riscv/kernel/module-sections.c | 156 ++++++++++++++++++++++++++++++=
++
>  arch/riscv/kernel/module.c          | 175 ++++++++++++++++++++++++++++++=
++++--
>  arch/riscv/kernel/module.lds        |   8 ++
>  9 files changed, 480 insertions(+), 6 deletions(-)
>  create mode 100644 arch/riscv/include/asm/module.h
>  create mode 100644 arch/riscv/kernel/module-sections.c
>  create mode 100644 arch/riscv/kernel/module.lds
>
> --=20
> 2.16.1
>
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv

--=-=-=
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEE6ESKvwKkwnxgMLnaXAvWlX2G/icFAlqoQekACgkQXAvWlX2G
/icQMg//Veuyxbf/COwIXZiii1KT+useeuSQjczLLKn90qwPc/5zeI2slINzxVWl
LWEjZ7P4KYwXjvGBpqpjUeJGR6M6lQgtVoF4n7srqibLox03f4P5E6nOZ4fRSqo+
wcLVNt2hzPJAfNjwh1/BInl3VwzvAKsXvGbVOEAXlXwYy4r3MlJ1+qoK5bQ+9JGT
6sRAyOt+9PV8NKwrvvfwhoqTFfFRh3lXSoRPBYV8FezyJ9cia7868tlSKydig4CS
6MxYTfYkc1q3N1Fn1/jZ8JbcHteWIQTLDatBsTWcuO0lcCdMN9QuFiBz3KwxCtuo
iblZcE3VBqn7MNnFtdTOZZk0sC9fNBC9hbp+TCwXtuWA3AhCk6dPC7a9AGC1iz8z
gll10K585IYBwy4u+HAzV7kBNhuEC93yf1LSQ83vXlSVSl/roycOluYLBV3rif82
ATuPptNGYdjPoIHBEAZ2szeIazOHYtxtiNDoIp9/pi32Qv6kQnMIaeC7EL4lMw9S
4FClEOMTIyaKIG1qvG11dkBFkPkXda0tVwmghPBB9SScBuHJD54CTSaFVuWy8SFy
Y/6G5DyLBXAfNJf7FakHV0gADfAkSzo5X46hYqMKQP4WrfUhPVX1JKcvrmERQu7k
bYpk+VMJYfjswIDjv32fCa+jddsMhIMD7o36clepefL2i38rf6s=
=lysD
-----END PGP SIGNATURE-----
--=-=-=--