My favorites ▼ | Sign in
Project Home Downloads Wiki Issues
New issue   Search
for
  Advanced search   Search tips
Issue 92964: Native Client examples not running on build 14.0.835.94 beta
15 people starred this issue and may be notified of changes. Back to list
 
Reported by project member mbo...@chromium.org, Aug 15, 2011 
Chrome Version (from the about:version page): 14.0.835.94 beta
Is this the most recent version: Yes
OS + version: Linux 11.04
CPU architecture (32-bit / 64-bit): 32-bit
Window manager:
URLs (if relevant):
Behavior in Linux Firefox:
Behavior in Windows Chrome (if you have access to it):

What steps will reproduce the problem?
1. Run Native client examples http://4.nacl-gallery.appspot.com/index_test.html on build 14.0.835.94 beta
2.
3.

What is the expected result?


What happens instead?
They are not running. It shows Status message "Loading...". Last week it's working on dev and beta builds.

Before running these examples I cleared my browser cache and config files, checked enabling of Native Client plugins on about:flags, chrome://plugins.

Please provide any additional information below. Attach a screenshot
and backtrace if possible.
Comment 1 by cstefan...@google.com, Aug 15, 2011 
What Linux  distribution was this? (Fedora, Ubuntu, ...)

Do you see anything in the JavaScript console?
Labels: -Area-Undefined Area-Internals
Comment 2 by mbo...@chromium.org, Aug 15, 2011 
OS: Ubuntu

URL: http://4.nacl-gallery.appspot.com/index_test.html
For example "Hello World in C", script console displays message "Uncaught TypeError....". see attachment.
Comment 3 by mbo...@chromium.org, Aug 15, 2011 
(No comment was entered for this change.)
NaCl.png
109 KB   View   Download
Comment 4 by vivi...@chromium.org, Aug 15, 2011 
(No comment was entered for this change.)
Cc: anan...@chromium.org kerz@google.com vivi...@chromium.org
Comment 5 by kerz@google.com, Aug 15, 2011 
(No comment was entered for this change.)
Status: Assigned
Owner: cstefan...@google.com
Labels: -Pri-2 Pri-1 ReleaseBlock-Stable
Comment 6 by kerz@google.com, Aug 16, 2011 
(No comment was entered for this change.)
Labels: Mstone-14
Comment 7 by erik...@chromium.org, Aug 16, 2011 
This looks like either NaCl or the NaCl plugin aren't enabled for you.  Could you try going to http://krasin.github.com/naclwtf/ and tell us what it says?
Comment 8 by bradc...@google.com, Aug 16, 2011 
To reporter: did you restart the browser after enabling? about:flags doesn't take effect until the browser is restarted. The about:flags page has a restart page if you scroll to the bottom.
Comment 9 by mbo...@chromium.org, Aug 16, 2011 
Here is the info from page: http://4.nacl-gallery.appspot.com/index_test.html

browser is Chrome: OK
Chrome version >= 14: OK
about:plugins: OK
about:flags: OK
hello_world loaded:: OK

Currently If I try to access one of the NaCl examples, it still shows Loading... Yesterday I clicked "Relaunch Now" button after enabling Native Client from about:flags
Comment 10 by mbo...@chromium.org, Aug 16, 2011 
Here is the text displayed on Console: 
NativeClient: NaCl module load failed: Insufficient memory to load file
NativeClient: Log message that should show up on the JavaScript console

Attached log file tmp.txt
tmp.txt
33.3 KB   View   Download
Comment 11 by bradc...@google.com, Aug 16, 2011 
This appears to be the same thing we are seeing on Windows with ASLR and virtual address space fragmentation (nativeclient:2131). The CL in flight for nativeclient:480 should fix this problem, but probably hasn't had enough testing to be ready for beta branch. Looking into other options.

See the failed call to NaClAllocateSpace in the log file.
Comment 12 by cstefan...@google.com, Aug 16, 2011 
The original report was invalid (explanation below) and the new problem is a duplicate of nativeclient:2131.

As for the original report:
Unable to reproduce on Ubuntu 10.04 (32 bit) with Chrome 14.0.835.94 beta.

I was able to load the Hello World and Pi Generator examples from http://code.google.com/chrome/nativeclient/docs/examples.html without any problems. 

I do get the error reported, however, if I enter via http://4.nacl-gallery.appspot.com/index_test.html and NaCl is not enabled under about:flags, because that page does not check for NaCl being enabled. This is expected behavior.

Please check that NaCl is enabled under about:flags and use the official gallery landing page as it contains a couple of checks.

