Integrated: 8344350: Add '.gdbinit' and '.lldbinit' to file '.gitignore'

2024-11-19 Thread Guoxiong Li
On Sat, 16 Nov 2024 16:29:20 GMT, Guoxiong Li wrote: > Hi all, > > Some developers want to run some initial scripts locally when the debugger > (gdb or lldb) starts. So they often add a file named `.gdbinit` or > `.lldbinit` in the project root path. It is good to ignore them.

Re: RFR: 8344350: Add '.gdbinit' and '.lldbinit' to file '.gitignore'

2024-11-19 Thread Guoxiong Li
On Sat, 16 Nov 2024 16:29:20 GMT, Guoxiong Li wrote: > Hi all, > > Some developers want to run some initial scripts locally when the debugger > (gdb or lldb) starts. So they often add a file named `.gdbinit` or > `.lldbinit` in the project root path. It is good to ignore them.

Re: RFR: 8344350: Add '.gdbinit' and '.lldbinit' to file '.gitignore'

2024-11-18 Thread Guoxiong Li
On Mon, 18 Nov 2024 15:44:14 GMT, Magnus Ihse Bursie wrote: >> Hi all, >> >> Some developers want to run some initial scripts locally when the debugger >> (gdb or lldb) starts. So they often add a file named `.gdbinit` or >> `.lldbinit` in the project root path. It is good to ignore them. >>

RFR: 8344350: Add '.gdbinit' and '.lldbinit' to file '.gitignore'

2024-11-16 Thread Guoxiong Li
Hi all, Some developers want to run some initial scripts locally when the debugger (gdb or lldb) starts. So they often add a file named `.gdbinit` or `.lldbinit` in the project root path. It is good to ignore them. Thanks for your review. Best Regards, -- Guoxiong - Commit messag

Re: RFR: 8329494: Serial: Merge GenMarkSweep into MarkSweep

2024-04-03 Thread Guoxiong Li
On Wed, 3 Apr 2024 09:56:42 GMT, Albert Mingkun Yang wrote: >> Hi all, >> >> This patch merges the class `GenMarkSweep` into `MarkSweep`. Just simply >> move. It may be better to merge the class `DeadSpacer` and `Compacter` into >> `MarkSweep` and then rename `MarkSweep`. The `MarkSweep` will

Integrated: 8329494: Serial: Merge GenMarkSweep into MarkSweep

2024-04-03 Thread Guoxiong Li
On Wed, 3 Apr 2024 03:10:26 GMT, Guoxiong Li wrote: > Hi all, > > This patch merges the class `GenMarkSweep` into `MarkSweep`. Just simply > move. It may be better to merge the class `DeadSpacer` and `Compacter` into > `MarkSweep` and then rename `MarkSweep`. The `MarkSweep`

RFR: 8329494: Serial: Merge GenMarkSweep into MarkSweep

2024-04-02 Thread Guoxiong Li
Hi all, This patch merges the class `GenMarkSweep` into `MarkSweep`. Just simply move. It may be better to merge the class `DeadSpacer` and `Compacter` into `MarkSweep` and then rename `MarkSweep`. The `MarkSweep` will be renamed at [JDK-8329521](https://bugs.openjdk.org/browse/JDK-8329521). An

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Guoxiong Li
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. > I have no idea what causes it. Nevermind. The patch looks good. - Marked as reviewed by gli (Committer). PR Review: https://git.openj

Re: RFR: 8326831 Clarify test harness control variables in make help

2024-02-28 Thread Guoxiong Li
On Tue, 27 Feb 2024 13:55:53 GMT, Ludvig Janiuk wrote: > Clarifying text in `make help` output on how to list variables for e.g. > JTREG='...'. I use the command `make test-only JTREG=help` locally, the outputed message seems an error (shown below). Does it work as expected? Building target

Re: RFR: 8322801: RISC-V: The riscv path of the debian sysroot had been changed [v2]

2024-01-02 Thread Guoxiong Li
On Tue, 2 Jan 2024 06:49:34 GMT, Robbin Ehn wrote: >> Guoxiong Li has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated html > > Thanks! @robehn @erikj79 Thanks for your reviews. Integrating. --

Integrated: 8322801: RISC-V: The riscv path of the debian sysroot had been changed

2024-01-02 Thread Guoxiong Li
On Mon, 1 Jan 2024 11:04:35 GMT, Guoxiong Li wrote: > Hi all, > > This trivial patch removes the information about the path `debian-ports` > because the riscv related packages had been moved to the path `debian` at > 2024-01-01. Now the arm and riscv use the sam

Re: RFR: 8322801: RISC-V: The riscv path of the debian sysroot had been changed

2024-01-02 Thread Guoxiong Li
On Mon, 1 Jan 2024 11:04:35 GMT, Guoxiong Li wrote: > Hi all, > > This trivial patch removes the information about the path `debian-ports` > because the riscv related packages had been moved to the path `debian` at > 2024-01-01. Now the arm and riscv use the sam

Re: RFR: 8322801: RISC-V: The riscv path of the debian sysroot had been changed [v2]

2024-01-02 Thread Guoxiong Li
; Thanks for your review. > > Best Regards, > -- Guoxiong Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision: Updated html - Changes: - all: https://git.openjdk.org/jdk/pull/17208/files - new: https://git.openjdk.

Re: RFR: 8322801: RISC-V: The riscv path of the debian sysroot had been changed

2024-01-02 Thread Guoxiong Li
On Tue, 2 Jan 2024 13:58:50 GMT, Erik Joelsson wrote: > The html file also needs to be regenerated. Let me know if you need help with > that. Yes indeed. Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/17208#issuecomment-1874073136

RFR: 8322801: RISC-V: The riscv path of the debian sysroot had been changed

2024-01-01 Thread Guoxiong Li
Hi all, This trivial patch removes the information about the path `debian-ports` because the riscv related packages had been moved to the path `debian` at 2024-01-01. Now the arm and riscv use the same url `https://httpredir.debian.org/debian/`. Thanks for your review. Best Regards, -- Guoxio

RFR: 8321688: Build on linux with GCC 7.5.0 fails after 8319577

2023-12-10 Thread Guoxiong Li
Hi all, This patch fixes the building failure introduced by [JDK-8319577](https://bugs.openjdk.org/browse/JDK-8319577) in old GCC version (linux & GCC 7.5.0 locally). Thanks for the review. Best Regards, -- Guoxiong - Commit messages: - JDK-8321688 Changes: https://git.openjdk

Integrated: JDK-8305118: Add RISC-V related content to building.md

2023-04-06 Thread Guoxiong Li
On Wed, 29 Mar 2023 10:14:49 GMT, Guoxiong Li wrote: > Hi all, > > This patch adds the RISC-V related content to `building.md`. But I didn't > generate `building.html` locally, because I know the `pandoc` version > may influencethe the output of `building.html` (Need hel

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v4]

2023-04-05 Thread Guoxiong Li
But it > failed locally. > Maybe someone who has built successfully can tell me how to change the > document, which mainly adds the right name to their supported target table. > > Thanks for the review and assist. > > Best Regards, > -- Guoxiong Guoxiong Li has updated the

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v3]

