From 465bf9e23403a94a79f19de752dc1bc035032d00 Mon Sep 17 00:00:00 2001
From: Remi Gacogne <remi.gacogne@powerdns.com>
Date: Mon, 22 Jan 2024 12:28:25 +0100
Subject: [PATCH] dnsdist: Clear the XSK responder notification queue right
 away

Otherwise we might discard a valid notification between our last
look at the queue and the clearing.
---
 pdns/dnsdistdist/dnsdist-xsk.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pdns/dnsdistdist/dnsdist-xsk.cc b/pdns/dnsdistdist/dnsdist-xsk.cc
index 2996683c95..fc77706aa0 100644
--- a/pdns/dnsdistdist/dnsdist-xsk.cc
+++ b/pdns/dnsdistdist/dnsdist-xsk.cc
@@ -48,6 +48,7 @@ void XskResponderThread(std::shared_ptr<DownstreamState> dss, std::shared_ptr<Xs
       bool needNotify = false;
       if ((pollfds[0].revents & POLLIN) != 0) {
         needNotify = true;
+        xskInfo->cleanSocketNotification();
 #if defined(__SANITIZE_THREAD__)
         xskInfo->incomingPacketsQueue.lock()->consume_all([&](XskPacket& packet) {
 #else
@@ -98,7 +99,6 @@ void XskResponderThread(std::shared_ptr<DownstreamState> dss, std::shared_ptr<Xs
           packet.updatePacket();
           xskInfo->pushToSendQueue(packet);
         });
-        xskInfo->cleanSocketNotification();
       }
       if (needNotify) {
         xskInfo->notifyXskSocket();
-- 
2.47.2