Revert "Summary: Disable JFR support in windows/mac build"

This reverts commit 174fa73f.

make/bsd/makefiles/buildtree.make

@@ -102,8 +102,6 @@ endif
 
 ALWAYS_EXCLUDE_DIRS     = $(SCM_DIRS)
 
-ALWAYS_EXCLUDE_DIRS += -o -name jfr
-
 ifeq		($(VARIANT),tiered)
 TOPLEVEL_EXCLUDE_DIRS	= $(ALWAYS_EXCLUDE_DIRS) -o -name adlc -o -name agent
 else

make/bsd/makefiles/trace.make

@@ -51,9 +51,7 @@ VPATH += $(Src_Dirs_V:%=%:)
 TraceGeneratedNames =     \
     traceEventClasses.hpp \
 	traceEventIds.hpp     \
-	traceTypes.hpp \
-	traceEventControl.hpp \
-	tracePeriodic.hpp
+	traceTypes.hpp
 
 ifeq ($(HAS_ALT_SRC), true)
 TraceGeneratedNames +=  \
@@ -94,12 +92,6 @@ $(TraceOutDir)/traceEventIds.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/traceE
 $(TraceOutDir)/traceTypes.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/traceTypes.xsl $(XML_DEPS)
 	$(GENERATE_CODE)
 
-$(TraceOutDir)/traceEventControl.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/traceEventControl.xsl $(XML_DEPS)
-	$(GENERATE_CODE)
-
-$(TraceOutDir)/tracePeriodic.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/tracePeriodic.xsl $(XML_DEPS)
-	$(GENERATE_CODE)
-
 ifeq ($(HAS_ALT_SRC), false)
 
 $(TraceOutDir)/traceEventClasses.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/traceEventClasses.xsl $(XML_DEPS)

make/bsd/makefiles/vm.make

@@ -38,8 +38,6 @@ default: build
 GENERATED     = ../generated
 DEP_DIR       = $(GENERATED)/dependencies
 
