Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp2261236pxa; Mon, 3 Aug 2020 11:27:34 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwyO6Lf9g2WgjH2883IaGFOaZwg6Cf08VbtnZIEY+MhsXW2e+GQdMz/SxlzBX/1gSICLm3s X-Received: by 2002:a50:fd19:: with SMTP id i25mr17507058eds.101.1596479254220; Mon, 03 Aug 2020 11:27:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1596479254; cv=none; d=google.com; s=arc-20160816; b=0stDrxdVELP2qIOuqVFCiL7NeQB/iNjTAB7PKT54s4J+rdE0FQUuMHWIblbXkkXMmi gu3H1/iE2PkFxJOW1lRsjNvTF9rI0VeN/o9ZOrKe2tlOtr/HyxiXXwEMaHF8EjblduQP fIvR7Nh+nkofGNqjDd1miWHd+JkinlnDC5B/AcrV/A+Lhd57JxKVbVRBtDb2HaQCxteI w7gFia0EmeGqQb+qxgw8i2jU0FY5OhovRmoujwebAB2X57Gdc16Qjvf0hvSbC/an6hFd iICw+dD/cFIWs0k0lpipb0Ia7RzPI6qOe4hbsJKT0Naul03MZW7bQqu89oa1tLg2W0h+ /GqQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=kP+FsGwT7Ong6g9Z4afLp/eWEQkLU5G56u7wCNQ8PbY=; b=rNSiBJDaswb7sF4zSFNut7cgvmryoPfW+9UgstuRZuJ/O1eRrH6YvF80f67ayEENXB iwZ1ijcgedQgky6DniDy/kPVTj1VJ7RLStgKs9xqji3sEYEzGfWr4eZ2pPpdOMc9WTU/ sBYN4IzHolm1Yis5wFhWQtI2axZNnBgnWwqh1n3MdbRi0E6oJTlSpJVp1JXxGMrTEwHh pleQ6o9y3dojmk45B7snIpt5YcAbWnt78FywRmHT8vohnH0pyR/TNVFNnM/YqFcW6aMv Ssg77W57t7OW/QnH1yntjWZ1Y36qf0u1CZeJpHiGPMIWg2flC1bMEPdGoUGQE6gNu6iU lYgg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=AJgTtJOf; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id qw20si813047ejb.371.2020.08.03.11.27.12; Mon, 03 Aug 2020 11:27:34 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=AJgTtJOf; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726130AbgHCS1E (ORCPT + 99 others); Mon, 3 Aug 2020 14:27:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41576 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728668AbgHCS1B (ORCPT ); Mon, 3 Aug 2020 14:27:01 -0400 Received: from mail-vs1-xe41.google.com (mail-vs1-xe41.google.com [IPv6:2607:f8b0:4864:20::e41]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C0E80C061756 for ; Mon, 3 Aug 2020 11:27:00 -0700 (PDT) Received: by mail-vs1-xe41.google.com with SMTP id b26so15182039vsa.13 for ; Mon, 03 Aug 2020 11:27:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=kP+FsGwT7Ong6g9Z4afLp/eWEQkLU5G56u7wCNQ8PbY=; b=AJgTtJOfq33QtwVrHYsaLOFspsbTiqgCrzwxWUY4KSq8nn719n+LK296FDkKFWb8T/ zvbXOvclUTRqLQF7WTykx9gCeghXvWxet31NMy9+UG+NrC+4/FTyOer5i1CmCmMo0PV1 yVvi/nIBZZJzUWIh9DtK+klhMtAxhPJ/ulejuNnLSgm60MrsJ2Q90/Typ5lUScJ085mD jNUdQOdm/4sVykvl/EH4WQG63v+mYA/ZS0d7HyrbkkRQjRXYZewhg6AiAQGcJcvP7FiZ olFj/Vgfz+QhA0pdRP+9yyIHjYV4J8WcoUK+EPB37dwI4LCLmCkX5KRmdpei3BwBnw5p +4Iw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=kP+FsGwT7Ong6g9Z4afLp/eWEQkLU5G56u7wCNQ8PbY=; b=IPJQBoECzvV0p4idgpY+f0ZX9vcpoxf2XwTbXN/ZbhJJsB1w2ghsWpLp9X4Yfjl886 ALxaieD3X8lfabf+K/mhFwOiAcSfmlwlsMwj4jbh0kCNAgChukc337duKvAu3mo0/36Z cutrEE5iaicUnked+Aypn/8bwc0tks63kaidBbzBSbBW7rGsm97Rc07aRD4L2zM8+Qw9 eHFqX2ygtnQW+vpV+2v27xP7KxiJBPGhrnxwOjN7lP1y1K3ULGYG8mahj60+Lww6YPYM JTz/lm6mi0tOoj2RZUHkIsmRF7ztbQ6HIVgolHBHhHMMNeb8lOiuBIFZJiQC5Yd5oM5q K4Lg== X-Gm-Message-State: AOAM531s0ToWV97xGU0lV00PwkRPSaE5xrPTi09uvviEIx5j9qujbozf DMDKoJWsSxGnNguzNQmkE2+SCA== X-Received: by 2002:a67:8084:: with SMTP id b126mr12585151vsd.163.1596479219744; Mon, 03 Aug 2020 11:26:59 -0700 (PDT) Received: from google.com (182.71.196.35.bc.googleusercontent.com. [35.196.71.182]) by smtp.gmail.com with ESMTPSA id o3sm1766447vka.42.2020.08.03.11.26.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Aug 2020 11:26:59 -0700 (PDT) Date: Mon, 3 Aug 2020 18:26:56 +0000 From: Kalesh Singh To: Christoph Hellwig Cc: Jonathan Corbet , Sumit Semwal , Alexander Viro , Steven Rostedt , Ingo Molnar , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org, linux-fsdevel@vger.kernel.org, Suren Baghdasaryan , Hridya Valsaraju , Ioannis Ilkos , John Stultz , kernel-team@android.com Subject: Re: [PATCH 1/2] fs: Add fd_install file operation Message-ID: <20200803182656.GA3230172@google.com> References: <20200803144719.3184138-1-kaleshsingh@google.com> <20200803144719.3184138-2-kaleshsingh@google.com> <20200803163429.GA15200@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200803163429.GA15200@infradead.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 03, 2020 at 05:34:29PM +0100, Christoph Hellwig wrote: > On Mon, Aug 03, 2020 at 02:47:18PM +0000, Kalesh Singh wrote: > > Provides a per process hook for the acquisition of file descriptors, > > despite the method used to obtain the descriptor. > > > > Signed-off-by: Kalesh Singh > > I strongly disagree with this. The file operation has no business > hooking into installing the fd. Hi Christoph. I am exploring the alternative suggested by Matthew in Patch 2/2. Thanks :)