Closing as invalid.
Status: Duplicate
Mergedinto: nativeclient:2131
Comment 13 by erik...@chromium.org, Aug 16, 2011 
I think we want to keep this open as a separate bug.  The fix for each platform is likely to be different and the bug could regress independently as well.  It seems worthwhile to track separately.
Comment 14 by mbo...@chromium.org, Aug 16, 2011 
Brad, Here is the file.
m
23.0 KB   View   Download
Comment 15 by bradc...@google.com, Aug 16, 2011 
From discussion with core runtime folks: This issue was previously identified for 32-bit MacOS and a temporary fix is in place. For 32-bit Linux, the team's consensus is that there is no acceptable temporary fix, and the risk of rushing the pending CL that will fix this in M15 is not an acceptable risk for the M14 release. We also observe that the population of developers on 32-bit Linux machines is relatively small.

To avoid shipping with a "sometimes it works" system we should disable NaCl for 32-bit Linux builds of Chrome. The release notes should clearly identify this bug, and direct users to M15 Beta where the fix in question should land shortly.
Comment 16 by vivi...@chromium.org, Aug 16, 2011 
reopen this as this is Linux specific
Status: Assigned
Mergedinto:
Comment 17 by bradc...@google.com, Aug 16, 2011 
Blocked on: nativeclient:2157
Comment 18 by vivi...@chromium.org, Aug 16, 2011 
(No comment was entered for this change.)
Labels: -Mstone-14 Mstone-15
Comment 19 by bradc...@chromium.org, Aug 16, 2011 
Re-opening to track linux-specific status.
Owner: bradc...@chromium.org
Blockedon: nativeclient:2157
Comment 20 by bugdro...@chromium.org, Aug 16, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=97056

------------------------------------------------------------------------
r97056 | bradchen@google.com | Tue Aug 16 16:32:15 PDT 2011

Changed paths:
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_exports.txt?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.cc?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/tools/ld_bfd/ld?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.h?r1=97056&r2=97055&pathrev=97056
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_bootstrap_linux.c?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_linux.cc?r1=97056&r2=97055&pathrev=97056
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl.gypi?r1=97056&r2=97055&pathrev=97056

Gyp fix for release build.

Reserve 1GB at the base of the address space of linux nacl_helper for Native Client module. Use PathService instead of command line flag to enable helper. Adds a nacl_helper_bootstrap executable that implements the actual space reservation.

See http://codereview.chromium.org/7599011 for extended revision history. This is a gyp fix following a revert.

TBR=agl,bradnelson

BUG=92964, nativeclient:480
TEST=manual for now

Review URL: http://codereview.chromium.org/7670011
------------------------------------------------------------------------
Comment 21 by bugdro...@chromium.org, Aug 16, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=97058

------------------------------------------------------------------------
r97058 | rvargas@google.com | Tue Aug 16 16:45:06 PDT 2011

Changed paths:
 D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_exports.txt?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.cc?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/tools/ld_bfd/ld?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.h?r1=97058&r2=97057&pathrev=97058
 D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_bootstrap_linux.c?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_linux.cc?r1=97058&r2=97057&pathrev=97058
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl.gypi?r1=97058&r2=97057&pathrev=97058

Revert 97056 - Gyp fix for release build.

Reserve 1GB at the base of the address space of linux nacl_helper for Native Client module. Use PathService instead of command line flag to enable helper. Adds a nacl_helper_bootstrap executable that implements the actual space reservation.

See http://codereview.chromium.org/7599011 for extended revision history. This is a gyp fix following a revert.

TBR=agl,bradnelson

BUG=92964, nativeclient:480
TEST=manual for now

Review URL: http://codereview.chromium.org/7670011

TBR=bradchen@google.com
Review URL: http://codereview.chromium.org/7671011
------------------------------------------------------------------------
Comment 22 by bugdro...@chromium.org, Aug 17, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=97137

------------------------------------------------------------------------
r97137 | bradchen@google.com | Wed Aug 17 08:30:13 PDT 2011

Changed paths:
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_exports.txt?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.cc?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/tools/ld_bfd/ld?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_paths.h?r1=97137&r2=97136&pathrev=97137
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_bootstrap_linux.c?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_helper_linux.cc?r1=97137&r2=97136&pathrev=97137
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl.gypi?r1=97137&r2=97136&pathrev=97137

Reserve 1GB at the base of the address space of linux nacl_helper for Native Client module.
Use PathService instead of command line flag to enable helper. Adds a nacl_helper_bootstrap executable that implements the actual space reservation.

This is for resubmit of reverted CL 7670011, fixing nacl.gypi for the linux_shared build.

See 7670011 and 7599011 for review history.

TBR=agl,bradnelson

BUG=92964, nativeclient:480
TEST=manual for now

