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

1 files changed, 26 insertions, 40 deletions

diff --git a/Kernel/Net/RTL8139NetworkAdapter.cpp b/Kernel/Net/RTL8139NetworkAdapter.cpp
index 44eafc4f45..6bdb612d2d 100644
--- a/Kernel/Net/RTL8139NetworkAdapter.cpp
+++ b/Kernel/Net/RTL8139NetworkAdapter.cpp
@@ -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.
  *
  * 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);
     set_interface_name("rtl8139");
 
-    klog() << "RTL8139: Found @ " << pci_address();
+    dmesgln("RTL8139: Found @ {}", pci_address());
 
     enable_bus_mastering(pci_address());
 
     m_interrupt_line = PCI::get_interrupt_line(pci_address());
-    klog() << "RTL8139: port base: " << m_io_base;
-    klog() << "RTL8139: Interrupt line: " << m_interrupt_line;
+    dmesgln("RTL8139: I/O port base: {}", m_io_base);
+    dmesgln("RTL8139: Interrupt line: {}", m_interrupt_line);
 
     // we add space to account for overhang from the last packet - the rtl8139
     // can optionally guarantee that packets will be contiguous by
     // 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++) {
         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();
 
     read_mac_address();
     const auto& mac = mac_address();
-    klog() << "RTL8139: MAC address: " << mac.to_string().characters();
+    dmesgln("RTL8139: MAC address: {}", mac.to_string());
 
     enable_irq();
 }
@@ -186,47 +186,41 @@ void RTL8139NetworkAdapter::handle_irq(const RegisterState&)
 
         m_entropy_source.add_random_event(status);
 
-#if RTL8139_DEBUG
-        klog() << "RTL8139NetworkAdapter::handle_irq status=0x" << String::format("%x", status);
-#endif
+        dbgln_if(RTL8139_DEBUG, "RTL8139: handle_irq status={:#04x}", status);
 
         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;
 
         if (status & INT_RXOK) {
-#if RTL8139_DEBUG
-            klog() << "RTL8139NetworkAdapter: rx ready";
-#endif
+            dbgln_if(RTL8139_DEBUG, "RTL8139: RX ready");
             receive();
         }
         if (status & INT_RXERR) {
-            klog() << "RTL8139NetworkAdapter: rx error - resetting device";
+            dmesgln("RTL8139: RX error - resetting device");
             reset();
         }
         if (status & INT_TXOK) {
-#if RTL8139_DEBUG
-            klog() << "RTL8139NetworkAdapter: tx complete";
-#endif
+            dbgln_if(RTL8139_DEBUG, "RTL8139: TX complete");
         }
         if (status & INT_TXERR) {
-            klog() << "RTL8139NetworkAdapter: tx error - resetting device";
+            dmesgln("RTL8139: TX error - resetting device");
             reset();
         }
         if (status & INT_RX_BUFFER_OVERFLOW) {
-            klog() << "RTL8139NetworkAdapter: rx buffer overflow";
+            dmesgln("RTL8139: RX buffer overflow");
         }
         if (status & INT_LINK_CHANGE) {
             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) {
-            klog() << "RTL8139NetworkAdapter: rx fifo overflow";
+            dmesgln("RTL8139: RX FIFO overflow");
         }
         if (status & INT_LENGTH_CHANGE) {
-            klog() << "RTL8139NetworkAdapter: cable length change";
+            dmesgln("RTL8139: Cable length change");
         }
         if (status & INT_SYSTEM_ERROR) {
-            klog() << "RTL8139NetworkAdapter: system error - resetting device";
+            dmesgln("RTL8139: System error - resetting device");
             reset();
         }
     }
@@ -292,12 +286,10 @@ UNMAP_AFTER_INIT void RTL8139NetworkAdapter::read_mac_address()
 
 void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload)
 {
-#if RTL8139_DEBUG
-    klog() << "RTL8139NetworkAdapter::send_raw length=" << payload.size();
-#endif
+    dbgln_if(RTL8139_DEBUG, "RTL8139: send_raw length={}", payload.size());
 
     if (payload.size() > PACKET_SIZE_MAX) {
-        klog() << "RTL8139NetworkAdapter: packet was too big; discarding";
+        dmesgln("RTL8139: Packet was too big; discarding");
         return;
     }
 
@@ -313,15 +305,13 @@ void RTL8139NetworkAdapter::send_raw(ReadonlyBytes payload)
     }
 
     if (hw_buffer == -1) {
-        klog() << "RTL8139NetworkAdapter: hardware buffers full; discarding packet";
+        dmesgln("RTL8139: Hardware buffers full; discarding packet");
         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());
     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.
     auto length = payload.size();
     if (length < 60) {
-#if RTL8139_DEBUG
-        klog() << "RTL8139NetworkAdapter: adjusting payload size from " << length << " to 60";
-#endif
+        dbgln_if(RTL8139_DEBUG, "RTL8139: adjusting payload size from {} to 60", length);
         length = 60;
     }
 
@@ -347,12 +335,10 @@ void RTL8139NetworkAdapter::receive()
     u16 status = *(const u16*)(start_of_packet + 0);
     u16 length = *(const u16*)(start_of_packet + 2);
 
-#if RTL8139_DEBUG
-    klog() << "RTL8139NetworkAdapter::receive status=0x" << String::format("%x", status) << " length=" << length << " offset=" << m_rx_buffer_offset;
-#endif
+    dbgln_if(RTL8139_DEBUG, "RTL8139: receive, status={:#04x}, length={}, offset={}", status, length, m_rx_buffer_offset);
 
     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();
         return;
     }