| commit | 10e45230940bc0db43945d7b1445557e1405ebe0 | [log] [tgz] |
|---|---|---|
| author | Jason Evans <je@fb.com> | Fri Jan 14 17:27:44 2011 -0800 |
| committer | Jason Evans <je@fb.com> | Fri Jan 14 17:27:44 2011 -0800 |
| tree | dc312549ab51e7c968817084f347f80a1598677e | |
| parent | 624f2f3cc9e6aa2a5eab18bd1eb84f89a6ac8b5c [diff] |
Fix a heap dumping deadlock. Restructure the ctx initialization code such that the ctx isn't locked across portions of the initialization code where allocation could occur. Instead artificially inflate the cnt_merged.curobjs field, just as is done elsewhere to avoid similar races to the one that would otherwise be created by the reduction in locking scope. This bug affected interval- and growth-triggered heap dumping, but not manual heap dumping.