Project

General

Profile

Bug #5831

Support #7559: Memory check issues for Sirius A

SimpleMemoryCheck affected by overflow errors

Added by Gianluca Petrillo over 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Infrastructure
Target version:
Start date:
04/04/2014
Due date:
% Done:

100%

Estimated time:
3.00 h
Spent time:
Occurs In:
Scope:
Internal
Experiment:
-
SSI Package:
art
Duration:

Description

It happens that the counters used in the SimpleMemoryCheck service code hit the 4Gi limit of the 32 bit integers and wrap up.
Please use the format specifiers suggested in `man 5 proc` to read /proc/PID/stat, and use matching data types.


Related issues

Related to art - Bug #2233: SimpleMemoryChecker needs to be able to cope with different procfs formats on different SLF versions.Closed11/29/201109/30/2013

History

#1 Updated by Christopher Green over 6 years ago

  • Subject changed from SimpleMemoryCkeck affected by overflow errors to SimpleMemoryCheck affected by overflow errors
  • Category set to Infrastructure
  • Status changed from New to Accepted
  • Target version set to 1.09.03
  • Estimated time set to 3.00 h

This (and the related issue #2233) should be addressed for the next release.

#2 Updated by Christopher Green over 6 years ago

  • Target version changed from 1.09.03 to 1.13.00

#3 Updated by Kyle Knoepfel almost 6 years ago

  • Parent task set to #7559

#4 Updated by Kyle Knoepfel almost 6 years ago

  • Assignee set to Kyle Knoepfel

#5 Updated by Kyle Knoepfel almost 6 years ago

  • Status changed from Accepted to Assigned

#6 Updated by Kyle Knoepfel almost 6 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

The integer types of vsize (unsigned long) and rss (long) are now compatible with those specified by procfs. Any wrapping issues due to exceeding the maximum allowed value of either type will now be due to limitations of the operating system.

Implemented with fbc282dd.

#7 Updated by Christopher Green almost 6 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF