PPC: Fixed RememberedSetRemoveRange to have chunk->size > PageSize

PPC: Fixed RememberedSetRemoveRange to have chunk->size > PageSize

Increased array size in the testcase such that the chunk size is
greater that the page size on PPC (4MB). This fixes the case when
(start + Page::kPageSize) memory location is tested to be part of
the chunk.
The testcase was added as part of https://codereview.chromium.org/2110213003/

R=ulan@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=
LOG=N

Committed: https://crrev.com/d05c2f02c5917c06373feeb3a6b4a8cf53f91b97
Cr-Commit-Position: refs/heads/master@{#38060}

[JaideepBajwa, 2016-07-26 13:41:39]

PTAL.

[ulan, 2016-07-26 13:45:32]

https://codereview.chromium.org/2182113002/diff/1/test/cctest/heap/test-heap.cc
File test/cctest/heap/test-heap.cc (right):

https://codereview.chromium.org/2182113002/diff/1/test/cctest/heap/test-heap....
test/cctest/heap/test-heap.cc:6842: Handle<FixedArray> array =
isolate->factory()->NewFixedArray(500000);
Instead of ifdef, maybe we can compute the number of elements based on
kPageSize?

[JaideepBajwa, 2016-07-26 14:22:53]

On 2016/07/26 13:45:32, ulan wrote:
>
https://codereview.chromium.org/2182113002/diff/1/test/cctest/heap/test-heap.cc
> File test/cctest/heap/test-heap.cc (right):
> 
>
https://codereview.chromium.org/2182113002/diff/1/test/cctest/heap/test-heap....
> test/cctest/heap/test-heap.cc:6842: Handle<FixedArray> array =
> isolate->factory()->NewFixedArray(500000);
> Instead of ifdef, maybe we can compute the number of elements based on
> kPageSize?

Thanks, fixed as suggested.

[ulan, 2016-07-26 14:43:36]

LGTM, thanks for fixing!

[JaideepBajwa, 2016-07-26 14:44:35]

The CQ bit was checked by bjaideep@ca.ibm.com

[commit-bot: I haz the power, 2016-07-26 14:44:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-26 15:07:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-26 15:10:23]

Description was changed from

==========
PPC: Fixed RememberedSetRemoveRange to have chunk->size > PageSize

Increased array size in the testcase such that the chunk size is
greater that the page size on PPC (4MB). This fixes the case when
(start + Page::kPageSize) memory location is tested to be part of
the chunk.
The testcase was added as part of https://codereview.chromium.org/2110213003/

R=ulan@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=
LOG=N
==========

to

==========
PPC: Fixed RememberedSetRemoveRange to have chunk->size > PageSize

Increased array size in the testcase such that the chunk size is
greater that the page size on PPC (4MB). This fixes the case when
(start + Page::kPageSize) memory location is tested to be part of
the chunk.
The testcase was added as part of https://codereview.chromium.org/2110213003/

R=ulan@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com, mbrandy@us.ibm.com

BUG=
LOG=N

Committed: https://crrev.com/d05c2f02c5917c06373feeb3a6b4a8cf53f91b97
Cr-Commit-Position: refs/heads/master@{#38060}
==========

[commit-bot: I haz the power, 2016-07-26 15:10:24]

Patchset 2 (id:??) landed as
https://crrev.com/d05c2f02c5917c06373feeb3a6b4a8cf53f91b97
Cr-Commit-Position: refs/heads/master@{#38060}