1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-23 10:17:41 +00:00

Kernel: Make various T::class_name() and similar return StringView

Instead of returning char const*, we can also give you a StringView.
This commit is contained in:
Andreas Kling 2021-07-11 01:46:09 +02:00
parent fa9111ac46
commit c9f6786e8b
58 changed files with 72 additions and 72 deletions

View file

@ -27,7 +27,7 @@ public:
virtual void disable_aml_interpretation() override;
virtual void try_acpi_shutdown() override;
virtual bool can_shutdown() override { return true; }
virtual const char* purpose() const override { return "ACPI Parser"; }
virtual StringView purpose() const override { return "ACPI Parser"; }
protected:
explicit DynamicParser(PhysicalAddress rsdp);

View file

@ -29,7 +29,7 @@ public:
virtual ~UHCIController() override;
virtual const char* purpose() const override { return "UHCI"; }
virtual StringView purpose() const override { return "UHCI"; }
void reset();
void stop();

View file

@ -27,7 +27,7 @@ public:
virtual bool can_write(const Kernel::FileDescription&, size_t) const override { return true; }
virtual Kernel::KResultOr<size_t> read(FileDescription&, u64, Kernel::UserOrKernelBuffer&, size_t) override;
virtual Kernel::KResultOr<size_t> write(FileDescription&, u64, const Kernel::UserOrKernelBuffer&, size_t) override;
virtual const char* class_name() const override { return "Console"; }
virtual StringView class_name() const override { return "Console"; }
void put_char(char);

View file

@ -28,7 +28,7 @@ private:
virtual KResultOr<size_t> write(FileDescription&, u64, const UserOrKernelBuffer&, size_t) override;
virtual bool can_read(const FileDescription&, size_t) const override;
virtual bool can_write(const FileDescription&, size_t) const override { return true; }
virtual const char* class_name() const override { return "FullDevice"; }
virtual StringView class_name() const override { return "FullDevice"; }
};
}

View file

@ -54,7 +54,7 @@ protected:
mutable SpinLock<u8> m_queue_lock;
CircularQueue<Event, 16> m_queue;
// ^CharacterDevice
virtual const char* class_name() const override { return "KeyboardDevice"; }
virtual StringView class_name() const override { return "KeyboardDevice"; }
u8 m_modifiers { 0 };
bool m_caps_lock_to_ctrl_pressed { false };

View file

@ -39,7 +39,7 @@ public:
protected:
MouseDevice();
// ^CharacterDevice
virtual const char* class_name() const override { return "MouseDevice"; }
virtual StringView class_name() const override { return "MouseDevice"; }
mutable SpinLock<u8> m_queue_lock;
CircularQueue<MousePacket, 100> m_queue;

View file

@ -26,7 +26,7 @@ public:
virtual ~PS2KeyboardDevice() override;
bool initialize();
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
// ^I8042Device
virtual void irq_handle_byte_read(u8 byte) override;
@ -42,7 +42,7 @@ private:
virtual bool handle_irq(const RegisterState&) override;
// ^CharacterDevice
virtual const char* class_name() const override { return "KeyboardDevice"; }
virtual StringView class_name() const override { return "KeyboardDevice"; }
};
}

View file

@ -23,7 +23,7 @@ public:
virtual ~PS2MouseDevice() override;
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
// ^I8042Device
virtual void irq_handle_byte_read(u8 byte) override;

View file

@ -27,7 +27,7 @@ public:
private:
MemoryDevice();
virtual const char* class_name() const override { return "MemoryDevice"; }
virtual StringView class_name() const override { return "MemoryDevice"; }
virtual bool can_read(const FileDescription&, size_t) const override { return true; }
virtual bool can_write(const FileDescription&, size_t) const override { return false; }
virtual bool is_seekable() const override { return true; }

View file

@ -29,7 +29,7 @@ private:
virtual KResultOr<size_t> write(FileDescription&, u64, const UserOrKernelBuffer&, size_t) override;
virtual bool can_write(const FileDescription&, size_t) const override { return true; }
virtual bool can_read(const FileDescription&, size_t) const override;
virtual const char* class_name() const override { return "NullDevice"; }
virtual StringView class_name() const override { return "NullDevice"; }
};
}

View file

@ -28,7 +28,7 @@ private:
virtual KResultOr<size_t> write(FileDescription&, u64, const UserOrKernelBuffer&, size_t) override;
virtual bool can_read(const FileDescription&, size_t) const override;
virtual bool can_write(const FileDescription&, size_t) const override { return true; }
virtual const char* class_name() const override { return "RandomDevice"; }
virtual StringView class_name() const override { return "RandomDevice"; }
};
}

