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.

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

Deprecate SbMemoryMap APIs. #2064

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Deprecate SbMemoryMap APIs. #2064

merged 1 commit into from
Dec 19, 2023

Conversation

y4vor
Copy link
Contributor

@y4vor y4vor commented Dec 11, 2023

  • Deprecated SbMemoryMap, SbMemoryUnmap, SbMemoryProtect and SbMemoryFlush.

b/302332972

Change-Id: I198963e11db32c14ecc35096691da00d6d690ddd

@y4vor y4vor force-pushed the mmap branch 5 times, most recently from 24f91cb to c049943 Compare December 11, 2023 21:22
Copy link

codecov bot commented Dec 11, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (cec32f1) 58.55% compared to head (7cd042b) 58.63%.
Report is 1 commits behind head on main.

❗ Current head 7cd042b differs from pull request most recent head b102763. Consider uploading reports for the commit b102763 to get more accurate results

Files Patch % Lines
starboard/elf_loader/program_table.cc 83.33% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2064      +/-   ##
==========================================
+ Coverage   58.55%   58.63%   +0.08%     
==========================================
  Files        1914     1901      -13     
  Lines       94432    94012     -420     
==========================================
- Hits        55298    55128     -170     
+ Misses      39134    38884     -250     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@gbournou gbournou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a few test failures. One of them looks unrelated for SbStringFormat, but I can't tell if the other failures are the same issue or not.

@y4vor y4vor force-pushed the mmap branch 3 times, most recently from 2b5fe3d to 3ad229a Compare December 18, 2023 18:37
@y4vor
Copy link
Contributor Author

y4vor commented Dec 18, 2023

I see a few test failures. One of them looks unrelated for SbStringFormat, but I can't tell if the other failures are the same issue or not.

That should be gone as I think it was coming from Max's change.

@y4vor y4vor closed this Dec 18, 2023
@y4vor y4vor reopened this Dec 18, 2023
@y4vor y4vor requested a review from gbournou December 18, 2023 18:53
starboard/elf_loader/exported_symbols.cc Outdated Show resolved Hide resolved
 - Deprecated SbMemoryMap, SbMemoryUnmap, SbMemoryProtect
   and SbMemoryFlush.

b/302332972

Change-Id: I198963e11db32c14ecc35096691da00d6d690ddd
@y4vor y4vor merged commit 139da0d into youtube:main Dec 19, 2023
329 of 332 checks passed
kaidokert added a commit to kaidokert/cobalt that referenced this pull request Dec 28, 2023
This reverts commit 139da0d.

Broke builds on Raspi-2

b/302332972
kaidokert added a commit that referenced this pull request Dec 28, 2023
This reverts commit 139da0d. (#2064)

Broke builds on Raspi-2

b/302332972
y4vor added a commit to y4vor/cobalt that referenced this pull request Jan 12, 2024
This reverts commit a684fb0.

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
y4vor added a commit to y4vor/cobalt that referenced this pull request Jan 12, 2024
This reverts commit a684fb0.

b/302332972

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
y4vor added a commit to y4vor/cobalt that referenced this pull request Jan 13, 2024
This reverts commit a684fb0.

Added mmap wrapping to manage the off_t type differences on
platforms like raspi-2.

b/302332972

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
y4vor added a commit to y4vor/cobalt that referenced this pull request Jan 13, 2024
This reverts commit a684fb0.

Added mmap wrapping to manage the off_t type differences on
platforms like raspi-2.

b/302332972

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
y4vor added a commit to y4vor/cobalt that referenced this pull request Jan 16, 2024
This reverts commit a684fb0.

Added mmap wrapping to manage the off_t type differences on
platforms like raspi-2.

b/302332972

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
y4vor added a commit that referenced this pull request Jan 17, 2024
This reverts commit a684fb0.

b/302332972

Change-Id: If38bea327255c4e5bce013e9e7c389ff57c675d5
@y4vor y4vor deleted the mmap branch May 3, 2024 14:35
dahlstrom-g pushed a commit that referenced this pull request Oct 30, 2024
Uploaded by https://ci.chromium.org/b/8738505171997425025

Change-Id: I97cb38c7db8029bc94e733819844b638921f1940
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5817034
Bot-Commit: ChromeOS bot <[email protected]>
Commit-Queue: Yuta Hijikata <[email protected]>
Cr-Commit-Position: refs/branch-heads/6099@{#2064}
Cr-Branched-From: e6ee4500f7d6549a9ac1354f8d056da49ef406be-refs/heads/main@{#1217362}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants