From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C894D462CD for ; Wed, 26 Feb 2025 21:13:06 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C01194060F; Wed, 26 Feb 2025 21:13:06 +0100 (CET) Received: from mail-il1-f178.google.com (mail-il1-f178.google.com [209.85.166.178]) by mails.dpdk.org (Postfix) with ESMTP id 02AFE4029A for ; Wed, 26 Feb 2025 21:13:04 +0100 (CET) Received: by mail-il1-f178.google.com with SMTP id e9e14a558f8ab-3cf8e017abcso966325ab.1 for ; Wed, 26 Feb 2025 12:13:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind.com; s=google; t=1740600784; x=1741205584; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=Ws+8PV/NOMOIjDbQwBA6QjuisGMCANFWr87lo8Mpkk4=; b=CraJRl5Tfxwek5kMygQH8uM8rNW6wgqgWxTOwYMFz3FBwoUIi0F2YY+nzWcwNhfxyf qK2qqq383aNFZzyJolieEqQsrl+R/Xh0+jQ2vKrU5RBI5OnmrqyK9L+p5iVKg5dGJ4T3 I/LMXF+Ni+BpNjrJtT4zFYQg+VxCRLjc30fDNNWqy5EbY9hY347mXUUofVc6a+a8wV/G bUmnK6hHPHwjxv1teYaZI4uFS+3qI4xsH1sD0+QIm5Flf39r7jWWwaQwS6Qa/jjIxrJL rlIUGB8UQoKHH36pQGSK83ru9r7PFxPrWfGXKjqX14d8PCYzpyVMLB/mIoGHA8UiVxot XYFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740600784; x=1741205584; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Ws+8PV/NOMOIjDbQwBA6QjuisGMCANFWr87lo8Mpkk4=; b=Inhdgr9HQmIHOjCEpFyuEUurtTbH/TYvX85DNnylbg8ersU5DJX92UDlK4lhN+yGhv 073VEkIPLaJunCzKDWW55fQHAeNgr9nOHGEL0jxLVv3IE3JBKRV+lGvStFTTsiRG/GIz CB97VdtvoaRqIqY67AfCC9Ex83DgXBoZOHKuGu1jmsqCbYrPv5IeBEeRzEhMP/j4TQdi D/b0XbxYepZx9u00oUFSJ7zkRPAW+9+3/cGZ5+gGAUa09EJi8sM7WjS5tZTFbzGc4hD6 59sQbFWisA9QHe7jHOEl/wX8lZ3OuI3p8HCWfEQAQMYgryATiSdVBPflpVDYHS/0+MvT AO6Q== X-Forwarded-Encrypted: i=1; AJvYcCU1qoisM0zEGskdSCQFFUGMArGxhmHZKmhvbik53KGjs7+9A7MpSCn9OcAG3r8A898c9tY+0So=@dpdk.org X-Gm-Message-State: AOJu0YzxWPeA6SNnA3TgMSC6cTYUB1SVJadfHaqv5HyrUaUh/tFkTYZq YzQIhz8ZaUorF7qXebpy+e3Jf9V6Z/osbi4WqSPkpkXIPylt8MB2NQCS6sNCpDXexnqND4hz+aC bqlh27Pg44VEwQPWnm86zNC92cG190Oq8bOtB5w== X-Gm-Gg: ASbGnctiq6ZdI1ILDblz8sdjDpNktvxra5Ydn/0Z3apqN/4xcLO6jlI6VTDi0CLtj/A SsoGjFHcUpWKGL75yjux9iX1wdcQtWZK3yvnv2GNQK6vdkpooNzBnyXvIA84eC9TOCSCHpuZqzJ LPPN22NB6tm3A7xmNi9oy9HvoXNxR97msWigCNn20= X-Google-Smtp-Source: AGHT+IGo3QrjiuLr/CcQ2zHvKzt0cl4IZfDEPDDN44HOEeG9OYWqI9NKj2rTygKglQP5qgHRtcR36msbtf+bQRkLHTI= X-Received: by 2002:a05:6e02:1608:b0:3d0:47cf:869c with SMTP id e9e14a558f8ab-3d30488b517mr87403805ab.19.1740600784221; Wed, 26 Feb 2025 12:13:04 -0800 (PST) MIME-Version: 1.0 References: <20250116195640.68885-1-ariel.otilibili@6wind.com> <20250131231018.2163893-3-ariel.otilibili@6wind.com> <20250224112534.684c19c6@hermes.local> In-Reply-To: <20250224112534.684c19c6@hermes.local> From: Ariel Otilibili Date: Wed, 26 Feb 2025 21:12:51 +0100 X-Gm-Features: AQ5f1JoF9B3JyrQnJ0O9SoqsW0UEGJWyTEYoxUrH_h-yhwj5iYv-0hCbtAl5I6M Message-ID: Subject: Re: [PATCH v6 2/2] net/af_xdp: Refactor af_xdp_tx_zc To: Stephen Hemminger Cc: dev@dpdk.org, stable@dpdk.org, Thomas Monjalon , David Marchand , Maryam Tahhan , Ciara Loftus Content-Type: multipart/alternative; boundary="0000000000007097f9062f113380" X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org --0000000000007097f9062f113380 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hello Stephen, On Mon, Feb 24, 2025 at 8:25=E2=80=AFPM Stephen Hemminger < stephen@networkplumber.org> wrote: > > Please avoid doing assignment in a conditional statement, can be error > prone. > Surprised checkpatch doesn't complain about it. > Thanks for the feedback. checkpatch did catch them. Here is the new series, https://inbox.dpdk.org/dev/20250226200841.2342632-3-ariel.otilibili@6wind.c= om/ Regards, Ariel > > Better as: > desc =3D reserve_and_fill(txq, mbuf, umem); > if (!desc) { > kick_tx(txq, cq); > desc =3D reserve_and_fill(txq, mbuf, umem= ); > if (!desc) > goto out; > --0000000000007097f9062f113380 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hello Stephen,

O= n Mon, Feb 24, 2025 at 8:25=E2=80=AFPM Stephen Hemminger <stephen@networkplumber.org> wrote:

Please avoid doing assignment in a conditional statement, can be error pron= e.
Surprised checkpatch doesn't complain about it.
Thanks for the feedback. checkpatch did catch them.
=

Regards,
Ariel

Better as:
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 desc =3D reserve_and_fill(txq, mbuf, umem);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 if (!desc) {
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 kick_tx(txq, cq);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 desc =3D reserve_and_fill(txq, mbuf,= umem);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 if (!desc)
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 goto out= ;
--0000000000007097f9062f113380--