system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-21 10:05:32 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Kernel: Avoid some copies during USBInterface enumeration/creation

Browse source
This commit is contained in:
Hendiadyoin1 2023-09-15 21:27:46 +02:00 committed by Andrew Kaster
parent b857c6b92f
commit c230a0d96f
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Kernel/Bus/USB/USBConfiguration.cpp
View file

@ -29,6 +29,7 @@ ErrorOr<void> USBConfiguration::enumerate_interfaces()
u8* interface_descriptors_base = descriptor_hierarchy_buffer.data() + sizeof(USBConfigurationDescriptor);
USBInterfaceDescriptor* interface_descriptor = reinterpret_cast<USBInterfaceDescriptor*>(interface_descriptors_base);
Vector<USBEndpointDescriptor> endpoint_descriptors;
TRY(m_interfaces.try_ensure_capacity(m_descriptor.number_of_interfaces));
for (auto interface = 0u; interface < m_descriptor.number_of_interfaces; interface++) {
endpoint_descriptors.ensure_capacity(interface_descriptor->number_of_endpoints);
@ -64,11 +65,11 @@ ErrorOr<void> USBConfiguration::enumerate_interfaces()
dbgln("Endpoint Poll Interval (in frames): {}", endpoint_descriptor.poll_interval_in_frames);
}
endpoint_descriptors.append(endpoint_descriptor);
endpoint_descriptors.unchecked_append(endpoint_descriptor);
}
USBInterface device_interface(*this, *interface_descriptor, endpoint_descriptors);
m_interfaces.append(device_interface);
USBInterface device_interface(*this, *interface_descriptor, move(endpoint_descriptors));
m_interfaces.unchecked_append(move(device_interface));
interface_descriptor += interface_descriptor->number_of_endpoints * sizeof(USBEndpointDescriptor);
}

4
Kernel/Bus/USB/USBInterface.h
View file

@ -16,10 +16,10 @@ class USBConfiguration;
class USBInterface final {
public:
USBInterface() = delete;
USBInterface(USBConfiguration const& configuration, USBInterfaceDescriptor const descriptor, Vector<USBEndpointDescriptor> const& endpoint_descriptors)
USBInterface(USBConfiguration const& configuration, USBInterfaceDescriptor const descriptor, Vector<USBEndpointDescriptor> endpoint_descriptors)
: m_configuration(configuration)
, m_descriptor(descriptor)
, m_endpoint_descriptors(endpoint_descriptors)
, m_endpoint_descriptors(move(endpoint_descriptors))
{
m_endpoint_descriptors.ensure_capacity(descriptor.number_of_endpoints);
}