Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8257228: G1: SIGFPE in G1ConcurrentRefine::create(int*) due to buffer…
…s_to_cards overflow

Reviewed-by: kbarrett, tschatzl
  • Loading branch information
DamonFool committed Dec 2, 2020
1 parent fe5cccc commit f2a0988
Show file tree
Hide file tree
Showing 2 changed files with 64 additions and 7 deletions.
25 changes: 18 additions & 7 deletions src/hotspot/share/gc/g1/g1ConcurrentRefine.cpp
Expand Up @@ -34,6 +34,7 @@
#include "runtime/java.hpp"
#include "runtime/thread.hpp"
#include "utilities/debug.hpp"
#include "utilities/formatBuffer.hpp"
#include "utilities/globalDefinitions.hpp"
#include "utilities/pair.hpp"
#include <math.h>
Expand Down Expand Up @@ -183,8 +184,16 @@ STATIC_ASSERT(max_yellow_zone <= max_red_zone);
// For logging zone values, ensuring consistency of level and tags.
#define LOG_ZONES(...) log_debug( CTRL_TAGS )(__VA_ARGS__)

static size_t buffers_to_cards(size_t value) {
return value * G1UpdateBufferSize;
// Convert configuration values in units of buffers to number of cards.
static size_t configuration_buffers_to_cards(size_t value, const char* value_name) {
if (value == 0) return 0;
size_t res = value * G1UpdateBufferSize;

if (res / value != G1UpdateBufferSize) { // Check overflow
vm_exit_during_initialization(err_msg("configuration_buffers_to_cards: "
"(%s = " SIZE_FORMAT ") * (G1UpdateBufferSize = " SIZE_FORMAT ") overflow!", value_name, value, G1UpdateBufferSize));
}
return res;
}

// Package for pair of refinement thread activation and deactivation
Expand All @@ -206,7 +215,7 @@ static Thresholds calc_thresholds(size_t green_zone,
// doing any processing, as that might lead to significantly more
// than green_zone buffers to be processed during pause. So limit
// to an extra half buffer per pause-time processing thread.
step = MIN2(step, buffers_to_cards(ParallelGCThreads) / 2.0);
step = MIN2(step, configuration_buffers_to_cards(ParallelGCThreads, "ParallelGCThreads") / 2.0);
}
size_t activate_offset = static_cast<size_t>(ceil(step * (worker_id + 1)));
size_t deactivate_offset = static_cast<size_t>(floor(step * worker_id));
Expand All @@ -232,7 +241,7 @@ jint G1ConcurrentRefine::initialize() {
}

static size_t calc_min_yellow_zone_size() {
size_t step = buffers_to_cards(G1ConcRefinementThresholdStep);
size_t step = configuration_buffers_to_cards(G1ConcRefinementThresholdStep, "G1ConcRefinementThresholdStep");
uint n_workers = G1ConcurrentRefine::max_num_threads();
if ((max_yellow_zone / step) < n_workers) {
return max_yellow_zone;
Expand All @@ -243,15 +252,17 @@ static size_t calc_min_yellow_zone_size() {

static size_t calc_init_green_zone() {
size_t green = G1ConcRefinementGreenZone;
const char* name = "G1ConcRefinementGreenZone";
if (FLAG_IS_DEFAULT(G1ConcRefinementGreenZone)) {
green = ParallelGCThreads;
name = "ParallelGCThreads";
}
green = buffers_to_cards(green);
green = configuration_buffers_to_cards(green, name);
return MIN2(green, max_green_zone);
}

static size_t calc_init_yellow_zone(size_t green, size_t min_size) {
size_t config = buffers_to_cards(G1ConcRefinementYellowZone);
size_t config = configuration_buffers_to_cards(G1ConcRefinementYellowZone, "G1ConcRefinementYellowZone");
size_t size = 0;
if (FLAG_IS_DEFAULT(G1ConcRefinementYellowZone)) {
size = green * 2;
Expand All @@ -266,7 +277,7 @@ static size_t calc_init_yellow_zone(size_t green, size_t min_size) {
static size_t calc_init_red_zone(size_t green, size_t yellow) {
size_t size = yellow - green;
if (!FLAG_IS_DEFAULT(G1ConcRefinementRedZone)) {
size_t config = buffers_to_cards(G1ConcRefinementRedZone);
size_t config = configuration_buffers_to_cards(G1ConcRefinementRedZone, "G1ConcRefinementRedZone");
if (yellow < config) {
size = MAX2(size, config - yellow);
}
Expand Down
46 changes: 46 additions & 0 deletions test/hotspot/jtreg/gc/g1/TestBuffersToCardsOverflow.java
@@ -0,0 +1,46 @@
/*
* Copyright (C) 2020 THL A29 Limited, a Tencent company. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/

/**
* @test
* @bug 8257228
* @library /test/lib
* @requires vm.bits == 64
* @build gc.g1.TestBuffersToCardsOverflow jdk.test.lib.process.*
* @run main gc.g1.TestBuffersToCardsOverflow
*/

package gc.g1;

import jdk.test.lib.process.ProcessTools;

public class TestBuffersToCardsOverflow {
public static void main(String... args) throws Exception {
ProcessTools.executeTestJava("-XX:G1ConcRefinementThresholdStep=16G",
"-XX:G1UpdateBufferSize=1G")
.outputTo(System.out)
.errorTo(System.out)
.stdoutShouldNotContain("SIGFPE")
.stdoutShouldNotContain("hs_err");
}
}

1 comment on commit f2a0988

@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.