Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[RISCV64 Branch][Hifive unleashed Borad] mjsunit/wasm/memory_1gb_oob failed #177

Closed
luyahan opened this issue Aug 20, 2020 · 4 comments · Fixed by #236
Closed

[RISCV64 Branch][Hifive unleashed Borad] mjsunit/wasm/memory_1gb_oob failed #177

luyahan opened this issue Aug 20, 2020 · 4 comments · Fixed by #236
Labels
bug Something isn't working
Milestone
native-build (09/...

Comments

@luyahan
Copy link
Collaborator

luyahan commented Aug 20, 2020 •
edited
Loading

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Build with options 'riscv64.native.debug'
  2. Run test/program riscv64.native/d8 --test test/mjsunit/mjsunit.js test/mjsunit/wasm/memory_1gb_oob.js --random-seed=-652826751 --nohard-abort --enable-slow-asserts --verify-heap --testing-d8-test-runner --wasm-max-mem-pages=16384
  3. See error

Expected behavior
A clear and concise description of what you expected to happen.

Errors/Logs

=== mjsunit/wasm/memory_1gb_oob ===
Warning: unknown flag --enable-slow-asserts.
Try --help for options
size=4 offset=0
size=4 offset=1
size=4 offset=2
size=4 offset=3
size=4 offset=4
size=4 offset=5
size=4 offset=7
size=4 offset=8
size=4 offset=9
size=4 offset=1073741816
size=4 offset=1073741820
size=4 offset=1073741824
size=4 offset=1073741825
size=4 offset=1073741826
size=4 offset=1073741827
size=4 offset=1073741828
size=4 offset=1073741829
size=4 offset=1073741831
size=4 offset=1073741832
size=4 offset=1073741833
size=4 offset=2147483640
size=4 offset=2147483644
size=4 offset=2147483648
size=4 offset=2147483649
size=4 offset=2147483650
size=4 offset=2147483651
size=4 offset=2147483652
size=4 offset=2147483653
size=4 offset=2147483655
size=4 offset=2147483656
size=4 offset=2147483657
size=4 offset=3221225464
size=4 offset=3221225468
size=4 offset=3221225472
size=4 offset=3221225473
size=4 offset=3221225474
size=4 offset=3221225475
size=4 offset=3221225476
size=4 offset=3221225477
size=4 offset=3221225479
size=4 offset=3221225480
size=4 offset=3221225481
size=4 offset=4294967287
size=4 offset=4294967288
size=4 offset=4294967289
size=4 offset=4294967291
size=4 offset=4294967292
size=4 offset=4294967293
size=4 offset=4294967294
size=4 offset=4294967295
size=1 offset=0
size=1 offset=1
size=1 offset=2
size=1 offset=3
size=1 offset=4
size=1 offset=5
size=1 offset=7
size=1 offset=8
size=1 offset=9
size=1 offset=1073741816
size=1 offset=1073741820
size=1 offset=1073741824
size=1 offset=1073741825
size=1 offset=1073741826
size=1 offset=1073741827
size=1 offset=1073741828
size=1 offset=1073741829
size=1 offset=1073741831
size=1 offset=1073741832
size=1 offset=1073741833
size=1 offset=2147483640
size=1 offset=2147483644
size=1 offset=2147483648
size=1 offset=2147483649
size=1 offset=2147483650
size=1 offset=2147483651
size=1 offset=2147483652
size=1 offset=2147483653
size=1 offset=2147483655
size=1 offset=2147483656
size=1 offset=2147483657
size=1 offset=3221225464
size=1 offset=3221225468
size=1 offset=3221225472
size=1 offset=3221225473
size=1 offset=3221225474
size=1 offset=3221225475
size=1 offset=3221225476
size=1 offset=3221225477
size=1 offset=3221225479
size=1 offset=3221225480
size=1 offset=3221225481
size=1 offset=4294967287
size=1 offset=4294967288
size=1 offset=4294967289
size=1 offset=4294967291
size=1 offset=4294967292
size=1 offset=4294967293
size=1 offset=4294967294
size=1 offset=4294967295
size=8 offset=0
size=8 offset=1
size=8 offset=2
size=8 offset=3
size=8 offset=4
size=8 offset=5
size=8 offset=7
size=8 offset=8
size=8 offset=9
size=8 offset=1073741816
test/mjsunit/mjsunit.js:316: Failure: expected <-1430532882> found <0>

Stack: MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 69:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 96:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 98:3
    throw new MjsUnitAssertionError(message);
    ^
MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 69:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 96:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 98:3
Received signal 6
Command: riscv64.native/d8 --test test/mjsunit/mjsunit.js test/mjsunit/wasm/memory_1gb_oob.js --random-seed=-652826751 --nohard-abort --enable-slow-asserts --verify-heap --testing-d8-test-runner --wasm-max-mem-pages=16384

Additional context
Add any other context about the problem here.
@luyahan luyahan added the bug Something isn't working label Aug 20, 2020
@luyahan luyahan added this to the native-build (09/04/2020) milestone Aug 20, 2020
@luyahan luyahan changed the title [RISCV64 Branch][HIFVIE Borad] mjsunit/wasm/memory_1gb_oob failed [RISCV64 Branch][Hifive unleashed Borad] mjsunit/wasm/memory_1gb_oob failed Aug 20, 2020
@luyahan
Copy link
Collaborator Author

luyahan commented Aug 24, 2020

hit a gdb issue.
riscvarchive/riscv-binutils-gdb#230

@luyahan
Copy link
Collaborator Author

luyahan commented Sep 15, 2020 •
edited
Loading

It is a probability of error on last code.

size=1 offset=4294967295
size=8 offset=0
size=8 offset=1
size=8 offset=2
size=8 offset=3
size=8 offset=4
size=8 offset=5
size=8 offset=7
size=8 offset=8
size=8 offset=9
size=8 offset=1073741816
test/mjsunit/mjsunit.js:316: Failure: expected <-1430532882> found <0>

Stack: MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 69:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 96:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 98:3
    throw new MjsUnitAssertionError(message);
    ^
MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 69:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 96:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 98:3

@luyahan
Copy link
Collaborator Author

luyahan commented Sep 15, 2020 •
edited
Loading 

  1 = -1430532882
  resut = -1430532882
  2 = -1430532882
  resut = -1430532882
  3 = -1430532882
  resut = -1430532882
  4 = -1430532882
  resut = -1430532882
  5 = -1430532882
  resut = -1430532882
  7 = -1430532882
  resut = -1430532882
  8 = -1430532882
  resut = -1430532882
  9 = -1430532882
  resut = -1430532882
size=8 offset=1073741816
  0 = -1430532882
  resut = -1430532882
size=8 offset=1073741820
size=8 offset=1073741824
size=8 offset=1073741825
size=8 offset=1073741826
size=8 offset=1073741827
size=8 offset=1073741828
size=8 offset=1073741829
size=8 offset=1073741831
size=8 offset=1073741832
size=8 offset=1073741833
size=8 offset=2147483640

  8 = -1430532882
  resut = -1430532882
  9 = -1430532882
  resut = -1430532882
size=8 offset=1073741816
  0 = -1430532882
  resut = 0
test/mjsunit/mjsunit.js:316: Failure: expected <-1430532882> found <0>

Stack: MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 70:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 97:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 99:3
    throw new MjsUnitAssertionError(message);
    ^
MjsUnitAssertionError
    at assertEquals test/mjsunit/mjsunit.js 406:7
    at probe        test/mjsunit/wasm/memory_1gb_oob.js 70:9
    at Test         test/mjsunit/wasm/memory_1gb_oob.js 97:5
    at              test/mjsunit/wasm/memory_1gb_oob.js 99:3

@luyahan
Copy link
Collaborator Author

luyahan commented Sep 16, 2020

Run on qemu is normal.

@luyahan luyahan mentioned this issue Sep 16, 2020
Merged
luyahan added a commit to luyahan/v8 that referenced this issue Sep 16, 2020
@luyahan
Disacard scratch reg in ULoadF/UStoref
788cd29 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
penguinwu pushed a commit that referenced this issue Sep 17, 2020
@luyahan @penguinwu
Disacard scratch reg in ULoadF/UStoref
3054498 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Oct 15, 2020
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
fa63469 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Nov 11, 2020
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
8522f23 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
luyahan added a commit to luyahan/v8 that referenced this issue Nov 30, 2020
@luyahan
Disacard scratch reg in ULoadF/UStoref
7e31490 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
ghost pushed a commit that referenced this issue Dec 3, 2020
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
34a2b39 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Dec 9, 2020
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
afc4925 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Jan 13, 2021
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
4811f2d 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Jan 26, 2021
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
53213d4 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
luyahan added a commit to luyahan/v8 that referenced this issue Jan 27, 2021
@luyahan
Disacard scratch reg in ULoadF/UStoref
9dfa8bd 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
luyahan added a commit to luyahan/v8 that referenced this issue Feb 1, 2021
@luyahan
Disacard scratch reg in ULoadF/UStoref
fc33829 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
ghost pushed a commit that referenced this issue Feb 1, 2021
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
1ba08a6 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
ghost pushed a commit that referenced this issue Feb 3, 2021
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
2ee9bca 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
luyahan added a commit to luyahan/v8 that referenced this issue Feb 7, 2021
@luyahan
Disacard scratch reg in ULoadF/UStoref
510d8ab 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
luyahan added a commit to luyahan/v8 that referenced this issue Feb 9, 2021
@luyahan
Disacard scratch reg in ULoadF/UStoref
4efcf85 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix riscv-collab#177
ghost pushed a commit that referenced this issue Feb 9, 2021
@luyahan @obycode
Disacard scratch reg in ULoadF/UStoref
e3b892a 
- Becasue improve management of register t5 and t6, so we not need
  to use scratch paramter.
- fix #177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
native-build (09/04/2020)
Development

Successfully merging a pull request may close this issue.

Disacard scratch reg in ULoadF/UStoref luyahan/v8
1 participant
@luyahan