Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Flutter GPU] Fix assert failure in createDeviceBufferWithCopy.#56039

Merged
auto-submit[bot] merged 1 commit intoflutter:mainfrom
bdero:bdero/bytedata-destruct
Oct 23, 2024
Merged

[Flutter GPU] Fix assert failure in createDeviceBufferWithCopy.#56039
auto-submit[bot] merged 1 commit intoflutter:mainfrom
bdero:bdero/bytedata-destruct

Conversation

@bdero
Copy link
Contributor

@bdero bdero commented Oct 22, 2024

Resolves flutter/flutter#157336

DartByteData gets raw pointers into the Dart heap via Dart_TypedDataAcquireData. So it must be destructed before AssociateWithDartWrapper is called, which mutates the heap.

Re-enables failing test for Linux unopt.

@bdero bdero self-assigned this Oct 22, 2024
@bdero bdero force-pushed the bdero/bytedata-destruct branch from 005c96b to 8c8fc5f Compare October 22, 2024 23:54
@bdero bdero marked this pull request as ready for review October 23, 2024 00:47
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@bdero bdero added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 23, 2024
@auto-submit auto-submit bot merged commit 62a8bae into flutter:main Oct 23, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 23, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 23, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 23, 2024
flutter/engine@50f6d98...ec6e28a

2024-10-23 [email protected] [display_list] grow display list backing store by power of two. (flutter/engine#56004)
2024-10-23 [email protected] Roll Fuchsia Linux SDK from xPbin_33tT-_omeQT... to NxrFCTty8wV4-6Cpl... (flutter/engine#56056)
2024-10-23 [email protected] Roll Dart SDK from 20470aaa17be to 2a4b728f6a03 (3 revisions) (flutter/engine#56055)
2024-10-23 [email protected] Move get_keyboard_state from FlKeyboardViewDelegate to FlKeyboardManager. (flutter/engine#56021)
2024-10-23 [email protected] Move send_key_event from FlKeyboardViewDelegate to FlKeyboardManager. (flutter/engine#56020)
2024-10-23 [email protected] [Flutter GPU] Fix assert failure in createDeviceBufferWithCopy. (flutter/engine#56039)
2024-10-22 [email protected] Roll Dart SDK from 8de85635d731 to 20470aaa17be (4 revisions) (flutter/engine#56038)
2024-10-22 [email protected] [web] Remove old futureToPromise. (flutter/engine#55847)
2024-10-22 [email protected] [dart:ui] Improve Clip's docs (flutter/engine#55978)
2024-10-22 [email protected] [Impeller] flush thread local resources during toImage/toImageSync. (flutter/engine#56037)
2024-10-22 [email protected] [canvaskit] Makes access to CkSurface null-safer (flutter/engine#54895)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from xPbin_33tT-_ to NxrFCTty8wV4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
M97Chahboun pushed a commit to M97Chahboun/flutter that referenced this pull request Oct 30, 2024
…57453)

flutter/engine@50f6d98...ec6e28a

2024-10-23 [email protected] [display_list] grow display list backing store by power of two. (flutter/engine#56004)
2024-10-23 [email protected] Roll Fuchsia Linux SDK from xPbin_33tT-_omeQT... to NxrFCTty8wV4-6Cpl... (flutter/engine#56056)
2024-10-23 [email protected] Roll Dart SDK from 20470aaa17be to 2a4b728f6a03 (3 revisions) (flutter/engine#56055)
2024-10-23 [email protected] Move get_keyboard_state from FlKeyboardViewDelegate to FlKeyboardManager. (flutter/engine#56021)
2024-10-23 [email protected] Move send_key_event from FlKeyboardViewDelegate to FlKeyboardManager. (flutter/engine#56020)
2024-10-23 [email protected] [Flutter GPU] Fix assert failure in createDeviceBufferWithCopy. (flutter/engine#56039)
2024-10-22 [email protected] Roll Dart SDK from 8de85635d731 to 20470aaa17be (4 revisions) (flutter/engine#56038)
2024-10-22 [email protected] [web] Remove old futureToPromise. (flutter/engine#55847)
2024-10-22 [email protected] [dart:ui] Improve Clip's docs (flutter/engine#55978)
2024-10-22 [email protected] [Impeller] flush thread local resources during toImage/toImageSync. (flutter/engine#56037)
2024-10-22 [email protected] [canvaskit] Makes access to CkSurface null-safer (flutter/engine#54895)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from xPbin_33tT-_ to NxrFCTty8wV4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…ter/engine#56039)

Resolves flutter#157336

`DartByteData` gets raw pointers into the Dart heap via `Dart_TypedDataAcquireData`. So it must be destructed before `AssociateWithDartWrapper` is called, which mutates the heap.

Re-enables failing test for Linux unopt.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App flutter-gpu

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

[Flutter GPU] Linux unopt: GpuContext.createDeviceBufferWithCopy triggers an assert in third_party/dart.

2 participants