8175542: JMX: Not enough JDP packets received

Fixed test case wrongly reporting timeout failures.

Reviewed-by: dholmes, hb
This commit is contained in:
Amit Sapre 2018-01-16 12:38:21 +05:30 committed by Amit Sapre
parent 5b466bcb74
commit 9820ed61cd
2 changed files with 7 additions and 4 deletions

View File

@ -1,6 +1,6 @@
########################################################################### ###########################################################################
# #
# Copyright (c) 2009, 2017, Oracle and/or its affiliates. All rights reserved. # Copyright (c) 2009, 2018, Oracle and/or its affiliates. All rights reserved.
# DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
# #
# This code is free software; you can redistribute it and/or modify it # This code is free software; you can redistribute it and/or modify it
@ -197,7 +197,6 @@ java/io/pathNames/GeneralWin32.java 8180264 windows-
com/sun/management/OperatingSystemMXBean/GetProcessCpuLoad.java 8030957 aix-all com/sun/management/OperatingSystemMXBean/GetProcessCpuLoad.java 8030957 aix-all
com/sun/management/OperatingSystemMXBean/GetSystemCpuLoad.java 8030957 aix-all com/sun/management/OperatingSystemMXBean/GetSystemCpuLoad.java 8030957 aix-all
sun/management/HotspotRuntimeMBean/GetSafepointSyncTime.java 8174734 generic-all sun/management/HotspotRuntimeMBean/GetSafepointSyncTime.java 8174734 generic-all
sun/management/jdp/JdpOffTest.java 8175542 generic-all
############################################################################ ############################################################################

View File

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2013, Oracle and/or its affiliates. All rights reserved. * Copyright (c) 2013, 2018, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
* *
* This code is free software; you can redistribute it and/or modify it * This code is free software; you can redistribute it and/or modify it
@ -77,11 +77,15 @@ public abstract class JdpTestCase {
onSocketTimeOut(e); onSocketTimeOut(e);
} }
if (!shouldContinue()) {
break;
}
if (hasTestLivedLongEnough()) { if (hasTestLivedLongEnough()) {
shutdown(); shutdown();
} }
} while (shouldContinue()); } while (true);
log.fine("Test ended successfully."); log.fine("Test ended successfully.");
} }