Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp464621ybz; Wed, 22 Apr 2020 01:34:56 -0700 (PDT) X-Google-Smtp-Source: APiQypJk6oq7dMhpyH5dH1bcvJF9ASS5plysShplFKUDCIHuICSAWklvgUQRptpvNoYSFV4FvFCo X-Received: by 2002:a17:906:7c11:: with SMTP id t17mr25708278ejo.73.1587544496308; Wed, 22 Apr 2020 01:34:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1587544496; cv=none; d=google.com; s=arc-20160816; b=x/1rhsvOcUuQIFmpP4+xSkez3HOctiy59vm62gs67oFh9cz6Sm2lGzr+FQnF0ZW6mW 6A6Snmt/fkeA/+2da7FHJ6QQxbk/1ww6RZuHbkMqbhcYR/cYYscdoqmP+hZKsb3q/S/Y kqwK0Q81ry9fVmgfKGFF7hAA74a8RvjIp/5u8xLc9JOKl+ROf+7RRc/EpTfWppz0Kjoi 4694QN8FCAOcbauqrw6mhIRtb7YBnNe12wdR8nQozI1R7l+he3pdTmXr2iyvM8CBsHqx y9Q4/2LvMuoTMUxEO8u0ngJMtJNf7OkQ7IvRr48bD9vjLRsmdq9MJg/G5TRXp/vvnsOi MSzA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:autocrypt:from:references:cc:to:subject; bh=agTMX2MselmdZuLLyKamBPc/VIJvqLXA70Pup6Zmgv4=; b=p74i0bgPtUx18VW//j393Cmu8eimxJ3i1sSsVpTUr5LucocIGVqjU3Dp1xrMPtTj0o BomMI6GwvBEmFiaUjl5YWguNGQzTDrSE0R8t5xcogDq2vq0up0iXtS55URhJqts1ajWS A+TKajt6ISw0TniO09R2kbdWq6HHPi/fiJKdq6DzLEdVINA6p9q3BK08n75mzaqIOKPp x50IEK0qGUJG0QqClYihaKCs3ajX+VeOrCsehVG0vCe3Vz2CjrMcUzuMDGMji3t8SpXS tYacf7wzr6WVxEYdo5SrR/tVdR6wRuUbihMxlw0smrAR37Je5F/BLxwbySIqnWr5hUCz Ujdg== 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 z18si2963547edx.121.2020.04.22.01.34.33; Wed, 22 Apr 2020 01:34:56 -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 S1726492AbgDVIcr (ORCPT + 99 others); Wed, 22 Apr 2020 04:32:47 -0400 Received: from mail-lj1-f194.google.com ([209.85.208.194]:33409 "EHLO mail-lj1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725968AbgDVIcq (ORCPT ); Wed, 22 Apr 2020 04:32:46 -0400 Received: by mail-lj1-f194.google.com with SMTP id w20so1381926ljj.0; Wed, 22 Apr 2020 01:32:44 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:autocrypt :message-id:date:user-agent:mime-version:in-reply-to :content-language:content-transfer-encoding; bh=agTMX2MselmdZuLLyKamBPc/VIJvqLXA70Pup6Zmgv4=; b=JsEpIBrpb0259kKPxYBORbtfEYLsVv6fvU6BMdXbp+W98qBiZJvU3oVgPHwit17X2z OCTjDRCZ1vdYLY/9/ZIUWqhznbwFTCRWXHuZ6sJf8HwMCgkpqz1+fr+Zm1gLppUAZfD+ 2qhrrBl2nt+rHgLPewC81ZWAz/peZnJS/vEypoYbSHaz9taK3jh+qaRDy7Hc2q7075EL e0uE5Z5PdkiIO6M5RRHPzm4R4qyc/FY7FAThD2HqGDat89Pi61dBEOAb4ct8mD6SUXHP a6Z7wSJBoFrv9L62qU3bJ3MOBcvBwIGk554gn+g/o2dgmpCMtl1t3Jooo9ykukih8kTV /34g== X-Gm-Message-State: AGi0PuY4l6Is1vX5zZEveoIrV1dbchVMS8YjeqhUM+Kntd6a8oByiWZr d9XRb8G0vTe9aiYTtJyTTNiHkqGVkYI= X-Received: by 2002:a2e:860a:: with SMTP id a10mr16374779lji.20.1587544363222; Wed, 22 Apr 2020 01:32:43 -0700 (PDT) Received: from [192.168.8.103] ([213.87.162.215]) by smtp.gmail.com with ESMTPSA id d15sm3910891lfl.77.2020.04.22.01.32.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 22 Apr 2020 01:32:42 -0700 (PDT) Subject: Re: [PATCH 3/3] floppy: suppress UBSAN warning in setup_rw_floppy() To: Christoph Hellwig Cc: Willy Tarreau , linux-block@vger.kernel.org, linux-kernel@vger.kernel.org References: <20200421125722.58959-1-efremov@linux.com> <20200421125722.58959-4-efremov@linux.com> <20200422070921.GA19116@infradead.org> <20200422071756.GA16814@1wt.eu> <20200422082400.GA30239@infradead.org> From: Denis Efremov Autocrypt: addr=efremov@linux.com; keydata= mQINBFsJUXwBEADDnzbOGE/X5ZdHqpK/kNmR7AY39b/rR+2Wm/VbQHV+jpGk8ZL07iOWnVe1 ZInSp3Ze+scB4ZK+y48z0YDvKUU3L85Nb31UASB2bgWIV+8tmW4kV8a2PosqIc4wp4/Qa2A/ Ip6q+bWurxOOjyJkfzt51p6Th4FTUsuoxINKRMjHrs/0y5oEc7Wt/1qk2ljmnSocg3fMxo8+ y6IxmXt5tYvt+FfBqx/1XwXuOSd0WOku+/jscYmBPwyrLdk/pMSnnld6a2Fp1zxWIKz+4VJm QEIlCTe5SO3h5sozpXeWS916VwwCuf8oov6706yC4MlmAqsQpBdoihQEA7zgh+pk10sCvviX FYM4gIcoMkKRex/NSqmeh3VmvQunEv6P+hNMKnIlZ2eJGQpz/ezwqNtV/przO95FSMOQxvQY 11TbyNxudW4FBx6K3fzKjw5dY2PrAUGfHbpI3wtVUNxSjcE6iaJHWUA+8R6FLnTXyEObRzTS fAjfiqcta+iLPdGGkYtmW1muy/v0juldH9uLfD9OfYODsWia2Ve79RB9cHSgRv4nZcGhQmP2 wFpLqskh+qlibhAAqT3RQLRsGabiTjzUkdzO1gaNlwufwqMXjZNkLYu1KpTNUegx3MNEi2p9 CmmDxWMBSMFofgrcy8PJ0jUnn9vWmtn3gz10FgTgqC7B3UvARQARAQABtCFEZW5pcyBFZnJl bW92IDxlZnJlbW92QGxpbnV4LmNvbT6JAlcEEwEIAEECGwMFCQPCZwAFCwkIBwIGFQoJCAsC BBYCAwECHgECF4AWIQR2VAM2ApQN8ZIP5AO1IpWwM1AwHwUCW3qdrQIZAQAKCRC1IpWwM1Aw HwF5D/sHp+jswevGj304qvG4vNnbZDr1H8VYlsDUt+Eygwdg9eAVSVZ8yr9CAu9xONr4Ilr1 I1vZRCutdGl5sneXr3JBOJRoyH145ExDzQtHDjqJdoRHyI/QTY2l2YPqH/QY1hsLJr/GKuRi oqUJQoHhdvz/NitR4DciKl5HTQPbDYOpVfl46i0CNvDUsWX7GjMwFwLD77E+wfSeOyXpFc2b tlC9sVUKtkug1nAONEnP41BKZwJ/2D6z5bdVeLfykOAmHoqWitCiXgRPUg4Vzc/ysgK+uKQ8 /S1RuUA83KnXp7z2JNJ6FEcivsbTZd7Ix6XZb9CwnuwiKDzNjffv5dmiM+m5RaUmLVVNgVCW wKQYeTVAspfdwJ5j2gICY+UshALCfRVBWlnGH7iZOfmiErnwcDL0hLEDlajvrnzWPM9953i6 fF3+nr7Lol/behhdY8QdLLErckZBzh+tr0RMl5XKNoB/kEQZPUHK25b140NTSeuYGVxAZg3g 4hobxbOGkzOtnA9gZVjEWxteLNuQ6rmxrvrQDTcLTLEjlTQvQ0uVK4ZeDxWxpECaU7T67khA ja2B8VusTTbvxlNYbLpGxYQmMFIUF5WBfc76ipedPYKJ+itCfZGeNWxjOzEld4/v2BTS0o02 0iMx7FeQdG0fSzgoIVUFj6durkgch+N5P1G9oU+H37kCDQRbCVF8ARAA3ITFo8OvvzQJT2cY nPR718Npm+UL6uckm0Jr0IAFdstRZ3ZLW/R9e24nfF3A8Qga3VxJdhdEOzZKBbl1nadZ9kKU nq87te0eBJu+EbcuMv6+njT4CBdwCzJnBZ7ApFpvM8CxIUyFAvaz4EZZxkfEpxaPAivR1Sa2 2x7OMWH/78laB6KsPgwxV7fir45VjQEyJZ5ac5ydG9xndFmb76upD7HhV7fnygwf/uIPOzNZ YVElGVnqTBqisFRWg9w3Bqvqb/W6prJsoh7F0/THzCzp6PwbAnXDedN388RIuHtXJ+wTsPA0 oL0H4jQ+4XuAWvghD/+RXJI5wcsAHx7QkDcbTddrhhGdGcd06qbXe2hNVgdCtaoAgpCEetW8 /a8H+lEBBD4/iD2La39sfE+dt100cKgUP9MukDvOF2fT6GimdQ8TeEd1+RjYyG9SEJpVIxj6 H3CyGjFwtIwodfediU/ygmYfKXJIDmVpVQi598apSoWYT/ltv+NXTALjyNIVvh5cLRz8YxoF sFI2VpZ5PMrr1qo+DB1AbH00b0l2W7HGetSH8gcgpc7q3kCObmDSa3aTGTkawNHzbceEJrL6 mRD6GbjU4GPD06/dTRIhQatKgE4ekv5wnxBK6v9CVKViqpn7vIxiTI9/VtTKndzdnKE6C72+ jTwSYVa1vMxJABtOSg8AEQEAAYkCPAQYAQgAJhYhBHZUAzYClA3xkg/kA7UilbAzUDAfBQJb CVF8AhsMBQkDwmcAAAoJELUilbAzUDAfB8cQALnqSjpnPtFiWGfxPeq4nkfCN8QEAjb0Rg+a 3fy1LiquAn003DyC92qphcGkCLN75YcaGlp33M/HrjrK1cttr7biJelb5FncRSUZqbbm0Ymj U4AKyfNrYaPz7vHJuijRNUZR2mntwiKotgLV95yL0dPyZxvOPPnbjF0cCtHfdKhXIt7Syzjb M8k2fmSF0FM+89/hP11aRrs6+qMHSd/s3N3j0hR2Uxsski8q6x+LxU1aHS0FFkSl0m8SiazA Gd1zy4pXC2HhCHstF24Nu5iVLPRwlxFS/+o3nB1ZWTwu8I6s2ZF5TAgBfEONV5MIYH3fOb5+ r/HYPye7puSmQ2LCXy7X5IIsnAoxSrcFYq9nGfHNcXhm5x6WjYC0Kz8l4lfwWo8PIpZ8x57v gTH1PI5R4WdRQijLxLCW/AaiuoEYuOLAoW481XtZb0GRRe+Tm9z/fCbkEveyPiDK7oZahBM7 QdWEEV8mqJoOZ3xxqMlJrxKM9SDF+auB4zWGz5jGzCDAx/0qMUrVn2+v8i4oEKW6IUdV7axW Nk9a+EF5JSTbfv0JBYeSHK3WRklSYLdsMRhaCKhSbwo8Xgn/m6a92fKd3NnObvRe76iIEMSw 60iagNE6AFFzuF/GvoIHb2oDUIX4z+/D0TBWH9ADNptmuE+LZnlPUAAEzRgUFtlN5LtJP8ph Message-ID: Date: Wed, 22 Apr 2020 11:32:40 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <20200422082400.GA30239@infradead.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/22/20 11:24 AM, Christoph Hellwig wrote: > > I don't think we can just change FD_RAW_CMD_SIZE or cmd as that could > break userspace. The second patch adds these defines FD_RAW_CMD_SIZE, FD_RAW_REPLY_SIZE. Currently they are not in the uapi. Ok, I will send v2. But otherwise, yes something very much like that: > >> #define FD_RAW_CMD_SIZE 16 >> #define FD_RAW_REPLY_SIZE 16 >> +#define FD_RAW_FULL_CMD_SIZE (FD_RAW_CMD_SIZE + 1 + FD_RAW_CMD_SIZE) >> >> unsigned char cmd_count; >> - unsigned char cmd[FD_RAW_CMD_SIZE]; >> - unsigned char reply_count; >> - unsigned char reply[FD_RAW_REPLY_SIZE]; >> + union { >> + struct { >> + unsigned char cmd[FD_RAW_CMD_SIZE]; >> + unsigned char reply_count; >> + unsigned char reply[FD_RAW_REPLY_SIZE]; >> + }; >> + unsigned char full_cmd[FD_RAW_FULL_CMD_SIZE]; >> + }; > >> int track; >> >> Denis > ---end quoted text--- >