View file

@ -32,7 +32,7 @@ public:
virtual KResultOr<size_t> write(FileDescription&, u64, const UserOrKernelBuffer&, size_t) override;
virtual bool can_write(const FileDescription&, size_t) const override { return true; }
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
// ^Device
virtual mode_t required_mode() const override { return 0220; }
@ -43,7 +43,7 @@ private:
virtual bool handle_irq(const RegisterState&) override;
// ^CharacterDevice
virtual const char* class_name() const override { return "SB16"; }
virtual StringView class_name() const override { return "SB16"; }
void initialize();
void wait_for_irq();

View file

@ -112,7 +112,7 @@ private:
SerialDevice(IOAddress base_addr, unsigned minor);
// ^CharacterDevice
virtual const char* class_name() const override { return "SerialDevice"; }
virtual StringView class_name() const override { return "SerialDevice"; }
void initialize();
void set_interrupts(bool interrupt_enable);

View file

@ -27,7 +27,7 @@ private:
virtual KResultOr<size_t> write(FileDescription&, u64, const UserOrKernelBuffer&, size_t) override;
virtual bool can_read(const FileDescription&, size_t) const override;
virtual bool can_write(const FileDescription&, size_t) const override { return true; }
virtual const char* class_name() const override { return "ZeroDevice"; }
virtual StringView class_name() const override { return "ZeroDevice"; }
};
}

View file

@ -22,7 +22,7 @@ public:
virtual KResultOr<Region*> mmap(Process&, FileDescription&, const Range&, u64 offset, int prot, bool shared) override;
private:
virtual const char* class_name() const override { return "AnonymousFile"; }
virtual StringView class_name() const override { return "AnonymousFile"; }
virtual String absolute_path(const FileDescription&) const override { return ":anonymous-file:"; }
virtual bool can_read(const FileDescription&, size_t) const override { return false; }
virtual bool can_write(const FileDescription&, size_t) const override { return false; }

View file

@ -46,7 +46,7 @@ private:
virtual bool can_read(const FileDescription&, size_t) const override;
virtual bool can_write(const FileDescription&, size_t) const override;
virtual String absolute_path(const FileDescription&) const override;
virtual const char* class_name() const override { return "FIFO"; }
virtual StringView class_name() const override { return "FIFO"; }
virtual bool is_fifo() const override { return true; }
explicit FIFO(uid_t);

View file

@ -97,7 +97,7 @@ public:
virtual KResult chown(FileDescription&, uid_t, gid_t) { return EBADF; }
virtual KResult chmod(FileDescription&, mode_t) { return EBADF; }
virtual const char* class_name() const = 0;
virtual StringView class_name() const = 0;
virtual bool is_seekable() const { return false; }

View file

@ -42,7 +42,7 @@ public:
virtual KResult chown(FileDescription&, uid_t, gid_t) override;
virtual KResult chmod(FileDescription&, mode_t) override;
virtual const char* class_name() const override { return "InodeFile"; }
virtual StringView class_name() const override { return "InodeFile"; }
virtual bool is_seekable() const override { return true; }
virtual bool is_inode() const override { return true; }

View file

@ -55,7 +55,7 @@ public:
virtual KResult close() override;
virtual String absolute_path(const FileDescription&) const override;
virtual const char* class_name() const override { return "InodeWatcher"; };
virtual StringView class_name() const override { return "InodeWatcher"; };
virtual bool is_inode_watcher() const override { return true; }
void notify_inode_event(Badge<Inode>, InodeIdentifier, InodeWatcherEvent::Type, String const& name = {});

View file

@ -38,7 +38,7 @@ public:
private:
// ^File
virtual const char* class_name() const override { return "FramebufferDevice"; }
virtual StringView class_name() const override { return "FramebufferDevice"; }
virtual bool can_read(const FileDescription&, size_t) const override final { return true; }
virtual bool can_write(const FileDescription&, size_t) const override final { return true; }

View file

@ -51,7 +51,7 @@ public:
u8* framebuffer_data();
private:
virtual const char* class_name() const override { return "VirtIOFrameBuffer"; }
virtual StringView class_name() const override { return "VirtIOFrameBuffer"; }
VirtIOGPURespDisplayInfo::VirtIOGPUDisplayOne const& display_info() const;
VirtIOGPURespDisplayInfo::VirtIOGPUDisplayOne& display_info();

View file

