# HG changeset patch # User zgu # Date 1378299308 14400 # Node ID baa7927dfbd2d32b96520fd61b3308c1ba8f3a5a # Parent 35b99e7e0af2217d3336fe4e33e87dab02744a2e 8022798: "assert(seq > 0) failed: counter overflow" in Kitchensink Summary: Removed incorrect assertion, sequence number can overflow Reviewed-by: dholmes, kamg diff -r 35b99e7e0af2 -r baa7927dfbd2 src/share/vm/services/memPtr.cpp --- a/src/share/vm/services/memPtr.cpp Sun Sep 01 10:37:01 2013 -0400 +++ b/src/share/vm/services/memPtr.cpp Wed Sep 04 08:55:08 2013 -0400 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2012, 2013, Oracle and/or its affiliates. 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 @@ -34,9 +34,9 @@ jint seq = Atomic::add(1, &_seq_number); if (seq < 0) { MemTracker::shutdown(MemTracker::NMT_sequence_overflow); + } else { + NOT_PRODUCT(_max_seq_number = (seq > _max_seq_number) ? seq : _max_seq_number;) } - assert(seq > 0, "counter overflow"); - NOT_PRODUCT(_max_seq_number = (seq > _max_seq_number) ? seq : _max_seq_number;) return seq; }