1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 08:27:45 +00:00

Kernel: Convert klog() => AK::Format in RTL8139NetworkAdapter

This commit is contained in:
Andreas Kling 2021-03-12 11:21:34 +01:00
parent 6a3224d040
commit b8ad3d7ccf

View file

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org> * Copyright (c) 2018-2021, Andreas Kling <kling@serenityos.org>
* All rights reserved. * All rights reserved.
* *
* Redistribution and use in source and binary forms, with or without * Redistribution and use in source and binary forms, with or without
@ -147,29 +147,29 @@ UNMAP_AFTER_INIT RTL8139NetworkAdapter::RTL8139NetworkAdapter(PCI::Address addre
m_tx_buffers.ensure_capacity(RTL8139_TX_BUFFER_COUNT); m_tx_buffers.ensure_capacity(RTL8139_TX_BUFFER_COUNT);
set_interface_name("rtl8139"); set_interface_name("rtl8139");
klog() << "RTL8139: Found @ " << pci_address(); dmesgln("RTL8139: Found @ {}", pci_address());
enable_bus_mastering(pci_address()); enable_bus_mastering(pci_address());
m_interrupt_line = PCI::get_interrupt_line(pci_address()); m_interrupt_line = PCI::get_interrupt_line(pci_address());
klog() << "RTL8139: port base: " << m_io_base; dmesgln("RTL8139: I/O port base: {}", m_io_base);
klog() << "RTL8139: Interrupt line: " << m_interrupt_line; dmesgln("RTL8139: Interrupt line: {}", m_interrupt_line);
// we add space to account for overhang from the last packet - the rtl8139 // we add space to account for overhang from the last packet - the rtl8139
// can optionally guarantee that packets will be contiguous by // can optionally guarantee that packets will be contiguous by
// purposefully overrunning the rx buffer // purposefully overrunning the rx buffer
klog() << "RTL8139: RX buffer: " << m_rx_buffer->physical_page(0)->paddr(); dbgln("RTL8139: RX buffer: {}", m_rx_buffer->physical_page(0)->paddr());
for (int i = 0; i < RTL8139_TX_BUFFER_COUNT; i++) { for (int i = 0; i < RTL8139_TX_BUFFER_COUNT; i++) {
m_tx_buffers.append(MM.allocate_contiguous_kernel_region(page_round_up(TX_BUFFER_SIZE), "RTL8139 TX", Region::Access::Write | Region::Access::Read)); m_tx_buffers.append(MM.allocate_contiguous_kernel_region(page_round_up(TX_BUFFER_SIZE), "RTL8139 TX", Region::Access::Write | Region::Access::Read));
klog() << "RTL8139: TX buffer " << i << ": " << m_tx_buffers[i]->physical_page(0)->paddr(); dbgln("RTL8139: TX buffer {}: {}", i, m_tx_buffers[i]->physical_page(0)->paddr());
} }
reset(); reset();
read_mac_address(); read_mac_address();
const auto& mac = mac_address(); const auto& mac = mac_address();
klog() << "RTL8139: MAC address: " << mac.to_string().characters(); dmesgln("RTL8139: MAC address: {}", mac.to_string());
enable_irq(); enable_irq();
} }
@ -186,47 +186,41 @@ void RTL8139NetworkAdapter::handle_irq(const RegisterState&)
m_entropy_source.add_random_event(status); m_entropy_source.add_random_event(status);
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: handle_irq status={:#04x}", status);
klog() << "RTL8139NetworkAdapter::handle_irq status=0x" << String::format("%x", status);
#endif
if ((status & (INT_RXOK | INT_RXERR | INT_TXOK | INT_TXERR | INT_RX_BUFFER_OVERFLOW | INT_LINK_CHANGE | INT_RX_FIFO_OVERFLOW | INT_LENGTH_CHANGE | INT_SYSTEM_ERROR)) == 0) if ((status & (INT_RXOK | INT_RXERR | INT_TXOK | INT_TXERR | INT_RX_BUFFER_OVERFLOW | INT_LINK_CHANGE | INT_RX_FIFO_OVERFLOW | INT_LENGTH_CHANGE | INT_SYSTEM_ERROR)) == 0)
break; break;
if (status & INT_RXOK) { if (status & INT_RXOK) {
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: RX ready");
klog() << "RTL8139NetworkAdapter: rx ready";
#endif
receive(); receive();
} }
if (status & INT_RXERR) { if (status & INT_RXERR) {
klog() << "RTL8139NetworkAdapter: rx error - resetting device"; dmesgln("RTL8139: RX error - resetting device");
reset(); reset();
} }
if (status & INT_TXOK) { if (status & INT_TXOK) {
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: TX complete");
klog() << "RTL8139NetworkAdapter: tx complete";
#endif
} }
if (status & INT_TXERR) { if (status & INT_TXERR) {
klog() << "RTL8139NetworkAdapter: tx error - resetting device"; dmesgln("RTL8139: TX error - resetting device");
reset(); reset();
} }
if (status & INT_RX_BUFFER_OVERFLOW) { if (status & INT_RX_BUFFER_OVERFLOW) {
klog() << "RTL8139NetworkAdapter: rx buffer overflow"; dmesgln("RTL8139: RX buffer overflow");
} }
if (status & INT_LINK_CHANGE) { if (status & INT_LINK_CHANGE) {
m_link_up = (in8(REG_MSR) & MSR_LINKB) == 0; m_link_up = (in8(REG_MSR) & MSR_LINKB) == 0;
klog() << "RTL8139NetworkAdapter: link status changed up=" << m_link_up; dmesgln("RTL8139: Link status changed up={}", m_link_up);
} }
if (status & INT_RX_FIFO_OVERFLOW) { if (status & INT_RX_FIFO_OVERFLOW) {
klog() << "RTL8139NetworkAdapter: rx fifo overflow"; dmesgln("RTL8139: RX FIFO overflow");
} }
if (status & INT_LENGTH_CHANGE) { if (status & INT_LENGTH_CHANGE) {
klog() << "RTL8139NetworkAdapter: cable length change"; dmesgln("RTL8139: Cable length change");
} }
if (status & INT_SYSTEM_ERROR) { if (status & INT_SYSTEM_ERROR) {
klog() << "RTL8139NetworkAdapter: system error - resetting device"; dmesgln("RTL8139: System error - resetting device");
reset(); reset();
} }
} }
@ -292,12 +286,10 @@ UNMAP_AFTER_INIT void RTL8139NetworkAdapter::read_mac_address()
void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload) void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload)
{ {
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: send_raw length={}", payload.size());
klog() << "RTL8139NetworkAdapter::send_raw length=" << payload.size();
#endif
if (payload.size() > PACKET_SIZE_MAX) { if (payload.size() > PACKET_SIZE_MAX) {
klog() << "RTL8139NetworkAdapter: packet was too big; discarding"; dmesgln("RTL8139: Packet was too big; discarding");
return; return;
} }
@ -313,15 +305,13 @@ void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload)
} }
if (hw_buffer == -1) { if (hw_buffer == -1) {
klog() << "RTL8139NetworkAdapter: hardware buffers full; discarding packet"; dmesgln("RTL8139: Hardware buffers full; discarding packet");
return; return;
} else {
#if RTL8139_DEBUG
klog() << "RTL8139NetworkAdapter: chose buffer " << hw_buffer << " @ " << PhysicalAddress(m_tx_buffers[hw_buffer]);
#endif
m_tx_next_buffer = (hw_buffer + 1) % 4;
} }
dbgln_if(RTL8139_DEBUG, "RTL8139: Chose buffer {}", hw_buffer);
m_tx_next_buffer = (hw_buffer + 1) % 4;
memcpy(m_tx_buffers[hw_buffer]->vaddr().as_ptr(), payload.data(), payload.size()); memcpy(m_tx_buffers[hw_buffer]->vaddr().as_ptr(), payload.data(), payload.size());
memset(m_tx_buffers[hw_buffer]->vaddr().as_ptr() + payload.size(), 0, TX_BUFFER_SIZE - payload.size()); memset(m_tx_buffers[hw_buffer]->vaddr().as_ptr() + payload.size(), 0, TX_BUFFER_SIZE - payload.size());
@ -331,9 +321,7 @@ void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload)
// 60 bytes if necessary to make sure the whole thing is large enough. // 60 bytes if necessary to make sure the whole thing is large enough.
auto length = payload.size(); auto length = payload.size();
if (length < 60) { if (length < 60) {
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: adjusting payload size from {} to 60", length);
klog() << "RTL8139NetworkAdapter: adjusting payload size from " << length << " to 60";
#endif
length = 60; length = 60;
} }
@ -347,12 +335,10 @@ void RTL8139NetworkAdapter::receive()
u16 status = *(const u16*)(start_of_packet + 0); u16 status = *(const u16*)(start_of_packet + 0);
u16 length = *(const u16*)(start_of_packet + 2); u16 length = *(const u16*)(start_of_packet + 2);
#if RTL8139_DEBUG dbgln_if(RTL8139_DEBUG, "RTL8139: receive, status={:#04x}, length={}, offset={}", status, length, m_rx_buffer_offset);
klog() << "RTL8139NetworkAdapter::receive status=0x" << String::format("%x", status) << " length=" << length << " offset=" << m_rx_buffer_offset;
#endif
if (!(status & RX_OK) || (status & (RX_INVALID_SYMBOL_ERROR | RX_CRC_ERROR | RX_FRAME_ALIGNMENT_ERROR)) || (length >= PACKET_SIZE_MAX) || (length < PACKET_SIZE_MIN)) { if (!(status & RX_OK) || (status & (RX_INVALID_SYMBOL_ERROR | RX_CRC_ERROR | RX_FRAME_ALIGNMENT_ERROR)) || (length >= PACKET_SIZE_MAX) || (length < PACKET_SIZE_MIN)) {
klog() << "RTL8139NetworkAdapter::receive got bad packet status=0x" << String::format("%x", status) << " length=" << length; dmesgln("RTL8139: receive got bad packet, status={:#04x}, length={}", status, length);
reset(); reset();
return; return;
} }