From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E4D711F4613 for ; Thu, 12 Feb 2026 23:08:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770937698; cv=none; b=oHr4hvDCeCNrbBZxvhKguZRohrp73JvaKQCE1r43+CeOFwz1X3EG0aQ8TfG1PaJ99LU0kl5Zyukk/N0HTzlUNz7oJ1UYHqoQcT5+xnlv8Uqzzfrf4OGkZ9GSLcRDd9fq8oAGtUbdUgbtUFehi5F0YqpQUpkBhLOZifj78tPBTe0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770937698; c=relaxed/simple; bh=+2qQ4Lqt58YN0m8NGbzAX704gO2yCJz3ZLqmn2FvBUI=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=RHL39Dbw1GidN1AawhgQURjyTU7S7bhzZweVU0ohk6eWDbfc5rotpXt8XdAf+XbmIIJj6hqBTtO4dO0I+q7+8qUK9FOFIHwYGT9SETVhN5L5RKXf6lQyChAJeo5/rIQo5GcCy0IAaqNmdkCv16925ll3UgzIVda5PD9QOPFMr7Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DMZxXFWE; arc=none smtp.client-ip=209.85.214.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DMZxXFWE" Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-2aaf9191da3so2046605ad.2 for ; Thu, 12 Feb 2026 15:08:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770937697; x=1771542497; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=NWqY5z6+CItkRRdlJX+08xGLqf2S+uEzXHa8ErcapMA=; b=DMZxXFWEFBEmWetI+KNI4lMwsDy2HSTB39ft41eCS7U0HkP/ei+JssxyEs0L/WqyvK lFhFzWIm2l5Frt/YiWLd+ntybd/dA95BAqaJgidolVwj9pw9FBAT1/gxpC8ENxSDE+5c 2Eb8wxtULBlermd45xxJYMPG7ilCXY/0z+B3N+31LTMfrNq1AkxkiYVQ2qQgY7uFvpl6 sGmo50akRDO6jdH/Vy2tDtT/V9xa6k7xZgXOfHwcJzHbDrlAc+GrbZ4Lv6Rs4hyZKN7S hkgkbtFH7Wy73NcdaXUu87p8KDiPBn4yYGa2GqN8Bmtrjav6UzogJ356vW6kmuUmmeAW pnCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770937697; x=1771542497; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=NWqY5z6+CItkRRdlJX+08xGLqf2S+uEzXHa8ErcapMA=; b=ImSLwLqjRvfgXPb8Okf4aEwMdBV7D+vFN8n6li5BrWAxYIz76SwxIM0d9V76g8mkYX dw7dAcev6px/prvUQbG8lh56qp8C411xX+NHNMykI5g8gPW2SCwsqYk5CbMOIeewtlpL bWu9HD7ObgmnEYDBnFdVAyy8LbQqX9JbnjY0oKGJ6tOLzsVG348WW/71PFjRwuQ3OYUR Sym5oULdzpOSxzU/eVCa2wPGWjrN5WFStcJSPIlFyMIRxYZo6kRAFRXS/FFiCy0SDjhL p8ANqdl92ru37/1IFgmt6bfXLSAJBNHk0Uqp8nWN9hGewjcJDC5DhfPNQyyWrJ7AgDma KCKw== X-Forwarded-Encrypted: i=1; AJvYcCWw4PIZj1pYz+TKIrz5HKdSLohLkrvyKmzFqcn1HlqJKTgEk6nOnWp0+N6K/uCmxV8dkcJ3riU4SLqsDvY=@vger.kernel.org X-Gm-Message-State: AOJu0YyC2LfXp2UAW/iFpgpLPnFNM8VeOhm9lRKSifdkBMRQvoa0W1fL wmduqlQEUY+7kglWGM3JMAPC75tgBparvYvuwgYBffv0kHkBWxBXtWxzZZWswdrB X-Gm-Gg: AZuq6aLykmjwWHcbgRDcIL1RBNlQlpKP4bShcmoVIY4nWvyjtIAJhqVTs1pYckXBcpJ vPJsbSQIs/FxRx/iolZ+o9mm5z2XSxGr5tTXyrlC/VZk9AFN6uPcaJ+8bjqdh47qoFu7izyvCtM ULkTurJ7Fr7zkwtn+VPpaLxTYbJ9PZj+lWPIbigWTQqYc+rbb7NxuHt3y29ciUeD3927egWMUhj aL8IRt5ijAgx5PTlfRxXH5KfUPwoPq9Lo0Dr3XlXdq2Qk+xfEuUttEkoQE5H2sFVzJMC/XUA7gh ik/BlRKgnsrZLH0kdJy/3yaRalNKYEz5ZLwThWKfxCZYbKJnuK/NJnT5Be/6BXKyBk2v/9gBJHd 6ofwArPqNnqHEVdNSXMMBPuXcMz3i14OOfxI8jp2SjIgQQ11Wkon7fOzfLvnQKx90tXjRjzmYDA 51J+OuXaJkcGSSRxRNlShFalDAFG1BsxAT8LvUt6l0wWSt3HAvY/s= X-Received: by 2002:a17:903:2441:b0:2a2:d2e7:1601 with SMTP id d9443c01a7336-2ab4d0193admr4925935ad.48.1770937697280; Thu, 12 Feb 2026 15:08:17 -0800 (PST) Received: from [192.168.0.56] ([38.34.87.7]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2ab29873c5esm64375435ad.36.2026.02.12.15.08.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Feb 2026 15:08:16 -0800 (PST) Message-ID: <95a233fe9a6058fb800cd0abca17f79e9bfa0c97.camel@gmail.com> Subject: Re: [PATCH bpf-next v1 05/14] selftests/bpf: Fix memory leaks in tests From: Eduard Zingerman To: Ihor Solodrai , Alexei Starovoitov , Andrii Nakryiko , Daniel Borkmann Cc: Amery Hung , Mykyta Yatsenko , Alexis =?ISO-8859-1?Q?Lothor=E9?= , bpf@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-team@meta.com Date: Thu, 12 Feb 2026 15:08:14 -0800 In-Reply-To: <20260212011356.3266753-6-ihor.solodrai@linux.dev> References: <20260212011356.3266753-1-ihor.solodrai@linux.dev> <20260212011356.3266753-6-ihor.solodrai@linux.dev> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.1 (3.58.1-1.fc43) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Wed, 2026-02-11 at 17:13 -0800, Ihor Solodrai wrote: [...] > @@ -2005,6 +2005,7 @@ int testapp_stats_rx_full(struct test_spec *test) > =C2=A0{ > =C2=A0 if (pkt_stream_replace(test, DEFAULT_UMEM_BUFFERS + DEFAULT_UMEM_B= UFFERS / 2, MIN_PKT_SIZE)) > =C2=A0 return TEST_FAILURE; > + pkt_stream_delete(test->ifobj_rx->xsk->pkt_stream); This one is a bit strange. pkt_stream_replace() allocates test->ifobj_rx->xsk->pkt_stream and test->ifobj_tx->xsk->pkt_stream (rx and tx streams), and proceeds to immediately free the rx stream and allocate another one for it. The following seem to be a better fit: - if (pkt_stream_replace(test, DEFAULT_UMEM_BUFFERS + DEFAULT_UMEM_= BUFFERS / 2, MIN_PKT_SIZE)) - return TEST_FAILURE; - pkt_stream_delete(test->ifobj_rx->xsk->pkt_stream); + test->ifobj_tx->xsk->pkt_stream =3D pkt_stream_generate(DEFAULT_U= MEM_BUFFERS + DEFAULT_UMEM_BUFFERS / 2, MIN_PKT_SIZE); test->ifobj_rx->xsk->pkt_stream =3D pkt_stream_generate(DEFAULT_U= MEM_BUFFERS, MIN_PKT_SIZE); (I effectively inlined pkt_stream_replace()). > =C2=A0 test->ifobj_rx->xsk->pkt_stream =3D pkt_stream_generate(DEFAULT_UM= EM_BUFFERS, MIN_PKT_SIZE); > =C2=A0 > =C2=A0 test->ifobj_rx->xsk->rxqsize =3D DEFAULT_UMEM_BUFFERS; > @@ -2017,6 +2018,7 @@ int testapp_stats_fill_empty(struct test_spec *test= ) > =C2=A0{ > =C2=A0 if (pkt_stream_replace(test, DEFAULT_UMEM_BUFFERS + DEFAULT_UMEM_B= UFFERS / 2, MIN_PKT_SIZE)) > =C2=A0 return TEST_FAILURE; > + pkt_stream_delete(test->ifobj_rx->xsk->pkt_stream); > =C2=A0 test->ifobj_rx->xsk->pkt_stream =3D pkt_stream_generate(DEFAULT_UM= EM_BUFFERS, MIN_PKT_SIZE); Same here. [...]