[heap] Disallow allocation on the last page in address range

[heap] Disallow allocation on the last page in address range

BUG=

Committed: https://crrev.com/871d0603d8ba3a9777b8b0935885c8240962adda
Cr-Commit-Position: refs/heads/master@{#35656}

[Hannes Payer (out of office), 2016-04-19 12:25:24]

hpayer@chromium.org changed reviewers:
+ hpayer@chromium.org

[Hannes Payer (out of office), 2016-04-19 12:25:24]

LGTM
As discussed offline, I would move the logic to AllocateChunk which is a
lower-level bottleneck.

[Michael Lippautz, 2016-04-19 12:47:23]

The CQ bit was checked by mlippautz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-19 12:47:30]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1899183002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1899183002/40001

[commit-bot: I haz the power, 2016-04-19 13:13:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-19 13:13:05]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-04-19 18:06:47]

+danno, +bmeurer: fyi

general:
- mips uses unsigned arithmetic which should've always been safe as far as I see

[Michael Lippautz, 2016-04-19 18:07:25]

Description was changed from

==========
[heap] Disallow allocation of the last page in address range

BUG=
==========

to

==========
[heap] Disallow allocation on the last page in address range

BUG=
==========

[Michael Lippautz, 2016-04-20 11:58:29]

The CQ bit was checked by mlippautz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-20 11:58:39]

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

[commit-bot: I haz the power, 2016-04-20 12:28:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-20 12:28:15]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-04-20 12:59:24]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2016-04-20 12:59:25]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/1899183002/#ps60001
(title: "SetCC to LeaveCC on arm")

[commit-bot: I haz the power, 2016-04-20 12:59:26]

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

[commit-bot: I haz the power, 2016-04-20 13:01:25]

Description was changed from

==========
[heap] Disallow allocation on the last page in address range

BUG=
==========

to

==========
[heap] Disallow allocation on the last page in address range

BUG=
==========

[commit-bot: I haz the power, 2016-04-20 13:01:26]

Committed patchset #4 (id:60001)

[nodir1, 2016-04-22 18:44:06]

Description was changed from

==========
[heap] Disallow allocation on the last page in address range

BUG=
==========

to

==========
[heap] Disallow allocation on the last page in address range

BUG=

Committed: https://crrev.com/871d0603d8ba3a9777b8b0935885c8240962adda
Cr-Commit-Position: refs/heads/master@{#35656}
==========

[nodir1, 2016-04-22 18:44:07]

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