Review URL: http://codereview.chromium.org/7655010
------------------------------------------------------------------------
Comment 23 by bugdro...@chromium.org, Aug 17, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=97218

------------------------------------------------------------------------
r97218 | bradchen@google.com | Wed Aug 17 15:02:43 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_installer.gypi?r1=97218&r2=97217&pathrev=97218
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/tools/build/linux/FILES?r1=97218&r2=97217&pathrev=97218
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/tools/build/linux/FILES.cfg?r1=97218&r2=97217&pathrev=97218

Adding nacl_helper to Chrome Linux installer.

Renamed nacl_helper to nacl_helper.so to reflect its true nature.
Attempt to add nacl_helper to chrome installer.

BUG=92964,nativeclient:480,nativeclient:2131
TEST=manual for now

Review URL: http://codereview.chromium.org/7624017
------------------------------------------------------------------------
Comment 24 by mbo...@chromium.org, Aug 17, 2011 
This issue is observed on build 13.0.782.215 stable(OS: Linux & Mac).
Comment 25 by bugdro...@chromium.org, Aug 24, 2011 
Commit: 6aa1aacac1ac858f2d0ed8cd2cd3d60817c30cfc
 Email: bradchen@google.com@fcba33aa-ac0c-11dd-b9e7-8d5594d729c2

Enable the service runtime to use a zero-based sandbox on Linux. 32-bit ARM and x86 only for now. I will do x86-64 once these changes have settled. I have tested these changes by copying the modified .c files into a Chromium build. There are still some tests that are not building properly, but I didn't want to delay getting this part out for review.

BUG=chromium:92964,nativeclient:480
TEST=manual for now


Review URL: http://codereview.chromium.org/7677036

git-svn-id: svn://svn.chromium.org/native_client/trunk/src/native_client@6521 fcba33aa-ac0c-11dd-b9e7-8d5594d729c2

M	src/trusted/service_runtime/arch/arm/sel_addrspace_arm.c
M	src/trusted/service_runtime/arch/x86_32/sel_addrspace_x86_32.c
M	src/trusted/service_runtime/build.scons
M	src/trusted/service_runtime/linux/sel_memory.c
M	src/trusted/service_runtime/service_runtime.gyp
M	tests/multiple_sandboxes/nacl.scons
M	tests/unittests/trusted/service_runtime/build.scons
Comment 26 by ligim...@chromium.org, Aug 30, 2011 
 The issue exist on 14.0.835.122.
OS : Linux Ubuntu 10.04 /64 Bit
Comment 27 by ligim...@chromium.org, Aug 30, 2011 
Getting the following message on Java Console:

NativeClient: NaCl module load failed: Data Execution Prevention is required but is not supported
Comment 28 by bradc...@google.com, Aug 30, 2011 
This is kinda interesting; it indicates that on this test machine heap memory is executable. See src/trusted/platform_qualify/posix/nacl_dep_qualify.c:NaClAttemptToExecuteData().
Comment 29 by bradc...@google.com, Aug 30, 2011 
Checking /proc/cpuinfo on the test machine, this machine has the NX bit turned off, so it is normal that NaCl fails. The NaCl sandbox relies on NX on the 64-bit, as checked by our platform qualification test.

I think this closes the issue for 64-bit Linux. The fix for 32-bit Linux is in progress...
Comment 30 by ligim...@chromium.org, Aug 30, 2011 
More Info :

etc/lsb-release
==============
DISTRIB_CODENAME=lucid
DISTRIB_DESCRIPTION="Ubuntu 10.04.3 LTS"
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=10.04
GOOGLE_CODENAME=lucid
GOOGLE_ID=Goobuntu
GOOGLE_RELEASE="10.04 201108LD1"
GOOGLE_ROLE=desktop
GOOGLE_TRACK=stable

System Info
==========

processor	: 1
vendor_id	: GenuineIntel
cpu family	: 6
model		: 15
model name	: Intel(R) Core(TM)2 CPU          6600  @ 2.40GHz
stepping	: 6
cpu MHz		: 1600.000
cache size	: 4096 KB
physical id	: 0
siblings	: 2
core id		: 1
cpu cores	: 2
apicid		: 1
initial apicid	: 1
fpu		: yes
fpu_exception	: yes
cpuid level	: 10
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall lm constant_tsc arch_perfmon pebs bts rep_good aperfmperf pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm lahf_lm tpr_shadow
bogomips	: 4788.25
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 48 bits virtual
power management:
Comment 31 by ligim...@chromium.org, Sep 1, 2011 
NaCl is working fine. I got the NX Bit enabled on my Linux / 64 Bit machine and the Native Client examples are running successfully.
Status: Verified
Comment 33 by bugdro...@chromium.org, Sep 4, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=99617

