From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0D210C5ACD6 for ; Wed, 18 Mar 2020 09:15:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D1C7820772 for ; Wed, 18 Mar 2020 09:15:45 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="AUPQPih5" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727516AbgCRJPp (ORCPT ); Wed, 18 Mar 2020 05:15:45 -0400 Received: from us-smtp-delivery-74.mimecast.com ([216.205.24.74]:31501 "EHLO us-smtp-delivery-74.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727466AbgCRJPp (ORCPT ); Wed, 18 Mar 2020 05:15:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1584522943; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=cdyCBIHnyr7Xc+U6VytosOiXyQzrj+T9xmE6ZmyOJuc=; b=AUPQPih5f2A0VzLB+Y7mlVkinRyhIrVPAnSEsOIeSN3Q9W6lDMMUjo+2sFrcG8wp/l4AxJ rn9NS3mYZrMgUmQT4SVrdh8xIthzQAjCsFoa7atAS2uy/apye4jY2fc89DKOP+p7woxtN4 6DjTk0GwkwlU+UoihIPA4q5sbPRDhwI= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-249-N-0fDEvdMAqL1myI4dPHLQ-1; Wed, 18 Mar 2020 05:15:42 -0400 X-MC-Unique: N-0fDEvdMAqL1myI4dPHLQ-1 Received: by mail-wr1-f71.google.com with SMTP id v7so2583397wrp.0 for ; Wed, 18 Mar 2020 02:15:42 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=cdyCBIHnyr7Xc+U6VytosOiXyQzrj+T9xmE6ZmyOJuc=; b=fFoemWk2eIYzpex7XfXhJxvVM3VN3zLIJ6+nh32ygdiLZEgRQOuCGaLArTjfcSg7n2 +8hXcdblAfYd5T1Zj7CPSOG359H17PZJtBw/49KZuMsELvQFl7ySQhGGjYB4OOplQSJT rLs5HBBZYx9/wKegGPKFTuY+NRnVE6INYkES7QvsfEuicIyQRrBgb6+5KGtvEooTNWYA 1vpY2fseguUg2Q+t6NysF9iTIj2enxQCqp/sYSNlmexBtoJwVzqjLJOv7MmvTbXxG47f bE1SBqkUf4mWz/UqUdKOIppIw0egAX1cVeWiUPu6tv7HutKlMyQpl5HUzJ8eVSTDfOgO RosA== X-Gm-Message-State: ANhLgQ2HLHvU4owf9h7UPumvEGmUeksG2nTIzPZRaPLHn3Qd1SV7nn7v YLs2mVPj6noIbM7+aRU6r/FVf1T12701Fb+FliTFL3jC5U1yFztvjCnUcUl1nD3Yn3wE9xPudkI Zbf8EslqZ0YaLP0ew X-Received: by 2002:a1c:408b:: with SMTP id n133mr3931089wma.182.1584522941559; Wed, 18 Mar 2020 02:15:41 -0700 (PDT) X-Google-Smtp-Source: ADFU+vtXC+ycWU260kpNdoWJH/ziotg2rWhVm2qN3+a+nSVwo11VcsvweA5wIQkst78hh7LDfN9Mtg== X-Received: by 2002:a1c:408b:: with SMTP id n133mr3930962wma.182.1584522940056; Wed, 18 Mar 2020 02:15:40 -0700 (PDT) Received: from alrua-x1.borgediget.toke.dk ([2a0c:4d80:42:443::2]) by smtp.gmail.com with ESMTPSA id q8sm3992856wrc.8.2020.03.18.02.15.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Mar 2020 02:15:38 -0700 (PDT) Received: by alrua-x1.borgediget.toke.dk (Postfix, from userid 1000) id 61060180362; Wed, 18 Mar 2020 10:15:38 +0100 (CET) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= To: Jesper Dangaard Brouer , sameehj@amazon.com Cc: Jesper Dangaard Brouer , netdev@vger.kernel.org, bpf@vger.kernel.org, zorik@amazon.com, akiyano@amazon.com, gtzalik@amazon.com, Daniel Borkmann , Alexei Starovoitov , John Fastabend , Alexander Duyck , Jeff Kirsher , David Ahern , Willem de Bruijn , Ilias Apalodimas , Lorenzo Bianconi Subject: Re: [PATCH RFC v1 09/15] xdp: clear grow memory in bpf_xdp_adjust_tail() In-Reply-To: <158446619342.702578.1522482431365026926.stgit@firesoul> References: <158446612466.702578.2795159620575737080.stgit@firesoul> <158446619342.702578.1522482431365026926.stgit@firesoul> X-Clacks-Overhead: GNU Terry Pratchett Date: Wed, 18 Mar 2020 10:15:38 +0100 Message-ID: <87v9n2koqt.fsf@toke.dk> MIME-Version: 1.0 Content-Type: text/plain Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Jesper Dangaard Brouer writes: > To reviewers: Need some opinions if this is needed? > > (TODO: Squash patch) > --- > net/core/filter.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/net/core/filter.c b/net/core/filter.c > index 0ceddee0c678..669f29992177 100644 > --- a/net/core/filter.c > +++ b/net/core/filter.c > @@ -3432,6 +3432,12 @@ BPF_CALL_2(bpf_xdp_adjust_tail, struct xdp_buff *, xdp, int, offset) > if (unlikely(data_end < xdp->data + ETH_HLEN)) > return -EINVAL; > > + // XXX: To reviewers: How paranoid are we? Do we really need to > + /* clear memory area on grow, as in-theory can contain uninit kmem */ > + if (offset > 0) { > + memset(xdp->data_end, 0, offset); > + } This memory will usually be recycled through page_pool or equivalent, right? So couldn't we clear the pages when they are first allocated? That way, the only data that would be left there would be packet data from previous packets... -Toke