Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 21:50:18 +01:00
Code Issues Projects Releases Packages Wiki Activity

Only send mapping back on unmap when MapMode = WRITE (#34054)

Browse source 
Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
This commit is contained in:
Samson 2024-10-29 19:42:07 +01:00 • committed by GitHub
parent d5554235fe
commit 43d1601016
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 20 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

15
components/script/dom/gpubuffer.rs
View file

@ -205,10 +205,15 @@ impl GPUBufferMethods for GPUBuffer {
// Step 5&7
if let Err(e) = self.channel.0.send(WebGPURequest::UnmapBuffer {
buffer_id: self.id().0,
array_buffer: IpcSharedMemory::from_bytes(mapping.data.data()),
write_back: mapping.mode >= GPUMapModeConstants::WRITE,
offset: mapping.range.start,
size: mapping.range.end - mapping.range.start,
mapping: if mapping.mode >= GPUMapModeConstants::WRITE {
Some(Mapping {
data: IpcSharedMemory::from_bytes(mapping.data.data()),
range: mapping.range.clone(),
mode: HostMap::Write,
})
} else {
None
},
}) {
warn!("Failed to send Buffer unmap ({:?}) ({})", self.buffer.0, e);
}
@ -249,8 +254,6 @@ impl GPUBufferMethods for GPUBuffer {
// Step 4
*self.pending_map.borrow_mut() = Some(promise.clone());
// Step 5
// This should be bitflags in wgpu-core
let host_map = match mode {
GPUMapModeConstants::READ => HostMap::Read,
GPUMapModeConstants::WRITE => HostMap::Write,

8
components/webgpu/ipc_messages/recv.rs
View file

@ -32,7 +32,7 @@ pub use {wgpu_core as wgc, wgpu_types as wgt};
use crate::identity::*;
use crate::render_commands::RenderCommand;
use crate::swapchain::WebGPUContextId;
use crate::{Error, ErrorFilter, WebGPUResponse, PRESENTATION_BUFFER_COUNT};
use crate::{Error, ErrorFilter, Mapping, WebGPUResponse, PRESENTATION_BUFFER_COUNT};
#[derive(Clone, Copy, Debug, Deserialize, Serialize)]
pub struct ContextConfiguration {
@ -276,10 +276,8 @@ pub enum WebGPURequest {
},
UnmapBuffer {
buffer_id: id::BufferId,
array_buffer: IpcSharedMemory,
write_back: bool,
offset: u64,
size: u64,
/// Return back mapping for writeback
mapping: Option<Mapping>,
},
WriteBuffer {
device_id: id::DeviceId,

20
components/webgpu/wgpu_thread.rs
View file

@ -995,25 +995,21 @@ impl WGPU {
};
self.maybe_dispatch_error(device_id, result.err());
},
WebGPURequest::UnmapBuffer {
buffer_id,
array_buffer,
write_back,
offset,
size,
} => {
WebGPURequest::UnmapBuffer { buffer_id, mapping } => {
let global = &self.global;
if write_back {
if let Ok((slice_pointer, range_size)) =
global.buffer_get_mapped_range(buffer_id, offset, Some(size))
{
if let Some(mapping) = mapping {
if let Ok((slice_pointer, range_size)) = global.buffer_get_mapped_range(
buffer_id,
mapping.range.start,
Some(mapping.range.end - mapping.range.start),
) {
unsafe {
slice::from_raw_parts_mut(
slice_pointer.as_ptr(),
range_size as usize,
)
}
.copy_from_slice(&array_buffer);
.copy_from_slice(&mapping.data);
}
}
// Ignore result because this operation always succeed from user perspective