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;
Subject: Re: [PATCH] fuse: Add ioctl flag for compat ioctl with 64-bit time_t
To:     Miklos Szeredi <miklos@szeredi.hu>
Cc:     linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20190301170831.23612-1-abbotti@mev.co.uk>
 <CAJfpegsQvUGz-YRQVrfkTWuSFe==Hg_Y=VpV0ZagKe0qGfxQ7g@mail.gmail.com>
 <d2bab6ae-0c52-5a6a-78e4-72a5f298a85f@mev.co.uk>
 <CAJfpegvhvS2KEP3kFsSGXONVuBdSdpgy8OGqd79ehN0uGAy5VA@mail.gmail.com>
From:   Ian Abbott <abbotti@mev.co.uk>
Organization: MEV Ltd.
Message-ID: <4dba1140-be46-6c1b-d88a-4cdf4d69cdda@mev.co.uk>
Date:   Wed, 24 Apr 2019 13:33:56 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.6.1
MIME-Version: 1.0
In-Reply-To: <CAJfpegvhvS2KEP3kFsSGXONVuBdSdpgy8OGqd79ehN0uGAy5VA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-GB
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 24/04/2019 11:52, Miklos Szeredi wrote:
> On Tue, Apr 23, 2019 at 5:14 PM Ian Abbott <abbotti@mev.co.uk> wrote:
>>
>> On 23/04/2019 13:55, Miklos Szeredi wrote:
>>> On Fri, Mar 1, 2019 at 6:08 PM Ian Abbott <abbotti@mev.co.uk> wrote:
>>>>
>>>> Currently, a CUSE server running on a 64-bit kernel can tell when an
>>>> ioctl request comes from a process running a 32-bit ABI, but cannot tell
>>>> whether the requesting process is using legacy IA32 emulation or x32
>>>> ABI, for example.  In particular, the server does not know the size of
>>>> the client process's `time_t` type.
>>>>
>>>> For 64-bit kernels, the `FUSE_IOCTL_COMPAT` and `FUSE_IOCTL_32BIT` flags
>>>> are currently set in the ioctl input request (`struct fuse_ioctl_in`
>>>> member `flags`) for a 32-bit requesting process.  This patch defines a
>>>> new flag `FUSE_IOCTL_COMPAT_64TIME` and sets it if the 32-bit requesting
>>>> process (running on a 64-bit kernel) uses a 64-bit `time_t` type.
>>>
>>> Hi,
>>>
>>> Thanks for the patch.
>>>
>>> I think it should rather use in_x32_syscall() helper and follow that
>>> naming because there's apparently at least one example in xfs of a
>>> non-time_t related ioctl that varies between the x32 vs ia32.
>>
>> Hi Miklos,
>>
>> It is conceivable that COMPAT_USE_64BIT_TIME could be true for some
>> other arch/ABI (although currently it is only ever set for x86/x32).
>> Should it have separate flags for "compat 64-bit time" and "compat x32"
>> (even though that is currently redundant)?
> 
> No, it should just be a single flag, something like
> FUSE_IOCTL_COMPAT_X32 and the documentation should say that it implies
> 64bit time values.

Okay, since the subject line, description and code will change, I'll 
submit the new patch as a new patch, rather than a "v2".  (I'll mention 
its history below the scissors line.)  Please drop this patch.

-- 
-=( Ian Abbott <abbotti@mev.co.uk> || Web: www.mev.co.uk )=-
-=( MEV Ltd. is a company registered in England & Wales. )=-
-=( Registered number: 02862268.  Registered address:    )=-
-=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=-