Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp261013pxb; Wed, 15 Sep 2021 01:08:49 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzg/qgclDwCdqETXJpdEPRlwoclHIfb51QEdE31+QjbGdjS17bCGtRzyl7lvrLPgSztODdw X-Received: by 2002:a05:6512:3a94:: with SMTP id q20mr6685282lfu.660.1631693329571; Wed, 15 Sep 2021 01:08:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631693329; cv=none; d=google.com; s=arc-20160816; b=0vF1BUMP8FxQ1B/PbN+OH3CwLpqCtUn35zqESZJZ0s9F7fvwfXXqlVkKrZ74CCunPE y7nNusI9dcQ7gvISFMtx3qJmu52PgzLKbWfxlrGuzLGqDGjgez61ffyN6kPI/LGX9+Uo Nq5eelIYyO1gQ84Ng2v61GoufbBHNkrFszALOwJ3sCNKhEbk4yF+MCLwpx9t8lyr2oHA UnskpWskhSW1Az3p/ZLJl0Flbfdih1R2OHgTrZhRQ0Nq88KYmqZAHm/0K53uztg0G78b tKCoro5KCm9zZyUxeXytm//1KrDBVJSwHRqhEIPDKFfqpzjIVltM3fztO96dH1VrmG8w B8Rw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=51p/U6NJ5IDKAiDkT/rLW4HSJpPwg7/L9XiK+9cYdpI=; b=x7jvTrOZue6v55kZKKtpZb9Ni+oZjwyEnHPT+zNx7AnnAlGpZKyS0Dwy4moFGujzUP w0oHIMvq/IrK89W/ECQhZjBAbYWr/0hPFGDke7nl9IZgqlUi9jhwlZPFijUFBWyRhX+v hXVePWYwDu0EGeUPC4z6j/ck+pcd3/ZwgWz3jm5f5IxqxGcM77TYx5p/eLwNfH6iDS0Y S/uvfBpCqdfRUTrftRAHvaXOORZ1iV53w848S/2STyoCSc37KmR5tMfhQT+Lq4i1+O/r GAOay30bXlO6juEO04/4HYoEhMLQvFnwnLI7vkJ8h16ASAJMMmAyWQ/I6Y8zdBx9if6S Rg1A== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x4si19317373ljp.374.2021.09.15.01.07.50; Wed, 15 Sep 2021 01:08:49 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236717AbhIOIHE (ORCPT + 99 others); Wed, 15 Sep 2021 04:07:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55672 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236674AbhIOIHE (ORCPT ); Wed, 15 Sep 2021 04:07:04 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92D34C061574 for ; Wed, 15 Sep 2021 01:05:45 -0700 (PDT) Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=bjornoya.blackshift.org) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mQPvI-0002tT-6R; Wed, 15 Sep 2021 10:05:40 +0200 Received: from pengutronix.de (unknown [IPv6:2a0a:edc0:0:701:8a97:9358:ae:5dc8]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: mkl-all@blackshift.org) by smtp.blackshift.org (Postfix) with ESMTPSA id DA0B567E519; Wed, 15 Sep 2021 08:05:38 +0000 (UTC) Date: Wed, 15 Sep 2021 10:05:37 +0200 From: Marc Kleine-Budde To: Jiri Slaby Cc: gregkh@linuxfoundation.org, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, Wolfgang Grandegger , "David S. Miller" , Jakub Kicinski , Andreas Koensgen , Paul Mackerras Subject: Re: [PATCH 05/16] tty: remove file from tty_mode_ioctl Message-ID: <20210915080537.mcndyf6oot4v6hew@pengutronix.de> References: <20210914091134.17426-1-jslaby@suse.cz> <20210914091134.17426-5-jslaby@suse.cz> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="2wqqhlezubhyee7b" Content-Disposition: inline In-Reply-To: <20210914091134.17426-5-jslaby@suse.cz> X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: mkl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --2wqqhlezubhyee7b Content-Type: text/plain; charset=utf-8 Content-Disposition: inline On 14.09.2021 11:11:23, Jiri Slaby wrote: > The only user of 'file' parameter in tty_mode_ioctl is a BUG_ON check. > Provided it never crashed for anyone, it's an overkill to pass the > parameter to tty_mode_ioctl only for this check. > > If we wanted to check 'file' there, we should handle it in more graceful > way anyway. Not by a BUG == crash. > > Signed-off-by: Jiri Slaby > Cc: Wolfgang Grandegger > Cc: Marc Kleine-Budde > Cc: "David S. Miller" > Cc: Jakub Kicinski > Cc: Andreas Koensgen > Cc: Paul Mackerras > --- > drivers/net/can/slcan.c | 2 +- For the slcan.c: Acked-by: Marc Kleine-Budde regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | --2wqqhlezubhyee7b Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEK3kIWJt9yTYMP3ehqclaivrt76kFAmFBqU8ACgkQqclaivrt 76muZgf+MUl8Y9j/6pTFNjbSF5vBrZhE1PXzkvVTwWonq9eouUYLtNgMEIqh0IC7 NWPnQOZf916t2rb6OuQEQlgp/AV1MFmfeLoSK1LfLKDj+075eSu7CIn2MGeeoC/7 wllxFuOXkiJpMe0EXhzHsPj+yBXO3P0ZsZyfEsJ3eBMLu4kc7hu6LzbYCtVj50VS biN+g7QUOgGBi5TI4+rA/LeRF4uzvAufuukV9vGEKLX5Y7dbyy2p/mtnNiCdNnZt E/BZIuBTVTd5Unzk8K51oM9DFVSnbC3Q+wQhlKB21epWE03gUH/G4asomwbegWVq 3k9JL+FQW1trObJ2HblArdqZ7ZLcyw== =kmTG -----END PGP SIGNATURE----- --2wqqhlezubhyee7b--