New issue
Advanced search Search tips

Issue 799392 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2018
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Chromium startup page crashed on Loongson(mips64el) platform.

Reported by wangqing...@loongson.cn, Jan 5 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Linux mips64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3096.2 Safari/537.36

Steps to reproduce the problem:
1. Build chromium on Loongson
2. Run chrome binary

What is the expected behavior?
Chromium startup page normally on Loongson.

What went wrong?
Chromium startup page crashed!

Did this work before? N/A 

Chrome version: 60.0.3096.2  Channel: n/a
OS Version: Fedora21
Flash Version:
 
Initial patch that Fix PartitionAlloc does not work properly on Loongson platform has just been submitted for a review.

It can be viewed at:
https://chromium-review.googlesource.com/c/chromium/src/+/851839
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 8 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0752dbc988222d7edef2ef7c4e19555089c35fdf

commit 0752dbc988222d7edef2ef7c4e19555089c35fdf
Author: Wang Qing <wangqing-hf@loongson.cn>
Date: Mon Jan 08 04:40:12 2018

Fix PartitionAlloc does not work properly on Loongson platform.

Because system pagesize is 4K on general system, such as x86 etc.
But Loongson have 16K system pagesize and chromium startup page
crashed when use PartitionAlloc.

Therefore, the relevant pagesize places needs to be changed.

BUG=  799392 

R= haraken@chromium.org

Change-Id: Id6900d136dc4c4ec870c355eea92949c044e05bb
Reviewed-on: https://chromium-review.googlesource.com/851839
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: 汪 清 <wangqing-hf@loongson.cn>
Cr-Commit-Position: refs/heads/master@{#527569}
[modify] https://crrev.com/0752dbc988222d7edef2ef7c4e19555089c35fdf/base/allocator/partition_allocator/page_allocator.h
[modify] https://crrev.com/0752dbc988222d7edef2ef7c4e19555089c35fdf/base/allocator/partition_allocator/partition_alloc.h

Status: Fixed (was: Unconfirmed)

Sign in to add a comment