Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp1248549imm; Sun, 15 Jul 2018 03:05:11 -0700 (PDT) X-Google-Smtp-Source: AAOMgpebC96ja8R9rjIHzbjR8P5R/O9bKWt/lnUYT4Xg2vZjFKljB/Zt4AQEe89er0bAdadppAjS X-Received: by 2002:a17:902:a60b:: with SMTP id u11-v6mr12694382plq.158.1531649111617; Sun, 15 Jul 2018 03:05:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531649111; cv=none; d=google.com; s=arc-20160816; b=X5gLHqZyPv+zu6ePzMFfba+FqPlNYLextYe9Rqcpns4kpSa5A/p68gbYgo0rzO4Tkg pQi1ZLWBOIElbYIoYhKcE50mxEZv3hyeemH2DsCnmqZckR2mYu9bWPwT879kbTQNXaVV jJewHCkYp5FRM6Qz3Ev1ZYD1+yfF3Zc356wgi2Mug2+YM3dGXx26UI0GoeBvBsj5gq8f Ay7q86X4DlKsIsopZyjRxemx4/R4ira80xiGErUzLykEXSbRR6Igc3kRZTC+RTRlZ/gH loyaLzoUjh1Xn3fcEMO3xF/CS50DWQDEd5UBqCh4QNSZIrsg+GIi09XOJ2BYm+ntQroG nZEA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=F5Rns9G0EPreFrKjym52tP2SgPNuDIZnZZz2Vbz5qWU=; b=NQpFNHruh6cHFTAv+bT+oNhy8HO/6m+QFvPWqnZoZSF/+nmYBlwL0FZVEwWqWuxA8c KzT9lvLU6fY3V3JZ5YlTyMXLbpvVsdiHSzTVseJCyBoLxsBGNUaa/lqmYZkew99wIa4B atPEysVhJ9yZZ5vQQ9aKPmyJmNvrbg7VA1acc/WDsLhgdxl9S+wK+4LJxoRJrs9FOjFr htnx9MOM+67aQQKOcS6GvF4gOvrnZgpHWvcX7HUI8fFFiEZg1vKzweWUc+XJUvNkDEkg NUV3JlSaoeoo/YwZwsgNNkX4g4ad7QPdbEqJthuXdbKE7rtMQor6PzDIAfklkPOi4C0M PoGw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id m18-v6si26422807pgi.688.2018.07.15.03.04.44; Sun, 15 Jul 2018 03:05:11 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726156AbeGOK0N (ORCPT + 99 others); Sun, 15 Jul 2018 06:26:13 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:34742 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726115AbeGOK0N (ORCPT ); Sun, 15 Jul 2018 06:26:13 -0400 Received: from localhost (LFbn-1-12247-202.w90-92.abo.wanadoo.fr [90.92.61.202]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id C38A54A3; Sun, 15 Jul 2018 10:03:47 +0000 (UTC) Date: Sun, 15 Jul 2018 12:03:45 +0200 From: Greg Kroah-Hartman To: Dmitry Torokhov Cc: devel@driverdev.osuosl.org, toddpoynor@gmail.com, frankhu@chromium.org, jnjoseph@google.com, lkml , Simon Que , rspringer@google.com, Guenter Roeck , toddpoynor@google.com Subject: Re: [PATCH 11/18] staging: gasket: always allow root open for write Message-ID: <20180715100345.GA17618@kroah.com> References: <20180714055816.223754-1-toddpoynor@gmail.com> <20180714055816.223754-12-toddpoynor@gmail.com> <20180715090544.GC23333@kroah.com> <20180715093216.GA16003@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.0 (2018-05-17) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jul 15, 2018 at 12:53:09PM +0300, Dmitry Torokhov wrote: > > I can't wait for people to just realize this whole "new" subsystem can > > be replaced with UIO, but that's a topic for a different thread... > > Yes, that is true and that is why I am not sure why we are going > through all this staging exercise. > > As far as I understand we'd still need to have quite a bit of kernel > code so that we can safely program DMA controller (it does not look > like uio_dmem_genirq.c is sufficient as is for gasket needs), but that > should be solvable. I agree, it should be solvable, and much smaller and simpler than this whole large chunk of "subsystem+driver" code. But I'm not the one having to do this work, and it provides a bunch of easy cleanups for people looking to get into kernel development, so I don't mind :) But the "maintainers" should keep this in mind, as it is, this code is _not_ acceptable for the main kernel tree because of the UIO framework already present. thanks, greg k-h