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