-EXCLUDE_JFR_PATHS:= -o -name jfr -prune
-
 # reads the generated files defining the set of .o's and the .o .h dependencies
 -include $(DEP_DIR)/*.d
 
@@ -50,7 +48,6 @@ else
   include $(MAKEFILES_DIR)/$(BUILDARCH).make
 endif
 
-
 # set VPATH so make knows where to look for source files
 # Src_Dirs_V is everything in src/share/vm/*, plus the right os/*/vm and cpu/*/vm
 # The adfiles directory contains ad_<arch>.[ch]pp.
@@ -171,7 +168,7 @@ SPECIAL_PATHS:=adlc c1 gc_implementation opto shark libadt
 
 SOURCE_PATHS=\
   $(shell find $(HS_COMMON_SRC)/share/vm/* -type d \! \
-      \( -name DUMMY $(foreach dir,$(SPECIAL_PATHS),-o -name $(dir)) $(EXCLUDE_JFR_PATHS) \))
+      \( -name DUMMY $(foreach dir,$(SPECIAL_PATHS),-o -name $(dir)) \))
 SOURCE_PATHS+=$(HS_COMMON_SRC)/os/$(Platform_os_family)/vm
 SOURCE_PATHS+=$(HS_COMMON_SRC)/os/posix/vm
 SOURCE_PATHS+=$(HS_COMMON_SRC)/cpu/$(Platform_arch)/vm
@@ -180,6 +177,12 @@ SOURCE_PATHS+=$(HS_COMMON_SRC)/os_cpu/$(Platform_os_arch)/vm
 CORE_PATHS=$(foreach path,$(SOURCE_PATHS),$(call altsrc,$(path)) $(path))
 CORE_PATHS+=$(GENERATED)/jvmtifiles $(GENERATED)/tracefiles
 
+ifneq ($(INCLUDE_TRACE), false)
+CORE_PATHS+=$(shell if [ -d $(HS_ALT_SRC)/share/vm/jfr ]; then \
+  find $(HS_ALT_SRC)/share/vm/jfr -type d; \
+  fi)
+endif
+
 COMPILER1_PATHS := $(call altsrc,$(HS_COMMON_SRC)/share/vm/c1)
 COMPILER1_PATHS += $(HS_COMMON_SRC)/share/vm/c1
 

make/windows/create_obj_files.sh

@@ -58,7 +58,7 @@ COMMONSRC=${WorkSpace}/${COMMONSRC_REL}
 ALTSRC=${WorkSpace}/${ALTSRC_REL}
 
 BASE_PATHS="`if [ -d ${ALTSRC}/share/vm ]; then $FIND ${ALTSRC}/share/vm ! -name vm -prune -type d \! \( -name adlc -o -name c1 -o -name gc_implementation -o -name opto -o -name shark -o -name libadt \); fi`"
-BASE_PATHS="${BASE_PATHS} ` $FIND ${COMMONSRC}/share/vm ! -name vm -prune -type d \! \( -name adlc -o -name c1 -o -name gc_implementation -o -name opto -o -name shark -o -name libadt -o -name jfr \)`"
+BASE_PATHS="${BASE_PATHS} ` $FIND ${COMMONSRC}/share/vm ! -name vm -prune -type d \! \( -name adlc -o -name c1 -o -name gc_implementation -o -name opto -o -name shark -o -name libadt \)`"
 
 for sd in \
     share/vm/gc_implementation/shared \

make/windows/makefiles/trace.make

@@ -38,9 +38,7 @@ TraceSrcDir = $(WorkSpace)/src/share/vm/trace
 TraceGeneratedNames =     \
     traceEventClasses.hpp \
     traceEventIds.hpp     \
-    traceTypes.hpp        \
-    traceEventControl.hpp \
-    tracePeriodic.hpp
+    traceTypes.hpp
 
 !if EXISTS($(TraceAltSrcDir))
 TraceGeneratedNames = $(TraceGeneratedNames) \
@@ -55,9 +53,7 @@ TraceGeneratedNames = $(TraceGeneratedNames) \
 TraceGeneratedFiles = \
     $(TraceOutDir)/traceEventClasses.hpp \
 	$(TraceOutDir)/traceEventIds.hpp     \
-	$(TraceOutDir)/traceTypes.hpp \
-        $(TraceOutDir)/traceEventControl.hpp \
-        $(TraceOutDir)/tracePeriodic.hpp
+	$(TraceOutDir)/traceTypes.hpp
 
 !if EXISTS($(TraceAltSrcDir))
 TraceGeneratedFiles = $(TraceGeneratedFiles) \
@@ -69,8 +65,7 @@ TraceGeneratedFiles = $(TraceGeneratedFiles) \
 XSLT = $(QUIETLY) $(REMOTE) $(RUN_JAVA) -classpath $(JvmtiOutDir) jvmtiGen
 
 XML_DEPS = $(TraceSrcDir)/trace.xml $(TraceSrcDir)/tracetypes.xml \
-        $(TraceSrcDir)/tracerelationdecls.xml \
-        $(TraceSrcDir)/traceevents.xml
+    $(TraceSrcDir)/trace.dtd $(TraceSrcDir)/xinclude.mod
 
 !if EXISTS($(TraceAltSrcDir))
 XML_DEPS = $(XML_DEPS) $(TraceAltSrcDir)/traceevents.xml
@@ -97,14 +92,6 @@ $(TraceOutDir)/traceEventClasses.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/tr
 	@echo Generating OpenJDK $@
 	@$(XSLT) -IN $(TraceSrcDir)/trace.xml -XSL $(TraceSrcDir)/traceEventClasses.xsl -OUT $(TraceOutDir)/traceEventClasses.hpp
 
-$(TraceOutDir)/traceEventControl.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/traceEventClasses.xsl $(XML_DEPS)
-	@echo Generating $@
-	@$(XSLT) -IN $(TraceSrcDir)/trace.xml -XSL $(TraceSrcDir)/traceEventControl.xsl -OUT $(TraceOutDir)/traceEventControl.hpp
-
-$(TraceOutDir)/tracePeriodic.hpp: $(TraceSrcDir)/trace.xml $(TraceSrcDir)/tracePeriodic.xsl $(XML_DEPS)
-	@echo Generating $@
-	@$(XSLT) -IN $(TraceSrcDir)/trace.xml -XSL $(TraceSrcDir)/tracePeriodic.xsl -OUT $(TraceOutDir)/tracePeriodic.hpp
-
 !else
 
 $(TraceOutDir)/traceEventClasses.hpp: $(TraceSrcDir)/trace.xml $(TraceAltSrcDir)/traceEventClasses.xsl $(XML_DEPS)

src/cpu/x86/vm/rdtsc_x86.cpp

@@ -32,14 +32,6 @@
 #include "os_linux_x86.inline.hpp"
 #endif
 
-#ifdef TARGET_OS_ARCH_windows_x86
-#include "os_windows_x86.inline.hpp"
-#endif
-
-#ifdef TARGET_OS_ARCH_bsd_x86
-#include "os_bsd_x86.inline.hpp"
-#endif
-
 static jlong baseline_counter = 0;
 static bool rdtsc_elapsed_counter_enabled = false;
 static jlong tsc_frequency = 0;

src/share/vm/opto/superword.hpp

@@ -187,29 +187,6 @@ public:
 
 
 // ========================= SuperWord =====================
-//------------------------------OrderedPair---------------------------
-// Ordered pair of Node*.
-class OrderedPair VALUE_OBJ_CLASS_SPEC {
- protected:
-  Node* _p1;
-  Node* _p2;
- public:
-  OrderedPair() : _p1(NULL), _p2(NULL) {}
-  OrderedPair(Node* p1, Node* p2) {
-    if (p1->_idx < p2->_idx) {
-      _p1 = p1; _p2 = p2;
-    } else {
-      _p1 = p2; _p2 = p1;
-    }
-  }
-
-  bool operator==(const OrderedPair &rhs) {
-    return _p1 == rhs._p1 && _p2 == rhs._p2;
-  }
-  void print() { tty->print("  (%d, %d)", _p1->_idx, _p2->_idx); }
-
-  static const OrderedPair initial;
-};
 
 // -----------------------------SWNodeInfo---------------------------------
 // Per node info needed by SuperWord
@@ -528,4 +505,29 @@ class SWPointer VALUE_OBJ_CLASS_SPEC {
   void print();
 };
 
+
+//------------------------------OrderedPair---------------------------
+// Ordered pair of Node*.
+class OrderedPair VALUE_OBJ_CLASS_SPEC {
+ protected:
+  Node* _p1;
+  Node* _p2;
+ public:
+  OrderedPair() : _p1(NULL), _p2(NULL) {}
+  OrderedPair(Node* p1, Node* p2) {
+    if (p1->_idx < p2->_idx) {
+      _p1 = p1; _p2 = p2;
+    } else {
+      _p1 = p2; _p2 = p1;
+    }
+  }
+
+  bool operator==(const OrderedPair &rhs) {
+    return _p1 == rhs._p1 && _p2 == rhs._p2;
+  }
+  void print() { tty->print("  (%d, %d)", _p1->_idx, _p2->_idx); }
+
+  static const OrderedPair initial;
+};
+
 #endif // SHARE_VM_OPTO_SUPERWORD_HPP

src/share/vm/runtime/arguments.cpp

@@ -2665,13 +2665,11 @@ bool Arguments::check_vm_args_consistency() {
   // Check the minimum number of compiler threads
   status &=verify_min_value(CICompilerCount, min_number_of_compiler_threads, "CICompilerCount");
 
-#if INCLUDE_TRACE
   if ((FlightRecorder || StartFlightRecording != NULL) && !EnableJFR) {
     jio_fprintf(defaultStream::error_stream(),
                 "The VM option -XX:+FlightRecorder or -XX:StartFlightRecording=... must be combined with -XX:+EnableJFR.\n");
     status = false;
   }
-#endif
 
   return status;
 }

src/share/vm/utilities/macros.hpp

@@ -161,11 +161,7 @@
 #endif // INCLUDE_NMT
 
 #ifndef INCLUDE_TRACE
-#ifdef LINUX
 #define INCLUDE_TRACE 1
-#else
-#define INCLUDE_TRACE 0
-#endif
 #endif // INCLUDE_TRACE
 
 // COMPILER1 variant

src/share/vm/utilities/ticks.hpp

@@ -116,7 +116,12 @@ class TraceElapsedCounterStamped : public TraceElapsedCounter {
   TraceElapsedCounterStamped();
 };
 
+#if INCLUDE_TRACE
 typedef TraceElapsedCounter  Ticks;
 typedef TraceElapsedInterval Tickspan;
+#else
+typedef ElapsedCounter       Ticks;
+typedef TicksInterval<Ticks> Tickspan;
+#endif
 
 #endif // SHARE_VM_UTILITIES_TICKS_HPP