RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-05 Thread SendaoYan
Hi all, This PR fix some typos bugs for RISC-V, and a typo bug in test directory. Trivial fix, no risk. - Commit messages: - 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp Changes: https://git.openjdk.org/jdk/pull/21368/files Webrev: https://webrevs.openjdk.org/?repo=jd

Re: RFR: 8336854: CAInterop.java#actalisauthenticationrootca conflicted with /manual and /timeout

2024-08-13 Thread SendaoYan
On Sat, 20 Jul 2024 15:32:37 GMT, SendaoYan wrote: > Hi all, > The testcase > `test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca` > run Error, because the `@run` set conflicted with `/manual` and `/timeout`. &

Integrated: 8336854: CAInterop.java#actalisauthenticationrootca conflicted with /manual and /timeout

2024-08-13 Thread SendaoYan
On Sat, 20 Jul 2024 15:32:37 GMT, SendaoYan wrote: > Hi all, > The testcase > `test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca` > run Error, because the `@run` set conflicted with `/manual` and `/timeout`. &

Re: RFR: 8336854: CAInterop.java#actalisauthenticationrootca conflicted with /manual and /timeout

2024-07-22 Thread SendaoYan
On Sat, 20 Jul 2024 15:32:37 GMT, SendaoYan wrote: > Hi all, > The testcase > `test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca` > run Error, because the `@run` set conflicted with `/manual` and `/timeout`. &

RFR: 8336854: CAInterop.java#actalisauthenticationrootca conflicted with /manual and /timeout

2024-07-20 Thread SendaoYan
Hi all, The testcase `test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca` run Error, because the `@run` set conflicted with `/manual` and `/timeout`. I think the `/timeout` not needed for manual test. Only this testcase has this

Withdrawn: 8332492: Mark CAInterop.java#globalsigne46 as intermittent

