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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F18BCCDB465 for ; Thu, 19 Oct 2023 15:40:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345508AbjJSPk7 (ORCPT ); Thu, 19 Oct 2023 11:40:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51520 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1345596AbjJSPk6 (ORCPT ); Thu, 19 Oct 2023 11:40:58 -0400 Received: from mail-il1-x133.google.com (mail-il1-x133.google.com [IPv6:2607:f8b0:4864:20::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4120313A for ; Thu, 19 Oct 2023 08:40:55 -0700 (PDT) Received: by mail-il1-x133.google.com with SMTP id e9e14a558f8ab-3574c225c14so10410115ab.0 for ; Thu, 19 Oct 2023 08:40:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20230601.gappssmtp.com; s=20230601; t=1697730054; x=1698334854; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=DEnVh39G2MAZ987aIaVUYRG5XCyY7oegMKrJw/1orDU=; b=uzXkV7cjvaVodluXe6r7PjchUiTcjp99p1vLRgfX1Dzlo5UusIbkIv348yTshly5QD F3GK1P4mEXkgFKDe25nh6g3wJYaRcLnNGsu2p1g2JLy/f+PB1NFXYaCYUVGa9vAt7xrG 06gr3GfLyhkqwldgdGmxfV/DY4xUt3nnpRvP2UBSodyhEssPq76F0yvC6E9XXO+0m4dS 2q5Fuh/1uHLzM3N20Fg74MFXVdkSs0XOaHibdGiINLyKsKBJDB9COCqmDVMVV7KoHyPQ UE6uxCVCtmmKLvBxTAzJXcWrzpFW4NmaSamdwbGUXQaVqnEVNHpLgDy8pxFY+nM1+/h7 NvxA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697730054; x=1698334854; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=DEnVh39G2MAZ987aIaVUYRG5XCyY7oegMKrJw/1orDU=; b=eReQozn3U4uE501ngc10n0fGycXq8ojC3N1X+4DwxBYLAAEHjd3GaTHdEmURcfQffe 92NoX4a+94T2x9vxC79QYWxHbb1EtFsewf3Y9M0rihthRNkUMXdz+ODJOoopv4vKdu/u Ox9urzhgaUps/e0tjD9PXfSEPrLBHAuJJ+s+bRkz+RTj1MORx6x353c/H3FdpiIG30tS /ujQm6B3lnNeZ+a8q0VMYhwM8TQ4OC1/R5hPozWc3tGpSYQz9gBps3ATesIENl/rukK8 nCCtSNLji+OtiXlYwtlqIQA6yD6FRf5VIE8vaWfyRvPNgUGx1ZeBlacKgR7rf1K59DfV EGRQ== X-Gm-Message-State: AOJu0YzM5J0Q6pS1z8vMCaiMB+CLGhQIdkjglEXmwV/wckQ9E2JjPx5v SzCSJQvR6D7FOUXyyn69AKujLQ== X-Google-Smtp-Source: AGHT+IFjQ3qpZyxj51SibTF8YykdYd4Gbz5spT1Xb9tdSDNeWvrpy8xIcdHmWjN10zegDqEeGeibwg== X-Received: by 2002:a92:d686:0:b0:351:54db:c1bb with SMTP id p6-20020a92d686000000b0035154dbc1bbmr2603572iln.0.1697730054404; Thu, 19 Oct 2023 08:40:54 -0700 (PDT) Received: from [192.168.1.94] ([96.43.243.2]) by smtp.gmail.com with ESMTPSA id i22-20020a056638381600b0045bc0378faasm2000238jav.29.2023.10.19.08.40.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 19 Oct 2023 08:40:53 -0700 (PDT) Message-ID: <11debbc9-de33-4466-a997-c8f49d27fd18@kernel.dk> Date: Thu, 19 Oct 2023 09:40:52 -0600 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v7 00/11] io_uring: Initial support for {s,g}etsockopt commands Content-Language: en-US To: Jakub Kicinski Cc: Breno Leitao , sdf@google.com, asml.silence@gmail.com, willemdebruijn.kernel@gmail.com, pabeni@redhat.com, martin.lau@linux.dev, krisman@suse.de, bpf@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, io-uring@vger.kernel.org References: <20231016134750.1381153-1-leitao@debian.org> <7bb74d5a-ebde-42fe-abec-5274982ce930@kernel.dk> <20231019083305.6d309f82@kernel.org> From: Jens Axboe In-Reply-To: <20231019083305.6d309f82@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: io-uring@vger.kernel.org On 10/19/23 9:33 AM, Jakub Kicinski wrote: > On Thu, 19 Oct 2023 08:58:59 -0600 Jens Axboe wrote: >> On 10/16/23 7:47 AM, Breno Leitao wrote: >>> This patchset adds support for getsockopt (SOCKET_URING_OP_GETSOCKOPT) >>> and setsockopt (SOCKET_URING_OP_SETSOCKOPT) in io_uring commands. >>> SOCKET_URING_OP_SETSOCKOPT implements generic case, covering all levels >>> and optnames. SOCKET_URING_OP_GETSOCKOPT is limited, for now, to >>> SOL_SOCKET level, which seems to be the most common level parameter for >>> get/setsockopt(2). >>> >>> In order to keep the implementation (and tests) simple, some refactors >>> were done prior to the changes, as follows: >> >> Looks like folks are mostly happy with this now, so the next question is >> how to stage it? > > Would be good to get acks from BPF folks but AFAICT first four patches Agree, those are still missing. BPF folks, do patches 1-2 look OK to you? > apply cleanly for us now. If they apply cleanly for you I reckon you > can take them directly with io-uring. It's -rc7 time, with a bit of > luck we'll get to the merge window without a conflict. I'll tentatively setup a branch for this just to see if we run into anything on the merge front. Depending on how the BPF side goes, I can rebase/collect reviews/whatever as we go. -- Jens Axboe