Parametrize the size of the non-sampler descriptor heap in dx12 (#4077)

This commit is contained in:
Nicolas Silva 2023-08-25 15:51:29 -04:00 committed by GitHub
parent 399637e2b8
commit 1ced67f056
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
8 changed files with 26 additions and 3 deletions

View File

@ -237,6 +237,7 @@ impl super::Adapter {
max_compute_workgroups_per_dimension, max_compute_workgroups_per_dimension,
// D3D11_BUFFER_DESC represents the buffer size as a 32 bit int. // D3D11_BUFFER_DESC represents the buffer size as a 32 bit int.
max_buffer_size: u32::MAX as u64, max_buffer_size: u32::MAX as u64,
max_non_sampler_bindings: u32::MAX,
}; };
// //

View File

@ -361,6 +361,7 @@ impl super::Adapter {
// This limit is chosen to avoid potential issues with drivers should they internally // This limit is chosen to avoid potential issues with drivers should they internally
// store buffer sizes using 32 bit ints (a situation we have already encountered with vulkan). // store buffer sizes using 32 bit ints (a situation we have already encountered with vulkan).
max_buffer_size: i32::MAX as u64, max_buffer_size: i32::MAX as u64,
max_non_sampler_bindings: 1_000_000,
}, },
alignments: crate::Alignments { alignments: crate::Alignments {
buffer_copy_offset: wgt::BufferSize::new( buffer_copy_offset: wgt::BufferSize::new(
@ -382,7 +383,7 @@ impl crate::Adapter<super::Api> for super::Adapter {
unsafe fn open( unsafe fn open(
&self, &self,
_features: wgt::Features, _features: wgt::Features,
_limits: &wgt::Limits, limits: &wgt::Limits,
) -> Result<crate::OpenDevice<super::Api>, crate::DeviceError> { ) -> Result<crate::OpenDevice<super::Api>, crate::DeviceError> {
let queue = { let queue = {
profiling::scope!("ID3D12Device::CreateCommandQueue"); profiling::scope!("ID3D12Device::CreateCommandQueue");
@ -399,6 +400,7 @@ impl crate::Adapter<super::Api> for super::Adapter {
let device = super::Device::new( let device = super::Device::new(
self.device.clone(), self.device.clone(),
queue.clone(), queue.clone(),
limits,
self.private_caps, self.private_caps,
&self.library, &self.library,
self.dx12_shader_compiler.clone(), self.dx12_shader_compiler.clone(),

View File

@ -16,6 +16,7 @@ impl super::Device {
pub(super) fn new( pub(super) fn new(
raw: d3d12::Device, raw: d3d12::Device,
present_queue: d3d12::CommandQueue, present_queue: d3d12::CommandQueue,
limits: &wgt::Limits,
private_caps: super::PrivateCapabilities, private_caps: super::PrivateCapabilities,
library: &Arc<d3d12::D3D12Lib>, library: &Arc<d3d12::D3D12Lib>,
dx12_shader_compiler: wgt::Dx12Compiler, dx12_shader_compiler: wgt::Dx12Compiler,
@ -92,7 +93,7 @@ impl super::Device {
}; };
// maximum number of CBV/SRV/UAV descriptors in heap for Tier 1 // maximum number of CBV/SRV/UAV descriptors in heap for Tier 1
let capacity_views = 1_000_000; let capacity_views = limits.max_non_sampler_bindings as u64;
let capacity_samplers = 2_048; let capacity_samplers = 2_048;
let shared = super::DeviceShared { let shared = super::DeviceShared {

View File

@ -576,6 +576,7 @@ impl super::Adapter {
}, },
max_compute_workgroups_per_dimension, max_compute_workgroups_per_dimension,
max_buffer_size: i32::MAX as u64, max_buffer_size: i32::MAX as u64,
max_non_sampler_bindings: std::u32::MAX,
}; };
let mut workarounds = super::Workarounds::empty(); let mut workarounds = super::Workarounds::empty();

View File

@ -904,6 +904,7 @@ impl super::PrivateCapabilities {
max_compute_workgroup_size_z: self.max_threads_per_group, max_compute_workgroup_size_z: self.max_threads_per_group,
max_compute_workgroups_per_dimension: 0xFFFF, max_compute_workgroups_per_dimension: 0xFFFF,
max_buffer_size: self.max_buffer_size, max_buffer_size: self.max_buffer_size,
max_non_sampler_bindings: std::u32::MAX,
}, },
alignments: crate::Alignments { alignments: crate::Alignments {
buffer_copy_offset: wgt::BufferSize::new(self.buffer_alignment).unwrap(), buffer_copy_offset: wgt::BufferSize::new(self.buffer_alignment).unwrap(),

View File

@ -755,6 +755,7 @@ impl PhysicalDeviceCapabilities {
max_compute_workgroup_size_z: max_compute_workgroup_sizes[2], max_compute_workgroup_size_z: max_compute_workgroup_sizes[2],
max_compute_workgroups_per_dimension, max_compute_workgroups_per_dimension,
max_buffer_size, max_buffer_size,
max_non_sampler_bindings: std::u32::MAX,
} }
} }

View File

@ -124,6 +124,7 @@ fn print_adapter(output: &mut impl io::Write, report: &AdapterReport, idx: usize
max_compute_workgroup_size_y, max_compute_workgroup_size_y,
max_compute_workgroup_size_z, max_compute_workgroup_size_z,
max_compute_workgroups_per_dimension, max_compute_workgroups_per_dimension,
max_non_sampler_bindings,
} = limits; } = limits;
writeln!(output, "\t\t Max Texture Dimension 1d: {max_texture_dimension_1d}")?; writeln!(output, "\t\t Max Texture Dimension 1d: {max_texture_dimension_1d}")?;
writeln!(output, "\t\t Max Texture Dimension 2d: {max_texture_dimension_2d}")?; writeln!(output, "\t\t Max Texture Dimension 2d: {max_texture_dimension_2d}")?;
@ -155,6 +156,10 @@ fn print_adapter(output: &mut impl io::Write, report: &AdapterReport, idx: usize
writeln!(output, "\t\t Max Compute Workgroup Size Z: {max_compute_workgroup_size_z}")?; writeln!(output, "\t\t Max Compute Workgroup Size Z: {max_compute_workgroup_size_z}")?;
writeln!(output, "\t\t Max Compute Workgroups Per Dimension: {max_compute_workgroups_per_dimension}")?; writeln!(output, "\t\t Max Compute Workgroups Per Dimension: {max_compute_workgroups_per_dimension}")?;
// This one reflects more of a wgpu implementation limitations than a hardware limit
// so don't show it here.
let _ = max_non_sampler_bindings;
////////////////////////// //////////////////////////
// Downlevel Properties // // Downlevel Properties //
////////////////////////// //////////////////////////

View File

@ -931,6 +931,12 @@ pub struct Limits {
/// - DX11 & OpenGL don't natively support push constants, and are emulated with uniforms, /// - DX11 & OpenGL don't natively support push constants, and are emulated with uniforms,
/// so this number is less useful but likely 256. /// so this number is less useful but likely 256.
pub max_push_constant_size: u32, pub max_push_constant_size: u32,
/// Maximum number of live non-sampler bindings.
///
/// This limit only affects the d3d12 backend. Using a large number will allow the device
/// to create many bind groups at the cost of a large up-front allocation at device creation.
pub max_non_sampler_bindings: u32,
} }
impl Default for Limits { impl Default for Limits {
@ -965,6 +971,7 @@ impl Default for Limits {
max_compute_workgroup_size_z: 64, max_compute_workgroup_size_z: 64,
max_compute_workgroups_per_dimension: 65535, max_compute_workgroups_per_dimension: 65535,
max_push_constant_size: 0, max_push_constant_size: 0,
max_non_sampler_bindings: 1_000_000,
} }
} }
} }
@ -1005,6 +1012,7 @@ impl Limits {
/// max_compute_workgroup_size_z: 64, /// max_compute_workgroup_size_z: 64,
/// max_compute_workgroups_per_dimension: 65535, /// max_compute_workgroups_per_dimension: 65535,
/// max_buffer_size: 256 << 20, // (256 MiB) /// max_buffer_size: 256 << 20, // (256 MiB)
/// max_non_sampler_bindings: 1_000_000,
/// }); /// });
/// ``` /// ```
pub fn downlevel_defaults() -> Self { pub fn downlevel_defaults() -> Self {
@ -1038,6 +1046,7 @@ impl Limits {
max_compute_workgroup_size_z: 64, max_compute_workgroup_size_z: 64,
max_compute_workgroups_per_dimension: 65535, max_compute_workgroups_per_dimension: 65535,
max_buffer_size: 256 << 20, max_buffer_size: 256 << 20,
max_non_sampler_bindings: 1_000_000,
} }
} }
@ -1076,7 +1085,8 @@ impl Limits {
/// max_compute_workgroup_size_y: 0, // + /// max_compute_workgroup_size_y: 0, // +
/// max_compute_workgroup_size_z: 0, // + /// max_compute_workgroup_size_z: 0, // +
/// max_compute_workgroups_per_dimension: 0, // + /// max_compute_workgroups_per_dimension: 0, // +
/// max_buffer_size: 256 << 20, // (256 MiB) /// max_buffer_size: 256 << 20, // (256 MiB),
/// max_non_sampler_bindings: 1_000_000,
/// }); /// });
/// ``` /// ```
pub fn downlevel_webgl2_defaults() -> Self { pub fn downlevel_webgl2_defaults() -> Self {
@ -1193,6 +1203,7 @@ impl Limits {
compare!(max_compute_workgroup_size_z, Less); compare!(max_compute_workgroup_size_z, Less);
compare!(max_compute_workgroups_per_dimension, Less); compare!(max_compute_workgroups_per_dimension, Less);
compare!(max_buffer_size, Less); compare!(max_buffer_size, Less);
compare!(max_non_sampler_bindings, Less);
} }
} }