RFR 15 8243099: SO_INCOMING_NAPI_ID support

Viswanathan, Sandhya sandhya.viswanathan at intel.com
Thu May 21 15:59:10 UTC 2020


Hi Daniel,

Vladimir just pointed our that I missed pushing the SO_INCOMING_NAPI_ID tests (forgot to do hg add).
I will create a separate JBS entry for tests. Vladimir could do this change as part of that.

Best Regards,
Sandhya

-----Original Message-----
From: net-dev <net-dev-bounces at openjdk.java.net> On Behalf Of Daniel Fuchs
Sent: Thursday, May 21, 2020 2:39 AM
To: Ivanov, Vladimir A <vladimir.a.ivanov at intel.com>; Patrick Concannon <patrick.concannon at oracle.com>; Alan Bateman <Alan.Bateman at oracle.com>; OpenJDK Network Dev list <net-dev at openjdk.java.net>
Subject: Re: RFR 15 8243099: SO_INCOMING_NAPI_ID support

Hi Vladimir,

On 14/05/2020 22:50, Ivanov, Vladimir A wrote:
> Thanks a lot Daniel! I missed these double checks.
> Updated webrev may be reviewed  as 
> http://cr.openjdk.java.net/~sviswanathan/Vladimir/8243099/webrev.14
> I use only one condition for the 'if' in the 'startup' method while kernel should support or not both types of sockets together.

Still one small mistake I missed in my previous review - sorry about that.

For the new tests under jdk/net/ExtendedSocketOption/ these are testng tests. They should be throwing `org.testng.SkipException` not `jtreg.SkippedException` in their @BeforeTest method, that is what the testng machinery expects.

   - import jtreg.SkippedException;
   + import org.testng.SkippedException;

...

   - throw new SkippedException("NAPI ID not supported on this system");
   + throw new SkipException("NAPI ID not supported on this system");

best regards,

-- daniel


More information about the net-dev mailing list