Just to be clear, the changes that have been put in place for the manifest cache are also a workaround. It doesn't address the root the underlying cause of the problem.
-j On Fri, Oct 24, 2008 at 08:44:28AM +0100, jmr wrote: > This webrev of actually rolls back Michals actions:__init__.py to r590. > I'm not sure if this is what Danek wants us to do, or if he wants to tweak > the code himself. Reverting is just a workaround and does not get to the > root cause of the memory leak, but given the time scales this may be our > only option at present. > > Danek, do you want this revert of actions:__init__.py to r590 to happen or > not? > > Michal can you respin this webrev without the reversion of > actions:__init__.py to r590 so we have two options available. > > Thanks. > > JR > > jmr wrote: >> Hi - these changes by Michal have been under discussion over the past week >> on the bug 4116 and reviewed by Brock and myself several times. This >> webrev addresses the issue from the PM and UM side of things by making >> various optimizations, including turning off the manifest cache only for >> PM and UM, and using Brock's new actions support in the api to further >> reduce the need to access the manifest directly. >> >> http://cr.opensolaris.org/~migi/23_10_2008_bug_4116_v1/ >> <http://cr.opensolaris.org/%7Emigi/23_10_2008_bug_4116_v1/> >> >> All the details including analysis are in the bug: >> http://defect.opensolaris.org/bz/show_bug.cgi?id=4116 >> >> Running PM with the gate: >> Mem: 600M >> >> Running PM with this webrev alone against gate: >> Mem: 480M >> >> Running PM with this webrev and actions:__init__.py at rev590 against gate: >> Mem: 180M >> >> There is still the outstanding issue of a memory increase caused by >> introduction of changes going from r590 to r591 to actions:__inti__.py >> that Danek is investigating. This is a stopper bug for PM and the 2008.11 >> release. >> >> JR >> >> >> <http://cr.opensolaris.org/%7Emigi/23_10_2008_bug_4116_v1/> >> >> _______________________________________________ >> pkg-discuss mailing list >> [email protected] >> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >> _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