2023-04-05 Thread Guoxiong Li
On Wed, 5 Apr 2023 13:11:15 GMT, Erik Joelsson wrote: > Generated html file here https://github.com/erikj79/jdk/commits/pull/13223 Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/13223#issuecomment-1497504282

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v2]

2023-04-04 Thread Guoxiong Li
On Thu, 30 Mar 2023 04:21:01 GMT, Chen Liang wrote: >> Guoxiong Li has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - Add riscv64 related content to 'Debian sysroots' >> - qemu-debootstrap has

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v3]

2023-04-04 Thread Guoxiong Li
But it > failed locally. > Maybe someone who has built successfully can tell me how to change the > document, which mainly adds the right name to their supported target table. > > Thanks for the review and assist. > > Best Regards, > -- Guoxiong Guoxiong Li has updated the

Re: RFR: JDK-8305118: Add RISC-V related content to building.md

2023-03-29 Thread Guoxiong Li
On Wed, 29 Mar 2023 10:28:37 GMT, Aleksey Shipilev wrote: > RISC-V cross-compiles with sysroots fine: > https://builds.shipilev.net/openjdk-jdk/configure-logs/configure-openjdk-jdk-linux-riscv64-server-release-gcc10-glibc2.31.log > > https://builds.shipilev.net/openjdk-jdk/build-logs/build-ope

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v2]

2023-03-29 Thread Guoxiong Li
On Wed, 29 Mar 2023 13:37:36 GMT, Erik Joelsson wrote: >> Guoxiong Li has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - Add riscv64 related content to 'Debian sysroots' >> - qemu-debootst

Re: RFR: JDK-8305118: Add RISC-V related content to building.md [v2]

2023-03-29 Thread Guoxiong Li
But it > failed locally. > Maybe someone who has built successfully can tell me how to change the > document, which mainly adds the right name to their supported target table. > > Thanks for the review and assist. > > Best Regards, > -- Guoxiong Guoxiong Li has updated th

RFR: JDK-8305118: Add RISC-V related content to building.md

2023-03-29 Thread Guoxiong Li
Hi all, This patch adds the RISC-V related content to `building.md`. But I didn't generate `building.html` locally, because I know the `pandoc` version may influencethe the output of `building.html` (Need help here). I know there are many steps and subtle points about cross-compiling the exter

Re: The building document about RISC-V

2023-03-28 Thread Guoxiong Li
> I guess there might be an issue on the server side. I cannot even access: https://cr.openjdk.org/~fyang/ My personal page works: https://cr.openjdk.org/~gli/ . > That's definitely a welcome :-) Filed https://bugs.openjdk.org/browse/JDK-8305118 to follow up.

The building document about RISC-V

2023-03-23 Thread Guoxiong Li
Hi all, I noticed there is no content about the risc-v port in the building document [1]. And the link [2] in the wiki, which may be related to the building, can't be opened now (it returns 404 status code). So it is good to add the risc-v content into the building document. Has somebody started

Re: Building failed when using the options `--always-make` and `JOBS=4`

2022-07-27 Thread Guoxiong Li
Hi Erik, Thanks for replying to this thread. > Are you sure you can't configure the plugin to not add this parameter? The configuration can be changed and I fixed it locally just now. But it needs more time to dive into the extension and its configuration. Maybe it is good to provide some inform

Building failed when using the options `--always-make` and `JOBS=4`

2022-07-27 Thread Guoxiong Li
Hi all, When using the command `make images --always-make CONF_CHECK=ignore JOBS=4` to build the JDK, it doesn't succeed and returns the error message (See below for detail). It seems that two or more processes concurrently process the target `module-deps.gmk` which exists in the file `make/common