Skip to content

Commit

Permalink
8237479: 8230305 causes slowdebug build failure
Browse files Browse the repository at this point in the history
Declare methods as pure virtual.

Reviewed-by: sgehwolf, andrew
Backport-of: 4ca06995855b5c974321d7b3622d661b8d27ba76
  • Loading branch information
Jonathan Dowland committed Dec 8, 2022
1 parent 97d00ca commit 362cf76
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions hotspot/src/os/linux/vm/cgroupSubsystem_linux.hpp
Expand Up @@ -57,7 +57,7 @@ typedef char * cptr;

class CgroupController: public CHeapObj<mtInternal> {
public:
virtual char *subsystem_path();
virtual char *subsystem_path() = 0;
};

PRAGMA_DIAG_PUSH
Expand Down Expand Up @@ -246,19 +246,19 @@ class CgroupSubsystem: public CHeapObj<mtInternal> {
jlong memory_limit_in_bytes();
int active_processor_count();

virtual int cpu_quota();
virtual int cpu_period();
virtual int cpu_shares();
virtual jlong memory_usage_in_bytes();
virtual jlong memory_and_swap_limit_in_bytes();
virtual jlong memory_soft_limit_in_bytes();
virtual jlong memory_max_usage_in_bytes();
virtual char * cpu_cpuset_cpus();
virtual char * cpu_cpuset_memory_nodes();
virtual jlong read_memory_limit_in_bytes();
virtual const char * container_type();
virtual CachingCgroupController* memory_controller();
virtual CachingCgroupController* cpu_controller();
virtual int cpu_quota() = 0;
virtual int cpu_period() = 0;
virtual int cpu_shares() = 0;
virtual jlong memory_usage_in_bytes() = 0;
virtual jlong memory_and_swap_limit_in_bytes() = 0;
virtual jlong memory_soft_limit_in_bytes() = 0;
virtual jlong memory_max_usage_in_bytes() = 0;
virtual char * cpu_cpuset_cpus() = 0;
virtual char * cpu_cpuset_memory_nodes() = 0;
virtual jlong read_memory_limit_in_bytes() = 0;
virtual const char * container_type() = 0;
virtual CachingCgroupController* memory_controller() = 0;
virtual CachingCgroupController* cpu_controller() = 0;
};

class CgroupSubsystemFactory: AllStatic {
Expand Down

1 comment on commit 362cf76

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.