This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2 00/12] RISC-V glibc port for the 32 bit


On Wed, 25 Jul 2018 08:19:51 PDT (-0700), zongbox@gmail.com wrote:
Palmer Dabbelt <palmer@dabbelt.com> 於 2018年7月25日 週三 上午6:08寫道:

On Mon, 16 Jul 2018 22:07:46 PDT (-0700), zong@andestech.com wrote:
> This patch set contains the glibc port for the 32 bit RISC-V. I ran the glibc
> test suite on QEMU, and remained the failed cases which caused by environment
> issue like 64 bit glibc port. In addition, there are some math test cases
> need to be checked but it looks unlike glibc's problem.
>
> There is a patch to fix the ld flags issue of tst-execstack-mod.so, it
> cause the fail on some test cases on RISC-V. In the v1 patch, I don't include
> this patch into patch set, in v2, I collect all patches together.
>
> There is a patch to add lack of implementation of 128 bit for soft-fp, these
> are necessary for building 32 bit RISC-V port.
>
> Thanks everyone for the help and Palmer's efforts during this work.
>
> Change in V2:
>  - Only include the ieee754/soft-fp path in rv32/nofpu/Implies.
>  - Add lack of implementation for 128 bit in soft-ft/op-8.h.
>  - Include the patch which fix the ld flags of tst-execstack-mod.so.
>  - Add the modification of URL of gcc's tarball.
>
> Zong Li (12):
>   Documentation for the 32 bit RISC-V port
>   RISC-V: Add dynamic loader for the 32 bit
>   RISC-V: Add path of library directories for the 32 bit
>   RISC-V: The ABI implementation for the 32-bit
>   RISC-V: Hard float support for the 32 bit
>   RISC-V: Split the soft-fp into rv32 and rv64
>   RISC-V: Add ABI lists
>   RISC-V: Build Infastructure for the 32 bit
>   Add 32 bit RISC-V to build-many-glibcs.py
>   soft-fp: Add the lack of implementation for 128 bit
>   Fix the ld flags not be applied to tst-execstack-mod.so
>   Change URL of gcc's tarball

Thanks for doing this Zong.  I don't think there's anything major in the patch
set, the bigger issues are externally.  As it's been pointed out, the Linux ABI
is still a bit mushy and I don't there's nearly enough time to get everything
in shape for the glibc release, which if I understand correctly is still
targeted for early in August.

I think we should focus on ensuring this release is sane in RV64 land, and then
add the RV32 port after the release so it has a whole cycle to calm down.

Thanks for doing all this work.  If you'd like I can take the patches from
here and deal with things like cleaning up the Linux ABI and the test suites,
but if you have time then feel free.

Thanks, again!

OK, maybe I can submit the version three to fix up the known issue in
version two
before you clean up ABI and test suites? or some way let you can get
the modification
if submitting the version three is inappropriate, or you want to take
version two patches.
How do you think?

Submitting a v3 would be great, thanks!

As the cover letter mention, there are math fail cases need to be
clarified, I will
keep to track and get you feedback when I get the point about that.

There is a patch about lack of implementation for 128 soft-fp. On the
Joseph recommend,
it should be out of the patch set, so I will submit it separately.

Makes sense.

Thanks Joseph, Palmer, DJ and  Richard's help and review the patches.

Well, thanks for doing the work :)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]