Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp1485723pxb; Sun, 17 Jan 2021 10:35:09 -0800 (PST) X-Google-Smtp-Source: ABdhPJzMSnlC7c/Eki86Q4HmH/ZMjWghFp7eo9s+VWF/B3ztjm8pMr5CWkBhtINIQNd+IPlo0dE+ X-Received: by 2002:a05:6402:d0a:: with SMTP id eb10mr16830839edb.249.1610908508901; Sun, 17 Jan 2021 10:35:08 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1610908508; cv=none; d=google.com; s=arc-20160816; b=EnULup+yuQyaTfctn9FVxIoo0O3W+rvRWogj6ChIcwBXYXogmMGCMbm/jutJwO2XxE ml34CGU6iQY4Bjt/t/T9YdWAjDP0uZ5IGyHwbwDcPYZqNmp4RHURxtmlfrTYCH21i+Ld rS15b1uV5HH+miOXvjBD9RHo5oFWz4MA+YcwQyxv4e9eXbGOY46LeV/Vx9Nb+BRjKMlF Bb3fpTkKbwwMP+QwHJnYvHQOWyMo4l7lNpAQ9PWnbraMq5M1rJk/ni7B/lUf7oGPREUY jN7ysmtWXeJ50fcCxYrrJO5JSdAK2u9FNjZv+F4+InllOqjaBxqetgZK32ZS0CArfmaK mErA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :mime-version:accept-language:in-reply-to:references:message-id:date :thread-index:thread-topic:subject:cc:to:from; bh=WqMkUDhnRaZi3OOOvmJ6ln8a4r1GXIiQ8aBdh4pijMY=; b=vugOHohiY9BvHq0Jp4L+Byy5IueeiBqHnIk+Wd7VB8lxBTsbUjWL/ZwuOZeXB02RG5 lh0oK/IYgc8SJ2dUkC6iebZa6+g+zXDAZlmzM7rSfFh8b1JtZSbVUi9CaNo/246lzaPB ugnlxsNw3ADJ6esDXU8XZIAcef2haXJtoBnjCP0JjRi1yirP3Rs7+0TAoe7e2Z6fNDx5 093Na+I7uJudvQfzwWM7Ao7zJfu82CvUsOslUFrCuzbQaQq6O0o8nmUbXD6aUj8OzH2J b/MEjgeDIXxYLwmc6e2RSqL2El45tUXV3FuzhzOUfeXUJtdBD9chbqV2tC3rGuPXZlVO VeGQ== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 13si3611099ejy.609.2021.01.17.10.34.45; Sun, 17 Jan 2021 10:35:08 -0800 (PST) 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729967AbhAQSdW convert rfc822-to-8bit (ORCPT + 99 others); Sun, 17 Jan 2021 13:33:22 -0500 Received: from eu-smtp-delivery-151.mimecast.com ([185.58.86.151]:55539 "EHLO eu-smtp-delivery-151.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729935AbhAQSdS (ORCPT ); Sun, 17 Jan 2021 13:33:18 -0500 Received: from AcuMS.aculab.com (156.67.243.126 [156.67.243.126]) (Using TLS) by relay.mimecast.com with ESMTP id uk-mta-165-qzM2FzfaOG6j69VeZBrcNw-1; Sun, 17 Jan 2021 18:31:37 +0000 X-MC-Unique: qzM2FzfaOG6j69VeZBrcNw-1 Received: from AcuMS.Aculab.com (fd9f:af1c:a25b:0:43c:695e:880f:8750) by AcuMS.aculab.com (fd9f:af1c:a25b:0:43c:695e:880f:8750) with Microsoft SMTP Server (TLS) id 15.0.1347.2; Sun, 17 Jan 2021 18:31:36 +0000 Received: from AcuMS.Aculab.com ([fe80::43c:695e:880f:8750]) by AcuMS.aculab.com ([fe80::43c:695e:880f:8750%12]) with mapi id 15.00.1347.000; Sun, 17 Jan 2021 18:31:36 +0000 From: David Laight To: 'Christoph Hellwig' , Andy Lutomirski CC: Arnd Bergmann , Ryan Houdek , Catalin Marinas , Will Deacon , Alexander Viro , Arnd Bergmann , Christian Brauner , Andrew Morton , Minchan Kim , Aleksa Sarai , Sargun Dhillon , Miklos Szeredi , Vincenzo Frascino , Amanieu d'Antras , Willem de Bruijn , YueHaibing , Xiaoming Ni , Heiko Carstens , "Eric W. Biederman" , Joe Perches , Jan Kara , David Rientjes , "Arnaldo Carvalho de Melo" , "David S. Miller" , Linux ARM , "linux-kernel@vger.kernel.org" , Linux FS-devel Mailing List , Linux API , linux-arch Subject: RE: [PATCH] Adds a new ioctl32 syscall for backwards compatibility layers Thread-Topic: [PATCH] Adds a new ioctl32 syscall for backwards compatibility layers Thread-Index: AQHW6+b/d1V6Thk8U0iiUgMFo8hTJqosJEIw Date: Sun, 17 Jan 2021 18:31:36 +0000 Message-ID: References: <20210106064807.253112-1-Sonicadvance1@gmail.com> <20210116090721.GA30277@lst.de> In-Reply-To: <20210116090721.GA30277@lst.de> Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.202.205.107] MIME-Version: 1.0 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=C51A453 smtp.mailfrom=david.laight@aculab.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: aculab.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Christoph Hellwig > Sent: 16 January 2021 09:07 ... > > I personally would like to see in_compat_syscall() go away, > > but some other people (Hi, Christoph!) disagree, and usage seems to be > > increasing, not decreasing. > > I'm absolutely against it going away. in_compat_syscall helped to > remove so much crap compared to the explicit compat syscalls. The only other real option is to pass the 'syscall type' explicitly through all the layers into every piece of code that might need it. So passing it as a 'parameter' that is (probably) current->syscall_type does make sense. It might even make sense have separate bits for the required emulations. So you'd have separate bits for '32bit pointers' and '64bit items 32bit aligned' (etc). David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)