@ -76,8 +76,8 @@ public:
virtual bool eoi() override;
virtual HandlerType type() const override { return HandlerType::IRQHandler; }
virtual const char* purpose() const override { return "IPI Handler"; }
virtual const char* controller() const override { return nullptr; }
virtual StringView purpose() const override { return "IPI Handler"; }
virtual StringView controller() const override { return nullptr; }
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }
@ -107,8 +107,8 @@ public:
virtual bool eoi() override;
virtual HandlerType type() const override { return HandlerType::IRQHandler; }
virtual const char* purpose() const override { return "SMP Error Handler"; }
virtual const char* controller() const override { return nullptr; }
virtual StringView purpose() const override { return "SMP Error Handler"; }
virtual StringView controller() const override { return nullptr; }
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }

View file

@ -45,8 +45,8 @@ public:
virtual bool is_sharing_with_others() const = 0;
virtual HandlerType type() const = 0;
virtual const char* purpose() const = 0;
virtual const char* controller() const = 0;
virtual StringView purpose() const = 0;
virtual StringView controller() const = 0;
virtual bool eoi() = 0;
ALWAYS_INLINE void increment_invoking_counter()

View file

@ -27,8 +27,8 @@ public:
virtual bool eoi() override;
virtual HandlerType type() const override { return HandlerType::IRQHandler; }
virtual const char* purpose() const override { return "IRQ Handler"; }
virtual const char* controller() const override { return m_responsible_irq_controller->model(); }
virtual StringView purpose() const override { return "IRQ Handler"; }
virtual StringView controller() const override { return m_responsible_irq_controller->model(); }
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }

View file

@ -33,8 +33,8 @@ public:
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::SharedIRQHandler; }
virtual const char* purpose() const override { return "Shared IRQ Handler"; }
virtual const char* controller() const override { return m_responsible_irq_controller->model(); }
virtual StringView purpose() const override { return "Shared IRQ Handler"; }
virtual StringView controller() const override { return m_responsible_irq_controller->model(); }
private:
void enable_interrupt_vector();

View file

@ -38,7 +38,7 @@ bool SpuriousInterruptHandler::eoi()
return false;
}
const char* SpuriousInterruptHandler::purpose() const
StringView SpuriousInterruptHandler::purpose() const
{
if (!m_real_handler)
return "Spurious Interrupt Handler";
@ -87,7 +87,7 @@ void SpuriousInterruptHandler::disable_interrupt_vector()
m_responsible_irq_controller->disable(*this);
}
const char* SpuriousInterruptHandler::controller() const
StringView SpuriousInterruptHandler::controller() const
{
if (m_responsible_irq_controller->type() == IRQControllerType::i82093AA)
return "";

View file

@ -29,8 +29,8 @@ public:
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::SpuriousInterruptHandler; }
virtual const char* purpose() const override;
virtual const char* controller() const override;
virtual StringView purpose() const override;
virtual StringView controller() const override;
private:
void enable_interrupt_vector();

View file

@ -21,8 +21,8 @@ public:
[[noreturn]] virtual bool eoi() override;
virtual HandlerType type() const override { return HandlerType::UnhandledInterruptHandler; }
virtual const char* purpose() const override { return "Unhandled Interrupt Handler"; }
virtual const char* controller() const override { VERIFY_NOT_REACHED(); }
virtual StringView purpose() const override { return "Unhandled Interrupt Handler"; }
virtual StringView controller() const override { VERIFY_NOT_REACHED(); }
virtual size_t sharing_devices_count() const override { return 0; }
virtual bool is_shared_handler() const override { return false; }

View file

@ -27,7 +27,7 @@ public:
virtual ~E1000ENetworkAdapter() override;
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
private:
E1000ENetworkAdapter(PCI::Address, u8 irq);

View file

@ -29,7 +29,7 @@ public:
virtual void send_raw(ReadonlyBytes) override;
virtual bool link_up() override;
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
protected:
void setup_interrupts();

View file

@ -75,7 +75,7 @@ public:
protected:
IPv4Socket(int type, int protocol);
virtual const char* class_name() const override { return "IPv4Socket"; }
virtual StringView class_name() const override { return "IPv4Socket"; }
PortAllocationResult allocate_local_port_if_needed();

View file

@ -52,7 +52,7 @@ public:
private:
explicit LocalSocket(int type);
virtual const char* class_name() const override { return "LocalSocket"; }
virtual StringView class_name() const override { return "LocalSocket"; }
virtual bool is_local() const override { return true; }
bool has_attached_peer(const FileDescription&) const;
DoubleBuffer* receive_buffer_for(FileDescription&);

View file

@ -25,7 +25,7 @@ public:
virtual void send_raw(ReadonlyBytes) override;
virtual bool link_up() override { return m_link_up; }
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
private:
NE2000NetworkAdapter(PCI::Address, u8 irq);

