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 97647461BA for ; Fri, 7 Feb 2025 11:48:48 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8CAD9427DC; Fri, 7 Feb 2025 11:48:48 +0100 (CET) Received: from mail-il1-f182.google.com (mail-il1-f182.google.com [209.85.166.182]) by mails.dpdk.org (Postfix) with ESMTP id 51065427D0 for ; Fri, 7 Feb 2025 11:48:47 +0100 (CET) Received: by mail-il1-f182.google.com with SMTP id e9e14a558f8ab-3ce915a8a25so6187495ab.1 for ; Fri, 07 Feb 2025 02:48:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind.com; s=google; t=1738925326; x=1739530126; 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=mW+bTkxVkogKB6jBkmw916TWdFgbh8PxbS80ausoG58=; b=WfH0IacS5Pc9yO9hH4bj9Q4ckTOvGN7ey+VXOYA+S14IZtFakNCSNI5z4CNfb83rv4 CJsTyVifgXkxZLK5wxxBQRdv8T1RmeTiHh/wn0sqb1rD+OxCg05k1fiXxixnBMQWg6zU Xbdl25va/IWuGCdQ8RYxm9bqb+R6RDYYrySDJI5rN6VDN9EZ5itibY2dK9nL214209uN Y7QveI3WxNSxc+0lzEiQEfCk7olqzCxgcONeeWnju28bICU4E228tZWOsCkR+Ecdwjo5 TH1tZwlDWPxTcUcui2gdfR2wa3PEV/NMEJqW0QrbWhUVKLwAjGwLnuFT6GeTRokW6IaM h+oQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1738925326; x=1739530126; 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=mW+bTkxVkogKB6jBkmw916TWdFgbh8PxbS80ausoG58=; b=NeoPpKbvMRPZVQjbtfQFDb9QOk2xDOVK3I/Ic/e54uPdWR5UsDKwHpPclZw57C54Ka +nogwkaBCP9Q55D1QGDcBKpY45zabSSQnU5z7oOk9SG4/pnav+Tb9yEIpDPe116Yzdzv InuBTNXE0a4AJ5JoVpAcR9SvzzRdfZI68iaDmG8TeFzvFc0Dshkylq/xHG69p/BzbPUQ JSxuyTNd43VcozC2ef+fPq4OFANWv3EFsoNF6LcqMyVpakF6pbURwjW5Y2wpQQGhgJ2V QOPE4ttvwK9bNOdbJyOjvk53o10G1oCuRCtoEp4I9HuACcaaWIFTN4rolUMjVQpmkxWF sc+w== X-Forwarded-Encrypted: i=1; AJvYcCVQK4TLIcE6CpPuJlFirnlShJF9RFKzT60jYjSZjWh65KWEjN81Iyfs28xh6VBUvkykLR8OuUk=@dpdk.org X-Gm-Message-State: AOJu0Yx8Yypjz0c3kOwkZyBxX+UzF857GknvmWhaeWo5jlxutn3Fg/ep oALdHUJVOWEMabdVmSnmOaIlYWuePrBeAFY8xAXdRH2hW3ZMUyGxImRyA7eivsmLaNciRbXBhGU hnbFxek3ZAwzEOFKlm1M2d2tPpW2/7eV6bvBB8A== X-Gm-Gg: ASbGncsH3x+AmMC6sQjOgYZJ+gjrgfKZUQ8/8PhpdfF2+pq89PFjlu8PDD9HKqmuOJ3 +lj2w53OAiQ0ajlizxgiQzbqSkT1SX+bhdbIWAbreQ1xEz2yqEX4bMeNpenaoeh92HuDMisQ8Q2 xU4XCmS85MRZJWAKYLmH+3qhpZxRfw X-Google-Smtp-Source: AGHT+IGDjw35fQJ9dPWTYTdDXed7HcPEV7lQdEtqgRuPLI7v1/6wxqJq+LFeHN2PmhwmPM8sDaNDMWvcToiXQYLIc6U= X-Received: by 2002:a05:6e02:3304:b0:3cf:c8ec:d375 with SMTP id e9e14a558f8ab-3d13da20395mr22516045ab.0.1738925326584; Fri, 07 Feb 2025 02:48:46 -0800 (PST) MIME-Version: 1.0 References: <20250116195640.68885-1-ariel.otilibili@6wind.com> <20250206204645.1564535-1-ariel.otilibili@6wind.com> <20250206204645.1564535-3-ariel.otilibili@6wind.com> In-Reply-To: From: Ariel Otilibili Date: Fri, 7 Feb 2025 11:48:35 +0100 X-Gm-Features: AWEUYZlRJ8Xim6CRHsIionNz0XCw_f2-kYnIzJluBERXUAn1eViGXbFf7UUukUE Message-ID: Subject: Re: [PATCH v8 2/2] net/af_xdp: Refactor af_xdp_tx_zc To: Maryam Tahhan Cc: "dev@dpdk.org" , "stable@dpdk.org" , Thomas Monjalon , David Marchand , Stephen Hemminger , Ciara Loftus Content-Type: multipart/alternative; boundary="00000000000061ee19062d8b1a52" 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 --00000000000061ee19062d8b1a52 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Maryam, hi Stephen; On Fri, Feb 7, 2025 at 10:14=E2=80=AFAM Maryam Tahhan = wrote: > Sorry if my remarks were confusing, it was just missing from the previous > patch and it needs to be: > desc->len =3D mbuf->pkt_len; > > We need to keep this the same as the original code. This is a scenario > where we need to copy the data from an mbuf that isn't in from the pool o= f > buffers allocated for the umem. So the desc->len needs to be set to that = of > the (non umem) mbuf. > > The other changes look good. Nearly there, > > Thanks again > Thanks for the feedback. There is it, http://inbox.dpdk.org/dev/20250207104552.1663519-1-ariel.otilibili@6wind.co= m/ > > > --00000000000061ee19062d8b1a52 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi Maryam, hi Stephen;

On Fri, Feb 7, 2025 at 10:14=E2=80=AFAM Maryam Tahhan <mtahhan@redhat.com> wrote:
Sorry if my remarks were confusing, it was j= ust missing from the previous patch and it needs to be:
desc->len =3D mbuf->pkt_len;

We need to keep this the same as the original code. This is = a scenario where we need to copy the data from an mbuf that isn't in fr= om the pool of buffers allocated for the umem. So the desc->len needs to= be set to that of the (non umem) mbuf.=C2=A0

The other changes look good. Nearly there,

Thanks again


<snip>
--00000000000061ee19062d8b1a52--