From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) (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 3A35134F494 for ; Tue, 2 Sep 2025 20:13:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756844024; cv=none; b=Tk6qdlVCKjblK2qyZMOckYE8NSph5Y2e2Zdf4IBkK+T2InmmdM8+nNd3OA1Rm/W8Xlz92LfZdg6TfK7JPlzfEn9XjGSvI9F8YsP1kjA+/vvAyO85ZPwvJevdq1pCUAcGunp18d3vRGy9/W/xPU+V3Q5Ja/Rcc0kz4HO2u8mzqmk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756844024; c=relaxed/simple; bh=JFMvSJKmvouio1S2V97JFSH+oyA42x64UOXLlbXqEEY=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=iEnihBRqbuctNdMDk83EEyPdBMrgwOxA8oJvEE6e+etyZPWpJQQEi0A+pXPzzxqpyWF0YUSqC1DUvfKCSACPff4FKkDlm7LnYOo42lw03aLe5+vo54Fvum7z9Z0OnWmurDs071/OFBx232J/G/zbsdqbRMSoj8oeIexQWQ0Nduk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=awkemBfs; arc=none smtp.client-ip=209.85.214.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="awkemBfs" Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-24b21006804so8942205ad.3 for ; Tue, 02 Sep 2025 13:13:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1756844022; x=1757448822; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=50VELORRmga2bsXVhAB0RUkFgMEc1EdgXr2M5pQzE2g=; b=awkemBfsfN0VFbiLtFC+o/J8zasnNGTzshHEw6q4JseloTHCSJ1dhvHTwMlXJzjpEa t5BiBEip1GFFImF8C5FAEgED82t8k/zKuje0TU39gnCEAFnQBz6raLMkwRExjCdoc+2Q VUtGtNnWYqtdhQG2CReU6KixRjBd3sfriqZ0ZNzifXMj4/puSqmP0B1IhTpwkVXVUFWD aHvqmik500vHG3U6Y7UUFE1AOJwnuqkdqHmmP6G1XFmHDvDDklZuqFLwSaV+PS13SENr 99Yq7r1hqJjRxejE2TqbMMgWpv1HMAFnFLZVDZXBsTrqYFaxsapZJsrcHeECF+bkRctV F40w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756844022; x=1757448822; h=content-transfer-encoding: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=50VELORRmga2bsXVhAB0RUkFgMEc1EdgXr2M5pQzE2g=; b=oPLMKEtk3KMS0FRSEVlIrvqIO9PdboRDbdz5AGC3UMErxqzPmPqqK7Vt9o4YDcClSG zcIdCpngT4UE9I/qket8txSe45HRaUwu4xRrGrp7+N5ad06A7M6mjaRGbM/TEpjZh7D9 xuQw0m3EWfhncRucPABnh2a/5w1go5K+tbCa3fSt2gzVoHVIjWKxBJbdeqpD1MRgDNXZ g1CiTSbI5qeIHmBk4uBwMchu0GRVGQjfLYJvz+yut1ssplpIiKAc0DGDZyaJT3Fo99CK o6DEuIPuJ37WxVC4Sgsgj09Mcglja+UV3AdjE7flUbHcLY4kEbigF05NlnB5RQRXYSJm e1cA== X-Forwarded-Encrypted: i=1; AJvYcCVa4ho1HF2qZF+wqMlgJWwh/YMunqy6J2CiHZhurtUiounxCyHEgD/sa5J88yQMB/eRhTRVofs=@vger.kernel.org X-Gm-Message-State: AOJu0YzQDW3VhSwzzp9mAD4/pRtnNqmSJdxQnUVxp9Vq40XgiJAgvbjy V3mFoUCyD7WXsUNfLa5nTS8Y6TQLDZ63QtbnHC6Ffp1BVUZNr60vnSQVFKKfRLSB9e92x6rfRMt +zITCiLsTVNSeV+R8PVxLgUjgy68rWX3tXwaUt4O7 X-Gm-Gg: ASbGnctf6uH+6YfycJsyMy5fZqWEBbwO1EEY4WiSDxnMmbKr2Sz/eGOMm4tI+wnGLTf eIB2KaGY22FWsVwDRswcUJyG55tQOJ5SnE43j6JLI+Pm7vYUHTnlIWTMFZehOL+ILSBhXJ9fF+V P0zwdylA0OWO4Y5J2zGTB1QOUB4j4dmDfIyIOFBYpsPn3+iQng/bM5e8Pfo723L3gxu1uqxQ8k1 pEKwKOzbn6k4Q78CPEMlASO229aJav+EfIXo5NYAMERlSDVxPSxCssGyzPJjA94CgVvWYtDMkvU hw== X-Google-Smtp-Source: AGHT+IF6Y+YbRwdDNXA1bFyeAXeEuQC7HJrV+I2THNQbBrLvAc/L/26mNWWGOWWKIr9NUWSQRYEgoem3+7Hig+8uuAY= X-Received: by 2002:a17:903:388c:b0:249:33db:34b with SMTP id d9443c01a7336-24944b3fa5amr148255155ad.42.1756844022235; Tue, 02 Sep 2025 13:13:42 -0700 (PDT) Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20250829010026.347440-1-kuniyu@google.com> <20250829010026.347440-4-kuniyu@google.com> In-Reply-To: From: Kuniyuki Iwashima Date: Tue, 2 Sep 2025 13:13:29 -0700 X-Gm-Features: Ac12FXxtUQxxFtFmkPSCPQp4CvvTcDOidnP1dpNPOA3b9fp5cg1RUDv7hT43kkM Message-ID: Subject: Re: [PATCH v4 bpf-next/net 3/5] bpf: Introduce SK_BPF_MEMCG_FLAGS and SK_BPF_MEMCG_SOCK_ISOLATED. To: Martin KaFai Lau Cc: Alexei Starovoitov , Andrii Nakryiko , Daniel Borkmann , John Fastabend , Stanislav Fomichev , Johannes Weiner , Michal Hocko , Roman Gushchin , Shakeel Butt , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Neal Cardwell , Willem de Bruijn , Mina Almasry , Kuniyuki Iwashima , bpf@vger.kernel.org, netdev@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Sep 2, 2025 at 1:02=E2=80=AFPM Martin KaFai Lau wrote: > > On 8/28/25 6:00 PM, Kuniyuki Iwashima wrote: > > static int sol_socket_sockopt(struct sock *sk, int optname, > > char *optval, int *optlen, > > bool getopt) > > @@ -5284,6 +5313,7 @@ static int sol_socket_sockopt(struct sock *sk, in= t optname, > > case SO_BINDTOIFINDEX: > > case SO_TXREHASH: > > case SK_BPF_CB_FLAGS: > > + case SK_BPF_MEMCG_FLAGS: > > if (*optlen !=3D sizeof(int)) > > return -EINVAL; > > break; > > @@ -5293,8 +5323,15 @@ static int sol_socket_sockopt(struct sock *sk, i= nt optname, > > return -EINVAL; > > } > > > > - if (optname =3D=3D SK_BPF_CB_FLAGS) > > + switch (optname) { > > + case SK_BPF_CB_FLAGS: > > return sk_bpf_set_get_cb_flags(sk, optval, getopt); > > + case SK_BPF_MEMCG_FLAGS: > > I would remove the getsockopt only support from the other hooks that cann= ot do > the setsockopt. There are other ways for them to read sk->sk_memcg if it = is > really needed. Ah, I forgot bpf_core_cast(). > > > + if (!IS_ENABLED(CONFIG_MEMCG) || !getopt) > > + return -EOPNOTSUPP; > > + > > + return sk_bpf_get_memcg_flags(sk, optval); > > Instead, do this only in bpf_sock_create_getsockopt. Will do. Thanks!