2024-07-13 Thread SendaoYan
On Sat, 18 May 2024 12:34:39 GMT, SendaoYan wrote: > Hi all, > Before `CAInterop.java#globalsigne46` imtermittent > [failure](https://bugs.openjdk.org/browse/JDK-8332433) has been resolved, > mark the test as intermittent. This pull request has been closed without bein

Re: RFR: 8332492: Mark CAInterop.java#globalsigne46 as intermittent

2024-07-13 Thread SendaoYan
On Sat, 18 May 2024 12:34:39 GMT, SendaoYan wrote: > Hi all, > Before `CAInterop.java#globalsigne46` imtermittent > [failure](https://bugs.openjdk.org/browse/JDK-8332433) has been resolved, > mark the test as intermittent. Since `test/jdk/security/infra/java/security/cert/CertP

Re: [jdk23] RFR: 8334441: Mark tests in jdk_security_infra group as manual

2024-06-23 Thread SendaoYan
On Sat, 22 Jun 2024 08:07:54 GMT, SendaoYan wrote: > Hi all, > > This pull request contains a backport of commit > [8e1d2b09](https://github.com/openjdk/jdk/commit/8e1d2b091c9a311d98a0b886a803fb18d4405d8a) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. >

[jdk23] Integrated: 8334441: Mark tests in jdk_security_infra group as manual

2024-06-23 Thread SendaoYan
On Sat, 22 Jun 2024 08:07:54 GMT, SendaoYan wrote: > Hi all, > > This pull request contains a backport of commit > [8e1d2b09](https://github.com/openjdk/jdk/commit/8e1d2b091c9a311d98a0b886a803fb18d4405d8a) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. >

Re: [jdk23] RFR: 8334441: Mark tests in jdk_security_infra group as manual

2024-06-22 Thread SendaoYan
On Sat, 22 Jun 2024 08:07:54 GMT, SendaoYan wrote: > Hi all, > > This pull request contains a backport of commit > [8e1d2b09](https://github.com/openjdk/jdk/commit/8e1d2b091c9a311d98a0b886a803fb18d4405d8a) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. >

[jdk23] RFR: 8334441: Mark tests in jdk_security_infra group as manual

2024-06-22 Thread SendaoYan
Hi all, This pull request contains a backport of commit [8e1d2b09](https://github.com/openjdk/jdk/commit/8e1d2b091c9a311d98a0b886a803fb18d4405d8a) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being backported was authored by Rajan Halade on 21 Jun 2024 and was rev

Re: RFR: 8333938: Exclude CAInterop.java#digicerttlsrsarootg5

2024-06-19 Thread SendaoYan
On Wed, 19 Jun 2024 15:36:44 GMT, Christoph Langer wrote: > Looks like this PR can be closed. The underlying issue has been resolved by > the CA. Okey. - PR Comment: https://git.openjdk.org/jdk/pull/19694#issuecomment-2179040063

Withdrawn: 8333938: Exclude CAInterop.java#digicerttlsrsarootg5

2024-06-19 Thread SendaoYan
On Thu, 13 Jun 2024 12:32:58 GMT, SendaoYan wrote: > Hi all, > Test > `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#digicerttlsrsarootg5` > report failure as failed to validate, before the validate issue has been > fixed, should we problem li

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105 [v2]

2024-06-19 Thread SendaoYan
On Fri, 14 Jun 2024 00:58:30 GMT, SendaoYan wrote: >> Thanks for the approved. > >> @sendaoYan As a best practice, it would be useful to first understand why >> the test is not working before putting it on the ProblemList. Depending on >> the severity of the problem

Withdrawn: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105

2024-06-19 Thread SendaoYan
On Thu, 13 Jun 2024 01:20:55 GMT, SendaoYan wrote: > Hi all, > Test > `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#quovadisrootca1g3` > report failure as `failed to validate`, before the validate issue has been > fixed, should we p

Re: RFR: 8332492: Mark CAInterop.java#globalsigne46 as intermittent

2024-06-15 Thread SendaoYan
On Sat, 18 May 2024 12:34:39 GMT, SendaoYan wrote: > Hi all, > Before `CAInterop.java#globalsigne46` imtermittent > [failure](https://bugs.openjdk.org/browse/JDK-8332433) has been resolved, > mark the test as intermittent. Hi, can anyone take look this PR. - PR Co

Re: RFR: 8333938: Exclude CAInterop.java#digicerttlsrsarootg5

2024-06-13 Thread SendaoYan
On Thu, 13 Jun 2024 12:32:58 GMT, SendaoYan wrote: > Hi all, > Test > `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#digicerttlsrsarootg5` > report failure as failed to validate, before the validate issue has been > fixed, should we problem li

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105 [v2]

2024-06-13 Thread SendaoYan
On Thu, 13 Jun 2024 14:38:08 GMT, SendaoYan wrote: >> SendaoYan has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Merge branch 'jbs8334106' of github.com:sendaoYan/jdk-ysd into jbs8334106 >> -

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105 [v2]

2024-06-13 Thread SendaoYan
On Thu, 13 Jun 2024 12:21:23 GMT, SendaoYan wrote: >> Hi all, >> Test >> `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#quovadisrootca1g3` >> report failure as `failed to validate`, before the validate issue has been >> fix

RFR: 8333938: Exclude CAInterop.java#digicerttlsrsarootg5

2024-06-13 Thread SendaoYan
Hi all, Test `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#digicerttlsrsarootg5` report failure as failed to validate, before the validate issue has been fixed, should we problem list the testcase. - Commit messages: - 8333938: Exclude CAInterop

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105 [v2]

2024-06-13 Thread SendaoYan
> Hi all, > Test > `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#quovadisrootca1g3` > report failure as `failed to validate`, before the validate issue has been > fixed, should we problem list the testcase. SendaoYan has updated t

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105

2024-06-13 Thread SendaoYan
On Thu, 13 Jun 2024 09:21:41 GMT, Christoph Langer wrote: > let's wait a bit with integration before the security folks had a chance to > look at this... Okey. - PR Comment: https://git.openjdk.org/jdk/pull/19685#issuecomment-2165126505

Re: RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105

2024-06-13 Thread SendaoYan
On Thu, 13 Jun 2024 01:20:55 GMT, SendaoYan wrote: > Hi all, > Test > `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#quovadisrootca1g3` > report failure as `failed to validate`, before the validate issue has been > fixed, should we p

RFR: 8334106: Problemlist CAInterop.java#quovadisrootca1g3 due to JDK-8334105

2024-06-12 Thread SendaoYan
Hi all, Test `security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#quovadisrootca1g3` report failure as `failed to validate`, before the validate issue has been fixed, should we problem list the testcase. - Commit messages: - Merge branch 'openjdk:ma

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote: >> Hi all, >> This PR several extra empty spaces and extra empty lines in several >> Makefiles. It's trivial fix, no risk. >> >> Thanks. > > SendaoYan has updated the pull request incrementally wit

Integrated: 8333477: Delete extra empty spaces in Makefiles

2024-06-07 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. This pull request has now been integrated. Changeset: d130d2f4 Author:SendaoYa

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 12:53:46 GMT, Chen Liang wrote: >> No, it's an extra newline. A file should end with a newline but one is >> enough. > > As confusing as they are, unfortunately GitHub UI does not render extra > trailing newlines. This is the only one I could find with grepWin. I find the ex

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote: >> Hi all, >> This PR several extra empty spaces and extra empty lines in several >> Makefiles. It's trivial fix, no risk. >> >> Thanks. > > SendaoYan has updated the pull request incrementally wit

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
> Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. SendaoYan has updated the pull request incrementally with one additional commit since the last revision: delete extra empty trailing bla

Re: RFR: 8333477: Delete extra empty spaces in Makefiles [v2]

2024-06-07 Thread SendaoYan
On Thu, 6 Jun 2024 17:49:08 GMT, Chen Liang wrote: >> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> delete extra empty trailing blank line in >> test/jdk/java/rmi/reliability/benchmark/bench/

Re: RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. Thanks for the review. Thanks all for the review. - PR Comment: https:/

Re: RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote: > Hi all, > This PR several extra empty spaces and extra empty lines in several > Makefiles. It's trivial fix, no risk. > > Thanks. > /label build Thanks. - PR Comment: https://git.openjdk.org/jdk/

RFR: 8333477: Delete extra empty spaces in Makefiles

2024-06-04 Thread SendaoYan
Hi all, This PR several extra empty spaces and extra empty lines in several Makefiles. It's trivial fix, no risk. Thanks. - Commit messages: - 8333477: Delete extra empty spaces in Makefiles Changes: https://git.openjdk.org/jdk/pull/19537/files Webrev: https://webrevs.openjdk.

Re: RFR: 8332492: Mark CAInterop.java#globalsigne46 as intermittent

2024-05-18 Thread SendaoYan
On Sat, 18 May 2024 12:44:52 GMT, Weijun Wang wrote: > Is there a related bug on the intermittent failure? I have reportd the itermittent failure [JDK-8332433](https://bugs.openjdk.org/browse/JDK-8332433), and [JDK-8316138](https://bugs.openjdk.org/browse/JDK-8316138) is subtask of JDK-833243

RFR: 8332492: Mark CAInterop.java#globalsigne46 as intermittent

2024-05-18 Thread SendaoYan
Hi all, Before `CAInterop.java#globalsigne46` imtermittent failure has been resolved, mark the test as intermittent. - Commit messages: - 8332492: Mark CAInterop.java#globalsigne46 as intermittent Changes: https://git.openjdk.org/jdk/pull/19291/files Webrev: https://webrevs.ope

Re: RFR: 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information

2024-04-08 Thread SendaoYan
On Thu, 1 Feb 2024 03:08:52 GMT, Fei Yang wrote: > Looks good and trivial. @RealFYang @jnimeh Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/17646#issuecomment-2043998825

Integrated: 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information

2024-01-31 Thread SendaoYan
On Wed, 31 Jan 2024 08:19:55 GMT, SendaoYan wrote: > 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect > comment information This pull request has now been integrated. Changeset: 432756b6 Author:SendaoYan Committer: Fei Yang URL:

Re: RFR: 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information

2024-01-31 Thread SendaoYan
On Wed, 31 Jan 2024 17:48:19 GMT, Jamil Nimeh wrote: > Looks good, but please label the JBS bug with noreg-trivial. Done - PR Comment: https://git.openjdk.org/jdk/pull/17646#issuecomment-1920357757

RFR: 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information

2024-01-31 Thread SendaoYan
8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information - Commit messages: - 8325024: java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java incorrect comment information Changes: https://git.openjdk.org/jdk/pull/17646/files Webrev: h