Skip to content

[wasm][debugger] Reusing buffer to avoid allocate every debugger message #59480

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

Merged
thaystg merged 9 commits into from
Sep 29, 2021
Merged

[wasm][debugger] Reusing buffer to avoid allocate every debugger message #59480

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
0206acc
Reusing buffer to avoid allocate every debugger message.
thaystg Sep 22, 2021
3bbdba6
Fixing what was suggested by @radical.
thaystg Sep 22, 2021
b262298
cleaning code.
thaystg Sep 22, 2021
8fd17bb
Changing what was suggested by @radical
thaystg Sep 22, 2021
15463c4
Merge branch 'main' into thays_reuse_buffer_debugger
thaystg Sep 28, 2021
c356045
Implementing again after merge
thaystg Sep 28, 2021
425cebb
Fix spacing before parentheses.
thaystg Sep 28, 2021
5f652fc
Addressing @pavelsavara comments.
thaystg Sep 28, 2021
85d891f
Fix _debugger_buffer type.
thaystg Sep 28, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Changing what was suggested by @radical
  • Loading branch information
@thaystg
thaystg committed Sep 22, 2021
commit 8fd17bb47f84246a674f8cc2721ac1ac7ca3f6fb
23 changes: 10 additions & 13 deletions src/mono/wasm/runtime/library_mono.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -601,24 +601,21 @@ var MonoSupportLib = {

mono_wasm_malloc_and_set_debug_buffer: function (command_parameters)
{
if (!this.debugger_buffer || command_parameters.length > this.debugger_buffer_len)
if (command_parameters.length > this._debugger_buffer_len)
{
if (this.debugger_buffer)
Module._free (this.debugger_buffer);
else
this.debugger_buffer_len = 256;
if (this.debugger_buffer_len < command_parameters.length)
this.debugger_buffer_len = command_parameters.length;
this.debugger_buffer = Module._malloc (this.debugger_buffer_len);
this.heap_bytes = new Uint8Array (Module.HEAPU8.buffer, this.debugger_buffer, this.debugger_buffer_len);
if (this._debugger_buffer)
Module._free (this._debugger_buffer);
this._debugger_buffer_len = Math.max(command_parameters.length, this._debugger_buffer_len, 256);
this._debugger_buffer = Module._malloc (this._debugger_buffer_len);
this._debugger_heap_bytes = new Uint8Array (Module.HEAPU8.buffer, this._debugger_buffer, this._debugger_buffer_len);
}
this.heap_bytes.set(this._base64_to_uint8 (command_parameters));
this._debugger_heap_bytes.set(this._base64_to_uint8 (command_parameters));
},

mono_wasm_send_dbg_command_with_parms: function (id, command_set, command, command_parameters, length, valtype, newvalue)
{
this.mono_wasm_malloc_and_set_debug_buffer(command_parameters);
this._c_fn_table.mono_wasm_send_dbg_command_with_parms_wrapper (id, command_set, command, this.debugger_buffer, length, valtype, newvalue.toString());
this._c_fn_table.mono_wasm_send_dbg_command_with_parms_wrapper (id, command_set, command, this._debugger_buffer, length, valtype, newvalue.toString());
let { res_ok, res } = MONO.commands_received;
if (!res_ok)
throw new Error (`Failed on mono_wasm_invoke_method_debugger_agent_with_parms`);
Expand All @@ -628,7 +625,7 @@ var MonoSupportLib = {
mono_wasm_send_dbg_command: function (id, command_set, command, command_parameters)
{
this.mono_wasm_malloc_and_set_debug_buffer(command_parameters);
this._c_fn_table.mono_wasm_send_dbg_command_wrapper (id, command_set, command, this.debugger_buffer, command_parameters.length);
this._c_fn_table.mono_wasm_send_dbg_command_wrapper (id, command_set, command, this._debugger_buffer, command_parameters.length);
let { res_ok, res } = MONO.commands_received;
if (!res_ok)
throw new Error (`Failed on mono_wasm_send_dbg_command`);
Expand Down Expand Up @@ -863,7 +860,7 @@ var MonoSupportLib = {
this._c_fn_table = {};
this._register_c_fn ('mono_wasm_send_dbg_command', 'bool', [ 'number', 'number', 'number', 'number', 'number' ]);
this._register_c_fn ('mono_wasm_send_dbg_command_with_parms', 'bool', [ 'number', 'number', 'number', 'number', 'number', 'number', 'string' ]);

this._debugger_buffer_len = -1;
// DO NOT REMOVE - magic debugger init function
if (globalThis.dotnetDebugger)
debugger;
Expand Down