[heap] Add two tiny free list categories.

[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3 words). This may reduce memory consumption (especially in map space), but may be worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add customizable free list categories for each old space.

BUG=chromium:587026
LOG=n

Committed: https://crrev.com/6b340ded84eb4c735253448454d505e15b410981
Cr-Commit-Position: refs/heads/master@{#34612}

[Hannes Payer (out of office), 2016-03-08 13:30:43]

Description was changed from

==========
[heap] Add a tiny free list category.

BUG=chromium:587026
LOG=n
==========

to

==========
[heap] Add a tiny free list category.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

BUG=chromium:587026
LOG=n
==========

[Hannes Payer (out of office), 2016-03-08 13:30:43]

hpayer@chromium.org changed reviewers:
+ mlippautz@chromium.org, ulan@chromium.org

[Hannes Payer (out of office), 2016-03-09 09:08:28]

Description was changed from

==========
[heap] Add a tiny free list category.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

BUG=chromium:587026
LOG=n
==========

to

==========
[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add
customizable free list categories for each old space.

BUG=chromium:587026
LOG=n
==========

[Michael Lippautz, 2016-03-09 09:22:11]

lgtm

Maybe update the CL description with the results obtained from local testing.

https://codereview.chromium.org/1774953003/diff/40001/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/1774953003/diff/40001/src/heap/spaces.h#newco...
src/heap/spaces.h:1791: // The tiny category is not used for fast allocation.
nit: *Both* tiny categories are not used.

[ulan, 2016-03-09 09:46:11]

lgtm

[Hannes Payer (out of office), 2016-03-09 11:04:53]

https://codereview.chromium.org/1774953003/diff/40001/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/1774953003/diff/40001/src/heap/spaces.h#newco...
src/heap/spaces.h:1791: // The tiny category is not used for fast allocation.
On 2016/03/09 09:22:11, Michael Lippautz wrote:
> nit: *Both* tiny categories are not used.

Done.

[Hannes Payer (out of office), 2016-03-09 11:43:47]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2016-03-09 11:43:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlippautz@chromium.org, ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/1774953003/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-03-09 11:43:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1774953003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1774953003/60001

[Hannes Payer (out of office), 2016-03-09 11:44:09]

The CQ bit was unchecked by hpayer@chromium.org

[Hannes Payer (out of office), 2016-03-09 11:55:50]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2016-03-09 11:55:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1774953003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1774953003/60001

[commit-bot: I haz the power, 2016-03-09 11:57:37]

Description was changed from

==========
[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add
customizable free list categories for each old space.

BUG=chromium:587026
LOG=n
==========

to

==========
[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add
customizable free list categories for each old space.

BUG=chromium:587026
LOG=n
==========

[commit-bot: I haz the power, 2016-03-09 11:57:38]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-09 11:58:09]

Description was changed from

==========
[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add
customizable free list categories for each old space.

BUG=chromium:587026
LOG=n
==========

to

==========
[heap] Add two tiny free list categories.

This CL allows the sweeper to free up all memory >= free list item size (3
words). This may reduce memory consumption (especially in map space), but may be
worse for allocation order as soon as we start using the tiny category.

This CL is just a first step in the right direction. A follow up CL will add
customizable free list categories for each old space.

BUG=chromium:587026
LOG=n

Committed: https://crrev.com/6b340ded84eb4c735253448454d505e15b410981
Cr-Commit-Position: refs/heads/master@{#34612}
==========

[commit-bot: I haz the power, 2016-03-09 11:58:10]

Patchset 4 (id:??) landed as
https://crrev.com/6b340ded84eb4c735253448454d505e15b410981
Cr-Commit-Position: refs/heads/master@{#34612}