View file

@ -27,7 +27,7 @@ public:
virtual void send_raw(ReadonlyBytes) override;
virtual bool link_up() override { return m_link_up; }
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
private:
RTL8139NetworkAdapter(PCI::Address, u8 irq);

View file

@ -27,7 +27,7 @@ public:
virtual void send_raw(ReadonlyBytes) override;
virtual bool link_up() override { return m_link_up; }
virtual const char* purpose() const override { return class_name(); }
virtual StringView purpose() const override { return class_name(); }
private:
// FIXME: should this be increased? (maximum allowed here is 1024) - memory usage vs packet loss chance tradeoff

View file

@ -124,7 +124,7 @@ protected:
size_t backlog() const { return m_backlog; }
void set_backlog(size_t backlog) { m_backlog = backlog; }
virtual const char* class_name() const override { return "Socket"; }
virtual StringView class_name() const override { return "Socket"; }
virtual void shut_down_for_reading() { }
virtual void shut_down_for_writing() { }

View file

@ -164,7 +164,7 @@ protected:
private:
explicit TCPSocket(int protocol);
virtual const char* class_name() const override { return "TCPSocket"; }
virtual StringView class_name() const override { return "TCPSocket"; }
static NetworkOrdered<u16> compute_tcp_checksum(const IPv4Address& source, const IPv4Address& destination, const TCPPacket&, u16 payload_size);

View file

@ -21,7 +21,7 @@ public:
private:
explicit UDPSocket(int protocol);
virtual const char* class_name() const override { return "UDPSocket"; }
virtual StringView class_name() const override { return "UDPSocket"; }
static Lockable<HashMap<u16, UDPSocket*>>& sockets_by_port();
virtual KResultOr<size_t> protocol_receive(ReadonlyBytes raw_ipv4_packet, UserOrKernelBuffer& buffer, size_t buffer_size, int flags) override;

View file

@ -37,7 +37,7 @@ public:
virtual ~AHCIPortHandler() override;
RefPtr<StorageDevice> device_at_port(size_t port_index) const;
virtual const char* purpose() const override { return "SATA Port Handler"; }
virtual StringView purpose() const override { return "SATA Port Handler"; }
AHCI::HBADefinedCapabilities hba_capabilities() const;
NonnullRefPtr<AHCIController> hba_controller() const { return m_parent_controller; }

View file

@ -99,7 +99,7 @@ public:
RefPtr<StorageDevice> master_device() const;
RefPtr<StorageDevice> slave_device() const;
virtual const char* purpose() const override { return "PATA Channel"; }
virtual StringView purpose() const override { return "PATA Channel"; }
virtual bool is_dma_enabled() const { return false; }

View file

@ -32,7 +32,7 @@ UNMAP_AFTER_INIT PATADiskDevice::~PATADiskDevice()
{
}
const char* PATADiskDevice::class_name() const
StringView PATADiskDevice::class_name() const
{
return "PATADiskDevice";
}

View file

@ -48,7 +48,7 @@ private:
PATADiskDevice(const IDEController&, IDEChannel&, DriveType, InterfaceType, u16, u64);
// ^DiskDevice
virtual const char* class_name() const override;
virtual StringView class_name() const override;
bool is_slave() const;

View file

@ -72,7 +72,7 @@ String DiskPartition::device_name() const
return String::formatted("{}{}", m_device->device_name(), partition_index + 1);
}
const char* DiskPartition::class_name() const
StringView DiskPartition::class_name() const
{
return "DiskPartition";
}

View file

@ -32,7 +32,7 @@ public:
const DiskPartitionMetadata& metadata() const;
private:
virtual const char* class_name() const override;
virtual StringView class_name() const override;
DiskPartition(BlockDevice&, unsigned, DiskPartitionMetadata);

View file

@ -28,7 +28,7 @@ RamdiskDevice::~RamdiskDevice()
{
}
const char* RamdiskDevice::class_name() const
StringView RamdiskDevice::class_name() const
{
return "RamdiskDevice";
}

View file

@ -25,7 +25,7 @@ public:
virtual void start_request(AsyncBlockDeviceRequest&) override;
// ^DiskDevice
virtual const char* class_name() const override;
virtual StringView class_name() const override;
virtual String device_name() const override;
bool is_slave() const;

View file

@ -28,7 +28,7 @@ SATADiskDevice::~SATADiskDevice()
{
}
const char* SATADiskDevice::class_name() const
StringView SATADiskDevice::class_name() const
{
return "SATADiskDevice";
}

View file

