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;
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
To:     =?UTF-8?q?Kai=20M=C3=A4kisara?= <Kai.Makisara@kolumbus.fi>,
        Nathan Chancellor <nathan@kernel.org>,
        "James E.J. Bottomley" <jejb@linux.ibm.com>,
        "Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        linux-scsi@vger.kernel.org, clang-built-linux@googlegroups.com,
        linux-kernel@vger.kernel.org,
        Nick Desaulniers <ndesaulniers@google.com>
Subject: Re: [PATCH] scsi: st: Add missing break in switch statement in st_ioctl()
Date:   Mon, 13 Sep 2021 23:43:57 -0400
Message-Id: <163159094716.20733.10807423364906570646.b4-ty@oracle.com>
In-Reply-To: <20210817235531.172995-1-nathan@kernel.org>
References: <20210817235531.172995-1-nathan@kernel.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Ynpud0IzU2JCYnZNSXFHNldOaDdhcHg1b0xieEZmZVJmb3NnWCtVUlBDRjJG?=
 =?utf-8?B?MUlqaDJzWTFMVlo3WDBBdmdwOHN3NU9hMjZXcW8vbkw3UmhMVTlZRjZRTmRS?=
 =?utf-8?B?cjJ3d3FLS2dPaW12NmtraFd5eTRTVktjQ2tRVkFLUXhLZjh3WmlHdU5rd082?=
 =?utf-8?B?eXBROG5JVWppNWFJalRDejVMN1FheHo1RmNkYUpMN3RjUDJjYnB6R3pTWDBJ?=
 =?utf-8?B?bGUrODFXcGRySE5ROEVrY1NRZS9MYUN2bVZ6NGY0M1dUQUZ5dnQyUmxiMlIy?=
 =?utf-8?B?VFZlVVBLbjZJS1lab1c3ZnVxMnE5Zk1ybS8zMy96UmtYc0c3UG50bThUNEpE?=
 =?utf-8?B?ZnRnZjhoQzh1ZllKMnY5cDM4ZU1RbDg0ZUFwV0cxSUpyeStEWEdNb2Q0c2pD?=
 =?utf-8?B?TXA1NGZpUHMxNVhkK0hxblZvSXdCNEM0OU9VMmYvS29RUmtWUnFWSVRFZTF2?=
 =?utf-8?B?dzZ1STZRVlU0cnVBc29OYTdkcUxFWGZPdFFpUTVBSURHaCsxOUFMRlNvYjUr?=
 =?utf-8?B?VWxaeStJOE9kVUJpYkdCcllNZmlpd1k3cVZBMlRrbFBFMWZFOFRDeVhSck02?=
 =?utf-8?B?MllxWnphSUdLMytZTERpUWdsQVpDalZJbjlOLzNPKy9kOEl5YXJHZHBrVDEz?=
 =?utf-8?B?VVk3bUx0MTZRcjltVDArajJiTjEzSkRiSThLOVZNYXJCN01ZeG5uRUZkekVa?=
 =?utf-8?B?ekdITnl4dDVJc2lFYll4RTQwTUJjUWNpQzVGdGRwaWtkRG04MWg2RHpmUE1i?=
 =?utf-8?B?OU1jYmtwZitrc3hlcUxKL1FBYVpiVTNzODFqbndvcnpGQnVQbjNOdCt5SG96?=
 =?utf-8?B?TVlhby94ZU9SelEyUGxNZXRXNVI2RTk1NmV5T0k2a3VBZXBjQTZIcFJvOGI4?=
 =?utf-8?B?Wmk1bnB0ZFZ1MHBqT3RRcldsNE9KdzQxa0pwQW1CVGR1bnlQUDNzNDRxaGE4?=
 =?utf-8?B?TUN1WjhsV3ZVZkZzMFllMHpycUdqRnlUSG9TTVVOUHlnQml6UzcwZjhZYTYr?=
 =?utf-8?B?SDJ0OC85UjhTbjhJb3RRUUdRZUUrUFp0K3ZmeVdXTkZobVFvV2NuMXhiZGFP?=
 =?utf-8?B?NlpyOCs3QS9rRzJ1dk1wcm1aNjZrRk5kRC9BOEtwUHZ0ZlJidDZaVG1GdWpZ?=
 =?utf-8?B?cit4YS9DY3lBaHdZdmFDUm9jUmtzbytSSXd3ekkxK1hZNE96SDZuQWppNWZB?=
 =?utf-8?B?eTZZOXZlZTR0NkhodGh4akE3SnBYbEFHRXdlb3lqUitXaGYvVUF1eC8ySXRo?=
 =?utf-8?B?OEtIZnhxN1RkaXBhdWNwZFNUV2RyQVEyUUN2NW05dWkvYmJoRjhoTXphM084?=
 =?utf-8?B?SFNkZnVLQitzdzZPK3grVGdPdExFVUs5MGRFREp0Z3Y3TGkrd0RsclcrZHlB?=
 =?utf-8?B?OVR6QlVnWk9ab1ZBL2lUNFVuTXdTaGdZK09HbURaM0tyRk9La3hlaDlOTysv?=
 =?utf-8?B?ays1N29uSG1ORktaS2JablB1a2taYkFlUUdmNzRmc2NYeEZwQitnVXF6Rktx?=
 =?utf-8?B?d0lhTDFsWXo5TjFsZkJwbkRkaEtOR1pOZ2JZQk9XNUttL2toUkZmYXV4d1My?=
 =?utf-8?B?MVRQa29Nam5rS00vOVJZWjdEKzJFSE12LzIrMldCR2pMcHNJZlZyT1QzQWxM?=
 =?utf-8?B?Sm90azl0ci9PZGJuNnFyQlMySEo2SzRXUE52amJGZWs5WDBncDFuNWNKUkJN?=
 =?utf-8?B?dzZhblNFZGtqb2FqSWtEbnRUTmswYkp5VFZkWndGODd1aENQTmZpREZleWpm?=
 =?utf-8?Q?9wpf6G7/j0xdwt6+Ma1pI8QJlf+DBjqCg4K/WC6?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e113acff-4ce6-443d-4d90-08d97731eefa
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Sep 2021 03:44:19.7912
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: VyHi6ChMCMUjc4SLig43+thkW5hxjPYYfecIXx73Rv8kpBL17MbwV93MwJjJT78uwpJOo9fchxrh21C8QdS6r1t3STE2GjwIfdmKyIFC7Q0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB4725
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10106 signatures=668682
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 adultscore=0 phishscore=0
 mlxlogscore=985 suspectscore=0 spamscore=0 bulkscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2109030001
 definitions=main-2109140019
X-Proofpoint-ORIG-GUID: -48xFlA6q05I8FFmgAX01dkP0RCEnyNG
X-Proofpoint-GUID: -48xFlA6q05I8FFmgAX01dkP0RCEnyNG
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, 17 Aug 2021 16:55:31 -0700, Nathan Chancellor wrote:

> Clang + -Wimplicit-fallthrough warns:
> 
> drivers/scsi/st.c:3831:2: warning: unannotated fall-through between
> switch labels [-Wimplicit-fallthrough]
>         default:
>         ^
> drivers/scsi/st.c:3831:2: note: insert 'break;' to avoid fall-through
>         default:
>         ^
>         break;
> 1 warning generated.
> 
> [...]

Applied to 5.15/scsi-fixes, thanks!

[1/1] scsi: st: Add missing break in switch statement in st_ioctl()
      https://git.kernel.org/mkp/scsi/c/6a2ea0d34af1

-- 
Martin K. Petersen	Oracle Linux Engineering