Bug 183962

Summary:	-ms and -mx options should be removed from startconsole java invocation
Product:	[Retired] 389	Reporter:	Patrick Macdonald <patrickm>
Component:	Directory Console	Assignee:	Rich Megginson <rmeggins>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Viktor Ashirov <vashirov>
Severity:	medium	Docs Contact:
Priority:	high
Version:	1.0	CC:	etraitel, herrold, nkinder
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-12-07 17:06:24 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	152373, 240316, 427409

Description Thomas Fitzsimmons 2006-03-04 03:22:09 UTC

Description of problem:

The value passed to -mx is not big enough for gij, which fails with garbage
collector out-of-memory errors.

Version-Release number of selected component (if applicable):

1.0.2

How reproducible:

Every time.

Steps to Reproduce:
1. Ensure java-gcj-compat is your currently selected java alternative.
2. Run startconsole.
  
Actual results:

gij fails to load the directory console and prints the following warning:

GC Warning: Out of Memory!  Returning NIL!

Expected results:

gij loads the directory console successfully.

Additional info:

On the fedora-directory-devel list Richard asked:

1) Is there a reliable way to tell the memory capabilities of a JVM?

No, not prior to running the JVM and not without profiling it.

2) Do we still need to use those options on 1.4.2 or later JVMs?

No, no 1.4.2 JVM should require setting the -ms nor the -mx option.

Which FDS-supported JVMs require these options?  I think these JVMs should be
tested for and the -ms and -mx options used only on them.

Comment 1 Rich Megginson 2006-03-04 03:47:27 UTC

We've been using these options for a long, long time.  We probably have memory
leak issues in the console that cause the jvm to run OOM when running for a long
time (a couple of days), so I don't know if these options were added to help
with that situation.  I would rather just get rid of them if we don't need them
anymore.

Comment 2 Eyal Traitel 2006-05-12 20:53:45 UTC

I've seen the same problem on FC5.
I was able to workaround this just by changing to -ms= and -mx= - the gij
options seem to require a = before the value.

Comment 3 Thomas Fitzsimmons 2006-12-08 16:32:59 UTC

(In reply to comment #1)
> We've been using these options for a long, long time.  We probably have memory
> leak issues in the console that cause the jvm to run OOM when running for a long
> time (a couple of days), so I don't know if these options were added to help
> with that situation.  I would rather just get rid of them if we don't need them
> anymore.

Any progress on this?

Comment 4 Rich Megginson 2006-12-08 16:36:28 UTC

> Any progress on this?

Not yet.  We have some time scheduled early next year for console issues, as
part of the RH DirSvr 7.2 work.

Comment 5 Rich Megginson 2007-10-17 18:31:39 UTC

Just a note - the Fedora DS console will simply not work with gcj, and probably
will never work with gcj.  Anyone who wants to use it will have to use a
proprietary jvm, or wait for icedtea/openjdk.

Comment 6 Nathan Kinder 2007-10-17 18:40:39 UTC

These diffs remove the -ms and -mx options from the fedora-idm-console start script.

Index: fedora-idm-console
===================================================================
RCS file: /cvs/dirsec/fedora-idm-console/fedora-idm-console,v
retrieving revision 1.1.1.1
diff -u -5 -t -r1.1.1.1 fedora-idm-console
--- fedora-idm-console  1 Aug 2007 23:08:51 -0000       1.1.1.1
+++ fedora-idm-console  17 Oct 2007 18:39:08 -0000
@@ -29,6 +29,6 @@
 CLASSDEST=@classdest@
 
 #
 # Launch the Console
 #
-java -ms8m -mx64m -cp
@jssjar@:@ldapjdkjar@:$CLASSDEST/@basejar@:$CLASSDEST/@mccjar@:$CLASSDEST/@mcclangjar@:$CLASSDEST/@nmclfjar@:$CLASSDEST/@nmclflangjar@:$CLASSDEST/@themejar@
-Djava.library.path=@libdir@ -Djava.util.prefs.systemRoot="$HOME/.@prefsdir@"
-Djava.util.prefs.userRoot="$HOME/.@prefsdir@"
com.netscape.management.client.console.Console $*
+java -cp
@jssjar@:@ldapjdkjar@:$CLASSDEST/@basejar@:$CLASSDEST/@mccjar@:$CLASSDEST/@mcclangjar@:$CLASSDEST/@nmclfjar@:$CLASSDEST/@nmclflangjar@:$CLASSDEST/@themejar@
-Djava.library.path=@libdir@ -Djava.util.prefs.systemRoot="$HOME/.@prefsdir@"
-Djava.util.prefs.userRoot="$HOME/.@prefsdir@"
com.netscape.management.client.console.Console $*

Comment 7 Noriko Hosoi 2007-10-17 22:10:41 UTC

Looks good.

Comment 8 Nathan Kinder 2007-10-17 22:20:29 UTC

Checked into fedora-idm-console (HEAD).  Thanks to Noriko for her review!

Checking in fedora-idm-console;
/cvs/dirsec/fedora-idm-console/fedora-idm-console,v  <--  fedora-idm-console
new revision: 1.2; previous revision: 1.1
done