@ -36,7 +36,7 @@ private:
SATADiskDevice(const AHCIController&, const AHCIPort&, size_t sector_size, u64 max_addressable_block);
// ^DiskDevice
virtual const char* class_name() const override;
virtual StringView class_name() const override;
NonnullRefPtr<AHCIPort> m_port;
};

View file

@ -27,7 +27,7 @@ StorageDevice::StorageDevice(const StorageController& controller, int major, int
{
}
const char* StorageDevice::class_name() const
StringView StorageDevice::class_name() const
{
return "StorageDevice";
}

View file

@ -36,7 +36,7 @@ protected:
StorageDevice(const StorageController&, size_t, u64);
StorageDevice(const StorageController&, int, int, size_t, u64);
// ^DiskDevice
virtual const char* class_name() const override;
virtual StringView class_name() const override;
private:
NonnullRefPtr<StorageController> m_storage_controller;

View file

@ -41,7 +41,7 @@ private:
virtual KResult close() override;
virtual bool is_master_pty() const override { return true; }
virtual int ioctl(FileDescription&, unsigned request, FlatPtr arg) override;
virtual const char* class_name() const override { return "MasterPTY"; }
virtual StringView class_name() const override { return "MasterPTY"; }
RefPtr<SlavePTY> m_slave;
unsigned m_index;

View file

@ -41,7 +41,7 @@ public:
private:
// ^CharacterDevice
virtual const char* class_name() const override { return "PTYMultiplexer"; }
virtual StringView class_name() const override { return "PTYMultiplexer"; }
Lock m_lock { "PTYMultiplexer" };
Vector<unsigned> m_freelist;

View file

@ -34,7 +34,7 @@ private:
virtual bool can_read(const FileDescription&, size_t) const override;
virtual KResultOr<size_t> read(FileDescription&, u64, UserOrKernelBuffer&, size_t) override;
virtual bool can_write(const FileDescription&, size_t) const override;
virtual const char* class_name() const override { return "SlavePTY"; }
virtual StringView class_name() const override { return "SlavePTY"; }
virtual KResult close() override;
// ^Device

View file

@ -105,7 +105,7 @@ private:
virtual void set_cursor_style(VT::CursorStyle) override;
// ^CharacterDevice
virtual const char* class_name() const override { return "VirtualConsole"; }
virtual StringView class_name() const override { return "VirtualConsole"; }
// ^Device
virtual String device_name() const override;

View file

@ -67,7 +67,7 @@ public:
IRQHandler::will_be_destroyed();
}
virtual const char* purpose() const override
virtual StringView purpose() const override
{
if (TimeManagement::the().is_system_timer(*this))
return "System Timer";
@ -118,7 +118,7 @@ public:
GenericInterruptHandler::will_be_destroyed();
}
virtual const char* purpose() const override
virtual StringView purpose() const override
{
return model();
}
@ -134,7 +134,7 @@ public:
virtual bool is_shared_handler() const override { return false; }
virtual bool is_sharing_with_others() const override { return false; }
virtual HandlerType type() const override { return HandlerType::IRQHandler; }
virtual const char* controller() const override { return nullptr; }
virtual StringView controller() const override { return nullptr; }
virtual bool eoi() override;
virtual u32 frequency() const override { return (u32)m_frequency; }

View file

@ -21,7 +21,7 @@ public:
VirtIOConsole(PCI::Address);
virtual ~VirtIOConsole() override = default;
virtual const char* purpose() const override { return "VirtIOConsole"; }
virtual StringView purpose() const override { return "VirtIOConsole"; }
unsigned device_id() const
{

View file

@ -32,7 +32,7 @@ public:
private:
constexpr static size_t RINGBUFFER_SIZE = 2 * PAGE_SIZE;
virtual const char* class_name() const override { return "VirtIOConsolePort"; }
virtual StringView class_name() const override { return "VirtIOConsolePort"; }
virtual bool can_read(const FileDescription&, size_t) const override;
virtual KResultOr<size_t> read(FileDescription&, u64, UserOrKernelBuffer&, size_t) override;

View file

@ -17,8 +17,8 @@ namespace Kernel {
class VirtIORNG final : public CharacterDevice
, public VirtIODevice {
public:
virtual const char* purpose() const override { return class_name(); }
virtual const char* class_name() const override { return m_class_name.characters(); }
virtual StringView purpose() const override { return class_name(); }
virtual StringView class_name() const override { return m_class_name; }
virtual bool can_read(const FileDescription&, size_t) const override { return false; }
virtual KResultOr<size_t> read(FileDescription&, u64, UserOrKernelBuffer&, size_t) override { return 0; }