Fix PagedSpace size accounting.

Fix PagedSpace size accounting.

R=hpayer@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=22433

[Hannes Payer (out of office), 2014-07-16 12:09:19]

The accounting is still not correct. For example
static intptr_t Free(PagedSpace* space,
                     FreeList* free_list,
                     Address start,
                     int size)
if we don't sweep on the main thread we drop the wasted bytes on the floor. Can
you fix that in this cl as well?

[Yang, 2014-07-16 12:18:09]

On 2014/07/16 12:09:19, Hannes Payer wrote:
> The accounting is still not correct. For example
> static intptr_t Free(PagedSpace* space,
>                      FreeList* free_list,
>                      Address start,
>                      int size)
> if we don't sweep on the main thread we drop the wasted bytes on the floor.
Can
> you fix that in this cl as well?

Not sure how I would do that. In updating the stats in the else-case here, I
would basically call PagedSpace::Free. Is that what we want?

[Yang, 2014-07-16 13:03:47]

On 2014/07/16 12:18:09, Yang wrote:
> On 2014/07/16 12:09:19, Hannes Payer wrote:
> > The accounting is still not correct. For example
> > static intptr_t Free(PagedSpace* space,
> >                      FreeList* free_list,
> >                      Address start,
> >                      int size)
> > if we don't sweep on the main thread we drop the wasted bytes on the floor.
> Can
> > you fix that in this cl as well?
> 
> Not sure how I would do that. In updating the stats in the else-case here, I
> would basically call PagedSpace::Free. Is that what we want?

Added a TODO since tracking wasted bytes in concurrent sweeping is not trivial.

[Hannes Payer (out of office), 2014-07-16 13:34:06]

lgtm, please fix the TODO

https://codereview.chromium.org/392163003/diff/20001/src/mark-compact.cc
File src/mark-compact.cc (right):

https://codereview.chromium.org/392163003/diff/20001/src/mark-compact.cc#newc...
src/mark-compact.cc:3950: // TODO(hpayer) account for wasted bytes in concurrent
sweeping too.
TODO(hpayer)":"

[Yang, 2014-07-16 13:36:16]

https://codereview.chromium.org/392163003/diff/20001/src/mark-compact.cc
File src/mark-compact.cc (right):

https://codereview.chromium.org/392163003/diff/20001/src/mark-compact.cc#newc...
src/mark-compact.cc:3950: // TODO(hpayer) account for wasted bytes in concurrent
sweeping too.
On 2014/07/16 13:34:06, Hannes Payer wrote:
> TODO(hpayer)":"

Done.

[Yang, 2014-07-16 13:37:06]

Committed patchset #3 manually as r22433 (presubmit successful).