Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp736978ybt; Wed, 24 Jun 2020 09:58:56 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxlq6U9fDHWYmfuA+39njQGaMPFx4cx1CsNxN01vdcjAIJhSo1Zi1KztDo8Ph+7P5tg29eK X-Received: by 2002:a05:6402:1486:: with SMTP id e6mr26849779edv.99.1593017936636; Wed, 24 Jun 2020 09:58:56 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1593017936; cv=pass; d=google.com; s=arc-20160816; b=dq21VcpCApK3QxBCoPj9awSYTmQ4DziQBS84ND2ufnuuAqHloVlIf/hum7F+hxsSgn zooKhMgXT9uKFMtMHopPwUMDD/NFynPCFMmga50oNgdZZYdstUnHCM2OSnn4QNP5cYQ9 PkUI5lniIX17NKeH+jQ9RuSR3y1J+T3JiRbNL2bKk2gnbheC9Znk20Xf2I4mo91pryph ERT55keUDHMRsb/6Nfy6bA0+3fL6twhBcSa2KwZq031Cs+o4bo/cNCewc4KlXPOUj5J5 BNxf6lS7T93bYQII+rkfE02y8SliSu/0hVp0Rae9bHZB1ZU+qSgIWTgmTJoumSdblsSE oQfA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :content-id:wdcipoutbound:content-language:accept-language :in-reply-to:references:message-id:date:thread-index:thread-topic :subject:cc:to:from:dkim-signature:ironport-sdr:dkim-signature; bh=vbOnvdKZdwqF+lV5ROUvnpHaozPwcjbKIv9muW+Hti0=; b=hdMy3AR4aVJ+IqdQMqrYlSAWEjfpCyMU0Q1JHGoXZMGEm3s16K22IC8uqWoBoeE9ts I8peFPvfMhGir1US9VePTMUaoEyqFH+5nrEKbWdLdCElmupI8c1fwpBgpdzH5+yq/Nx4 vvtgU+hOJKmG9Sz9Qw/t0+rVcfL36fURYroEi/yfuIZahCEUYFYqsRfUDhflT627ZNsu M472MiebkGWs/Je2918DhRPGoB/ln9q1vMTT6KP8lcZn2aG+jQ+HlDWMDx1Xh2EcbDbG WcgFkU3odrUk+2sXvDO3UIe1bZG0TlofZPmdyYurbrYasSa3pTbMvh9fbkdgHfukw/S7 0xZA== ARC-Authentication-Results: i=2; mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b="L/avEqAv"; dkim=pass header.i=@sharedspace.onmicrosoft.com header.s=selector2-sharedspace-onmicrosoft-com header.b=EshlFUKt; arc=pass (i=1 spf=pass spfdomain=wdc.com dkim=pass dkdomain=wdc.com dmarc=pass fromdomain=wdc.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=wdc.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id y93si15420516edy.109.2020.06.24.09.58.32; Wed, 24 Jun 2020 09:58: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; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b="L/avEqAv"; dkim=pass header.i=@sharedspace.onmicrosoft.com header.s=selector2-sharedspace-onmicrosoft-com header.b=EshlFUKt; arc=pass (i=1 spf=pass spfdomain=wdc.com dkim=pass dkdomain=wdc.com dmarc=pass fromdomain=wdc.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=wdc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2405291AbgFXQ5w (ORCPT + 99 others); Wed, 24 Jun 2020 12:57:52 -0400 Received: from esa6.hgst.iphmx.com ([216.71.154.45]:18612 "EHLO esa6.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404919AbgFXQ5v (ORCPT ); Wed, 24 Jun 2020 12:57:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1593017871; x=1624553871; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=vbOnvdKZdwqF+lV5ROUvnpHaozPwcjbKIv9muW+Hti0=; b=L/avEqAv+hIHiO0/7Bldjl1TNTvIsXBCta+TuNaqXsC/S+KWbmjJG/Fx ZwQbmKHY8Qs7OUPjpJxS3g9hEInMqJBBCQ3ru7k9Sh4wbxj4jBvLqEnDF iRM7nlF8odcZsj4zl8zTOnp/aAWtd4FBOZwNCcHJEW5TiN+j2h7/QzSPz 0JZsp37ounw0+xPouzThCbvavr4iezUeT2doXwbh4sreUDUZDgSJakCbK JdJL/Ue3+MouJcvKVCbHzT3ZgvO2hqyVJbGT3+nJIOdaQEo7WM7gGVXDU 3lxcUAcyMVe/fbUn4G+KVnTSKRIRJx2O+7jZR8AinCgLHPXzVVrbGATEp w==; IronPort-SDR: sxXxlg5ZjvmTT+l093OlBga6DC3VgbZDVm3QdvRvx2d02suBlEW9M6BrJkdgrOGr/qUxATrFfy K5J60ZUFK7AXcu4nTJzcVECh/mvpztRaVmnEqJNMgQNb1QuZlKXexqV0y0otJ9f6f4gvxweh8P JiC4vKr2ggf20Mhcdd5j1iLz3c5SmQLshVQLohY2kDM/3GYguxrJL1hX+v3bpTlzeoYsa+ZF2z O89N/1Ky9uBkiickaXj+F/sIG3R5Vg4wc4lkPAm15P9i074IwIK/5SM8QMAYGhRXqSU8WVbtPb 70E= X-IronPort-AV: E=Sophos;i="5.75,276,1589212800"; d="scan'208";a="142189675" Received: from mail-bn7nam10lp2106.outbound.protection.outlook.com (HELO NAM10-BN7-obe.outbound.protection.outlook.com) ([104.47.70.106]) by ob1.hgst.iphmx.com with ESMTP; 25 Jun 2020 00:57:50 +0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=W/roMZneJgxbmSIcwVypCtkbSe32OE0TFWdwRG2JlboBBrjkeuQycTluJqAX6lAZkEDUbaqM1MNhzEC5+65Xczd96FRZwNKSAVpPmOfTxHLdIyzh6+e5ThAcqYykjO72U9O3d7YO00RucR4PumkY56z9gf8gFiVbfnWdVPMoinYZqLvc132PDnL/Hbyq7QHni7zuvu9RPldghD5CcuNMnp+tWKEFXcACSWbq286Z2JoCkqX6yUfoxIhXCRm+LZwqGB0y5+T6f1KYszDwnSEe9o7IYRUVypBBb7w9OK2XQlzqC5wlc/auGGQ0q9aQM4eWTp9rNV6oCiYIVON/zdPiVQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vbOnvdKZdwqF+lV5ROUvnpHaozPwcjbKIv9muW+Hti0=; b=OBxlOwndiGcaK7BvKtWJHN2XfNkvSLkrtoWWkgUBtHyDvX+AEiQvtIGCwVhkYoRCuWn0Yrw7w+92JgQwwVYzuN1zx/mO5iynaYb22R2rDQBL780L5DChNsrg3lQn13vHndANsF1jdDF6Hw5l+iOQXfK7hQlRRMCxpdo4ySdjeNo4lH56X+pN3b4KuRKOYbmgqds0hhmL1pTwwHuCZeJ/9VLPGkmgaKWiCKON3c42tkRnmNSeUQ7y4wvqnO2v932wxp40kBcp8kb1crnnG+OmAv0FMM5PU37I6BobTW5zAa+gUin6YmQlcf9lY0n+PAwUgxeyaAbiyIcyF6wXgie2fQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wdc.com; dmarc=pass action=none header.from=wdc.com; dkim=pass header.d=wdc.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sharedspace.onmicrosoft.com; s=selector2-sharedspace-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vbOnvdKZdwqF+lV5ROUvnpHaozPwcjbKIv9muW+Hti0=; b=EshlFUKt4pSab94BEkO/UhkZKZxKAwX/Xrlt2O8JZ8Gn48feED81DTElO7Ff14PXyuZcIJB380H4XjKBxcOSIl95t4ovQWsuu7/OYRTAwgnswrNIPzyyqamdx+PjFbQiZyXxPI1bxAR7vlBq6MQvegkSgEGif8HZgjruuNjVq/8= Received: from BYAPR04MB5112.namprd04.prod.outlook.com (2603:10b6:a03:45::10) by BYAPR04MB4072.namprd04.prod.outlook.com (2603:10b6:a02:b3::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.21; Wed, 24 Jun 2020 16:57:49 +0000 Received: from BYAPR04MB5112.namprd04.prod.outlook.com ([fe80::a442:4836:baba:c84b]) by BYAPR04MB5112.namprd04.prod.outlook.com ([fe80::a442:4836:baba:c84b%6]) with mapi id 15.20.3131.020; Wed, 24 Jun 2020 16:57:49 +0000 From: Niklas Cassel To: Christoph Hellwig CC: Keith Busch , Jens Axboe , Sagi Grimberg , Chaitanya Kulkarni , "linux-nvme@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2 0/2] remove workarounds for gcc bug wrt unnamed fields in initializers Thread-Topic: [PATCH v2 0/2] remove workarounds for gcc bug wrt unnamed fields in initializers Thread-Index: AQHWRatsK9pqpN/HlEmT0NtnEgioaqjoAiyAgAADqAA= Date: Wed, 24 Jun 2020 16:57:48 +0000 Message-ID: <20200624165746.GA394355@localhost.localdomain> References: <20200618200235.1104587-1-niklas.cassel@wdc.com> <20200624164441.GA24816@lst.de> In-Reply-To: <20200624164441.GA24816@lst.de> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: lst.de; dkim=none (message not signed) header.d=none;lst.de; dmarc=none action=none header.from=wdc.com; x-originating-ip: [85.224.200.150] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 40bf9e77-3c2b-46c7-813f-08d8185fb9c1 x-ms-traffictypediagnostic: BYAPR04MB4072: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: wdcipoutbound: EOP-TRUE x-ms-oob-tlc-oobclassifiers: OLM:8882; x-forefront-prvs: 0444EB1997 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 0Eucfr+w8S8LxklxH1lZbutABYDUXPaKzBGu3SmvobU1UJmiMp1h7RXyqqaYiWJ8skiXPOtWD3pU8CxlqKUiHnI++m5XIC9tSFP13C+sbnE959I6QStlYEmP5aq68QwUCCrMFv++lEBCUdypLP9ffPpGljmpwCCns67sljCA6dqSnHkSrNWQvJHqDxZkI+7RNmiGtDdafPnQkxvDHuf69k7CrQeAMJrWOLzjmzyryEIJaQazS1x9FU8lBARdNFbJJFBO7xHXCnwppeZjOh2zLMpAri5L1qK1dBAwji/9aSpZ8J+vH5d/eeaBJIasQSB8lxVufGsqWt1MypKa4azY5w== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR04MB5112.namprd04.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(346002)(396003)(366004)(39860400002)(136003)(376002)(66446008)(66476007)(66556008)(64756008)(66946007)(4744005)(86362001)(5660300002)(2906002)(71200400001)(91956017)(1076003)(54906003)(76116006)(4326008)(316002)(26005)(6506007)(186003)(6486002)(8676002)(8936002)(6512007)(6916009)(478600001)(33656002)(9686003);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: f7a39Ge5AfDlTP7mdkEurh2xFT1zZOJrJNfd8r342Cg8891fwmWlvqlK14F1UMFa/smnPpKHr5C6SeXkchHJ/0h2DwW3n7SvlOp0AmIPaNlJlIDJdwoY5GHOO/3kzPmnevRKAlRtPya0vBaKXIXOJDGt9bJqiX8a8cBxfXcsvpic7IjfSQ8ttcGi+mkPkO6WAH42gE29RdzYdxK5EGke6ZuIwm0GTLhiS+R6hoA3kzg0YvFqzUUYiBAKpbmH7T5h/gQGPr8n+oExHrr9Cu0OqT4nqpyUxR8dH4pq809gLbhIxFhwNRavth4jTjgGktF72ft6pl03jUx6vHLhZDMdcPnNwuzqqp1IDFUKycCV61EeBzku8WGq695E90UJ4OSHVyBpD4KKfbzvE9AlJ7Y2mZDJ2p1lALudeCxp51xaJE5VpjwW5rW7OiwGa3bIHnDC7PGrOiiX1fQeaj3Llyff+DmTNSuTJtQ7tIbrtl4ahYq7cXi72hO7cvm4blkSQZ0o Content-Type: text/plain; charset="us-ascii" Content-ID: <5C75A69D9D59AE4EA4D39151D77F3FAD@namprd04.prod.outlook.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: wdc.com X-MS-Exchange-CrossTenant-Network-Message-Id: 40bf9e77-3c2b-46c7-813f-08d8185fb9c1 X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Jun 2020 16:57:48.8272 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: iMF1qhQ0zlr66JpKXOVLpPeje8LWVff5fz80Kw0ci4QBCiC1fTB0MCVQ00+n+y88tgd07S6VGhG8/Wvg/zji3g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR04MB4072 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 24, 2020 at 06:44:41PM +0200, Christoph Hellwig wrote: > This looks good to me, but I'd rather wait a few releases to > avoid too mush backporting pain. Chaitanya made me realize that about half of the nvme functions are using "struct nvme_command c" on the stack, and then memsets it, and half of the nvme functions are using an initializer. IMHO, using an initializer is more clear. memset has to be used if the function needs to reset an existing struct, but in none of the functions that I've seen, are we given an existing nvme_command that we need to reset. All the functions that I've seen declares a new nvme_command on the stack (so an initializer makes more sense). What do you think about me unifying this later on? Kind regards, Niklas=