[Bug libgcj/22189] New: Table Full in gcj-dbtool if -m option used with smallest possible input

[Bug libgcj/22189] New: Table Full in gcj-dbtool if -m option used with smallest possible input

[greenrd at greenrd dot org]

The attached test case aot-compiles HelloWorld.java, builds a .db file, and then
"merges" that single .db file into another .db file. (Because it is only
"merging" a single file, and the destination .db does not exist, it is in effect
a copy rather than a merge.) But gcj-dbtool fails with this error message:

java.lang.IllegalAccessException: Table Full: 0
   at gnu.gcj.runtime.PersistentByteMap.put(byte[], byte[])
(/usr/lib/libgcj.so.6.0.0)
   at
gnu.gcj.runtime.PersistentByteMap.putAll(gnu.gcj.runtime.PersistentByteMap)
(/usr/lib/libgcj.so.6.0.0)
   at gnu.java.lang.MainThread.call_main() (/usr/lib/libgcj.so.6.0.0)
   at gnu.java.lang.MainThread.run() (/usr/lib/libgcj.so.6.0.0)

This is with gcj-dbtool (GNU libgcj) 4.0.0 20050622 (Red Hat 4.0.0-13)

-- 
           Summary: Table Full in gcj-dbtool if -m option used with smallest
                    possible input
           Product: gcc
           Version: 4.0.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: libgcj
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: greenrd at greenrd dot org
                CC: gcc-bugs at gcc dot gnu dot org,java-prs at gcc dot gnu
                    dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189

[Bug libgcj/22189] Table Full in gcj-dbtool if -m option used with smallest possible input

[greenrd at greenrd dot org]

------- Additional Comments From greenrd at greenrd dot org  2005-06-26 02:42 -------
Created an attachment (id=9151)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9151&action=view)
test case


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189

[Bug libgcj/22189] Table Full in gcj-dbtool if -m option used with smallest possible input

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-06-26 02:48 -------
Confirmed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-06-26 02:48:41
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189

[Bug libgcj/22189] Table Full in gcj-dbtool if -m option used with smallest possible input

[greenrd at greenrd dot org]

------- Additional Comments From greenrd at greenrd dot org  2005-06-26 12:39 -------
The problem is that a PersistentByteMap of capacity 1 is created by -m, but when
a PersistentByteMap is created with capacity 1, its capacity method incorrectly
returns 0.

This is because, when it is created, confusingly, the capacity field is set to
(int) (capacity*3/2),which in this case is 1, and the capacity() method then
returns (int) (the capacity field * 2 / 3), which is (int) (2 * 1 / 3) = 0.

The first thing I would do is, we have 3 things (initial capacity, capacity
field, capacity method), all with the same name, but not meaning the same thing,
so I would do some renames to avoid confusion. Then I would fix the actual bug.

Preparing a patch.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |greenrd at greenrd dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189

[Bug libgcj/22189] Table Full in gcj-dbtool if -m option used with smallest possible input

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-06-28 14:31 -------
Subject: Bug 22189

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	aph@gcc.gnu.org	2005-06-28 14:31:29

Modified files:
	libjava/gnu/gcj/runtime: PersistentByteMap.java 
	libjava        : ChangeLog 

Log message:
	2005-06-28  Robin Green  <greenrd@greenrd.org>
	
	PR java/22189
	* gnu/gcj/runtime/PersistentByteMap.java (init): Fix rounding
	error.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/gnu/gcj/runtime/PersistentByteMap.java.diff?cvsroot=gcc&r1=1.5&r2=1.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/ChangeLog.diff?cvsroot=gcc&r1=1.3680&r2=1.3681



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189

[Bug libgcj/22189] Table Full in gcj-dbtool if -m option used with smallest possible input

[greenrd at greenrd dot org]

------- Additional Comments From greenrd at greenrd dot org  2005-08-09 00:09 -------
Fixed on head (weeks ago).

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.1.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22189