[Impeller] Reland: add interface for submitting multiple command buffers at once.#50180
Conversation
| std::shared_ptr<CommandBuffer> command_buffer) const { | ||
| pending_command_buffers_->command_buffers.push_back( | ||
| std::move(command_buffer)); | ||
| // Metal systems seem to have a limit on the number of command buffers that |
There was a problem hiding this comment.
This is the big change.
There was a problem hiding this comment.
This still seems arbitrary. Can we flush pending command buffers when we can't create more or is there a delay between the flush and when we can acquire more buffers.
There was a problem hiding this comment.
Or have the magic number of 10 as well as the flush and retry on allocation failure. Just having a backend specific magic number seems icky.
There was a problem hiding this comment.
Thinking about it more I should just make it immediately flush on metal, since we get no benefit from batching.
There was a problem hiding this comment.
unfortunately there is no allocation failure, instead [queue commandBufferWithDescriptor:desc]; appears to hang.
There was a problem hiding this comment.
removed the limit, now we Record -> Flush.
There was a problem hiding this comment.
Oh well. Not having the arbitrary limit looks cleaner though.
There was a problem hiding this comment.
Yeah but it doesn't really matter for metal, I don't think we'll have a way to batch submit with that API ever.
jonahwilliams
added
the
autosubmit
jonahwilliams
removed
the
autosubmit
|
Did you mean to remove the autosub label? |
jonahwilliams
added
the
autosubmit
…and buffers at once. (flutter/engine#50180)
flutter/engine@d20ed24...e6e1d6b 2024-01-30 [email protected] Roll Dart SDK from 734aae9604e8 to 488e33cd39de (1 revision) (flutter/engine#50185) 2024-01-30 [email protected] [Impeller] Reland: add interface for submitting multiple command buffers at once. (flutter/engine#50180) 2024-01-30 [email protected] Support running sound null safe kernels from flutter_jit_runner (flutter/engine#50002) 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],[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
2 participants
Reland of: #50139
Metal does not seem to like it when we collect 50+ command buffers at once. Adjust the aiks context logic to regularly flush the cmd buffers.