------------------------------------------------------------------------
r99617 | bradchen@google.com | Sun Sep 04 18:17:48 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=99617&r2=99616&pathrev=99617
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_exe.gypi?r1=99617&r2=99616&pathrev=99617

Fix nacl_helper startup bug with command line construction.
Add proper chrome dependencies. Re-enable nacl_helper.





BUG=92964,nativeclient:480,95168
TEST=nacl_integration tests on bots


Review URL: http://codereview.chromium.org/7833017
------------------------------------------------------------------------
Comment 34 by bugdro...@chromium.org, Sep 4, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=99619

------------------------------------------------------------------------
r99619 | hbono@chromium.org | Sun Sep 04 19:28:27 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=99619&r2=99618&pathrev=99619
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_exe.gypi?r1=99619&r2=99618&pathrev=99619

Revert 99617 - Fix nacl_helper startup bug with command line construction.
Add proper chrome dependencies. Re-enable nacl_helper.





BUG=92964,nativeclient:480,95168
TEST=nacl_integration tests on bots


Review URL: http://codereview.chromium.org/7833017

TBR=bradchen@google.com
Review URL: http://codereview.chromium.org/7779029
------------------------------------------------------------------------
Comment 35 by bugdro...@chromium.org, Sep 4, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=99622

------------------------------------------------------------------------
r99622 | bradchen@google.com | Sun Sep 04 22:18:26 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=99622&r2=99621&pathrev=99622
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_exe.gypi?r1=99622&r2=99621&pathrev=99622

Fix nacl_helper argv bug, re-enable nacl_helper, build on linux except ARM. Previously reviewed as http://codereview.chromium.org/7833017; this time ARM build is disabled.

TBR=mcgrathr,mseaborn,evanm

BUG=92964,nativeclient:480,95196
TEST=nacl_integration on linux

Review URL: http://codereview.chromium.org/7800026
------------------------------------------------------------------------
Comment 36 by bugdro...@chromium.org, Sep 7, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=100031

------------------------------------------------------------------------
r100031 | bradchen@google.com | Wed Sep 07 15:03:33 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/tools/ld_bfd/ld?r1=100031&r2=100030&pathrev=100031
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=100031&r2=100030&pathrev=100031
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_exe.gypi?r1=100031&r2=100030&pathrev=100031
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/nacl.gypi?r1=100031&r2=100030&pathrev=100031

Modify the build recipe for nacl_helper_bootstrap to invoke the linker explicitly via an action rather than invoking via g++. This addresses build problems that occurred on certain developer machines.

Also, modified tools/ld_bfd/ld script to find the loader within the Chrome OS build chroot.

Also re-enable the nacl_helper.

BUG=92964,nativeclient:480
TEST=nacl_integration tests on bots

Review URL: http://codereview.chromium.org/7841008
------------------------------------------------------------------------
Comment 37 by cstefan...@google.com, Sep 9, 2011 
 Issue nativeclient:2180  has been merged into this issue.
Comment 38 by bradc...@chromium.org, Sep 12, 2011 
(No comment was entered for this change.)
Cc: mcgra...@chromium.org
Labels: Merge-Requested
Comment 39 by kar...@google.com, Sep 22, 2011 
(No comment was entered for this change.)
Labels: -Merge-Requested Merge-Approved
Comment 40 by bugdro...@chromium.org, Sep 22, 2011 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=102301

------------------------------------------------------------------------
r102301 | bradchen@google.com | Thu Sep 22 10:39:10 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/874/src/tools/ld_bfd/ld?r1=102301&r2=102300&pathrev=102301
 M http://src.chromium.org/viewvc/chrome/branches/874/src/chrome/nacl/nacl_fork_delegate_linux.cc?r1=102301&r2=102300&pathrev=102301
 M http://src.chromium.org/viewvc/chrome/branches/874/src/chrome/chrome_exe.gypi?r1=102301&r2=102300&pathrev=102301
 M http://src.chromium.org/viewvc/chrome/branches/874/src/chrome/nacl.gypi?r1=102301&r2=102300&pathrev=102301

Merge 100031 - Modify the build recipe for nacl_helper_bootstrap to invoke the linker explicitly via an action rather than invoking via g++. This addresses build problems that occurred on certain developer machines.

Also, modified tools/ld_bfd/ld script to find the loader within the Chrome OS build chroot.

Also re-enable the nacl_helper.

BUG=92964,nativeclient:480
TEST=nacl_integration tests on bots

Review URL: http://codereview.chromium.org/7841008

TBR=bradchen@google.com
Review URL: http://codereview.chromium.org/7992013
------------------------------------------------------------------------
Labels: -merge-approved merge-merged-874
► Sign in to add a comment

Powered by Google Project Hosting