From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5C9A3559D for ; Mon, 28 Feb 2022 13:15:48 +0000 (UTC) Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9nQ1e021519; Mon, 28 Feb 2022 13:14:21 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efat1v6sk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Cc: Linus Torvalds , linux-arch , Thomas Gleixner , Arnd Bergman , Andy Shevchenko , Andrew Morton , Kees Cook , Mike Rapoport , "Gustavo A. R. Silva" , Brian Johannesmeyer , Cristiano Giuffrida , "Bos, H.J." , Christophe JAILLET , Jason Gunthorpe , Rasmus Villemoes , Nathan Chancellor , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-sgx@vger.kernel.org, drbd-dev@lists.linbit.com, linux-block@vger.kernel.org, linux-iio@vger.kernel.org, linux-crypto@vger.kernel.org, dmaengine@vger.kernel.org, linux1394-devel@lists.sourceforge.net, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, nouveau@lists.freedesktop.org, linux-rdma@vger.kernel.org, linux-media@vger.kernel.org, intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, linux-pm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-staging@lists.linux.dev, linux-usb@vger.kernel.org, linux-aspeed@lists.ozlabs.org, bcm-kernel-feedback-list@broadcom.com, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org, kvm@vger.kernel.org, linux-cifs@vger.kernel.org, samba-technical@lists.samba.org, linux-f2fs-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, kgdb-bugreport@lists.sourceforge.net, v9fs-developer@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, alsa-devel@alsa-project.org Subject: Re: [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR1001MB2365.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Proofpoint-ORIG-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.sourceforge.net (lists.sourceforge.net [216.105.38.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 560A4C433FE for ; Mon, 28 Feb 2022 13:16:11 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-4.v29.lw.sourceforge.com) by sfs-ml-4.v29.lw.sourceforge.com with esmtp (Exim 4.94.2) (envelope-from ) id 1nOfsp-0003Sx-Qx; Mon, 28 Feb 2022 13:16:10 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-4.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nOfso-0003SL-Dr; Mon, 28 Feb 2022 13:16:09 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=MIME-Version:In-Reply-To:Content-Type:References: Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=HHXKbPYcc5ocKx07V1BINX7iG0 5wYFmXx9CwxNVBsu6bpmtIeYNmB6EuknmYFyCBCCWMIjAVLeqJm6e+emvHapzzTdFaZEzQ24t6gX2 2IpqrQHfQ0RqkyctxPGuwJcq+Lc9/yMvm61ZuHzjNrpr/dRu5+i6wVkDhf6jyWo0QXDI=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=MIME-Version:In-Reply-To:Content-Type:References:Message-ID:Subject:Cc:To :From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=IyirsYo5p63QEOjSTXUDo52sA4 LOa4Ac5g0PEsxgqXXIvCfiExTdUzeWdtkshVGHt8VnWs/nn/mjIiSWyUaSI4KDxHmtNgKYaRPNc+a n/g3Lj7qeqqFct1vYXaYo4BJV+o4ufq/lsbNKBkZadLFgr1Q2ejSCu6qouDzBAqO2f/E=; Received: from mx0a-00069f02.pphosted.com ([205.220.165.32]) by sfi-mx-1.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.94.2) id 1nOfsj-0008Qn-Ab; Mon, 28 Feb 2022 13:16:09 +0000 Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9nQ1e021519; Mon, 28 Feb 2022 13:14:21 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efat1v6sk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR1001MB2365.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Proofpoint-ORIG-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Headers-End: 1nOfsj-0008Qn-Ab Subject: Re: [f2fs-dev] [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr X-BeenThere: linux-f2fs-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alsa-devel@alsa-project.org, linux-aspeed@lists.ozlabs.org, "Gustavo A. R. Silva" , linux-iio@vger.kernel.org, nouveau@lists.freedesktop.org, Rasmus Villemoes , dri-devel@lists.freedesktop.org, Cristiano Giuffrida , amd-gfx@lists.freedesktop.org, samba-technical@lists.samba.org, linux1394-devel@lists.sourceforge.net, drbd-dev@lists.linbit.com, linux-arch , linux-cifs@vger.kernel.org, kvm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-rdma@vger.kernel.org, linux-staging@lists.linux.dev, "Bos, H.J." , Jason Gunthorpe , intel-wired-lan@lists.osuosl.org, kgdb-bugreport@lists.sourceforge.net, bcm-kernel-feedback-list@broadcom.com, linux-media@vger.kernel.org, Kees Cook , Arnd Bergman , linux-pm@vger.kernel.org, intel-gfx@lists.freedesktop.org, Brian Johannesmeyer , linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christophe JAILLET , v9fs-developer@lists.sourceforge.net, linux-tegra@vger.kernel.org, Thomas Gleixner , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, linux-sgx@vger.kernel.org, Nathan Chancellor , Linus Torvalds , linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, linux-crypto@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mediatek@lists.infradead.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, Mike Rapoport Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 56852C433EF for ; Wed, 2 Mar 2022 08:44:54 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id A55AD1B44; Wed, 2 Mar 2022 09:44:02 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz A55AD1B44 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1646210692; bh=d8eOgB48KC/ERDSlbr8fxj7lod3lX74tjO1NN1L9JLw=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=d8HbOUegfa87rfzlj839oXgCJB9+x2X2LKd/57qxlHZLzn/fOby/vi9fPpoRGJcXC G04Uacp0Yg3u7lUr/EtATcVFTk/F0+jC3Qt8KrybPng9Jj8X8l7HmaWUFMwwpBAPu1 XXIraa3s1g5/kU+X0iJbC6WG0f8f9JKTGcPdjfpk= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 855B1F80602; Wed, 2 Mar 2022 09:34:01 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 716DDF8013C; Mon, 28 Feb 2022 14:14:27 +0100 (CET) Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 08659F80054 for ; Mon, 28 Feb 2022 14:14:22 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 08659F80054 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=oracle.com header.i=@oracle.com header.b="qFDmxGdu"; dkim=pass (1024-bit key) header.d=oracle.onmicrosoft.com header.i=@oracle.onmicrosoft.com header.b="Cp7cfYsq" Received: from pps.filterd (m0246617.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9mMMr025471; Mon, 28 Feb 2022 13:14:20 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efcrtc2ej-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Subject: Re: [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR1001MB2365.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: 8ChfBIBWHpSjtFAksRi1OvGFEv78ayCj X-Proofpoint-ORIG-GUID: 8ChfBIBWHpSjtFAksRi1OvGFEv78ayCj X-Mailman-Approved-At: Wed, 02 Mar 2022 09:33:35 +0100 Cc: alsa-devel@alsa-project.org, linux-aspeed@lists.ozlabs.org, "Gustavo A. R. Silva" , linux-iio@vger.kernel.org, nouveau@lists.freedesktop.org, Rasmus Villemoes , dri-devel@lists.freedesktop.org, Cristiano Giuffrida , amd-gfx@lists.freedesktop.org, samba-technical@lists.samba.org, linux1394-devel@lists.sourceforge.net, drbd-dev@lists.linbit.com, linux-arch , linux-cifs@vger.kernel.org, kvm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-rdma@vger.kernel.org, linux-staging@lists.linux.dev, "Bos, H.J." , Jason Gunthorpe , intel-wired-lan@lists.osuosl.org, kgdb-bugreport@lists.sourceforge.net, bcm-kernel-feedback-list@broadcom.com, linux-media@vger.kernel.org, Kees Cook , Arnd Bergman , linux-pm@vger.kernel.org, intel-gfx@lists.freedesktop.org, Brian Johannesmeyer , linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christophe JAILLET , v9fs-developer@lists.sourceforge.net, linux-tegra@vger.kernel.org, Thomas Gleixner , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, linux-sgx@vger.kernel.org, Nathan Chancellor , Linus Torvalds , linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, linux-crypto@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mediatek@lists.infradead.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, Mike Rapoport X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0EEE2C433F5 for ; Mon, 28 Feb 2022 13:15:44 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 362DA10E5DE; Mon, 28 Feb 2022 13:15:44 +0000 (UTC) Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by gabe.freedesktop.org (Postfix) with ESMTPS id 020E110E5DE; Mon, 28 Feb 2022 13:15:42 +0000 (UTC) Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9nQ1e021519; Mon, 28 Feb 2022 13:14:21 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efat1v6sk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR1001MB2365.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Proofpoint-ORIG-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 Subject: Re: [Intel-gfx] [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alsa-devel@alsa-project.org, linux-aspeed@lists.ozlabs.org, "Gustavo A. R. Silva" , linux-iio@vger.kernel.org, nouveau@lists.freedesktop.org, Rasmus Villemoes , dri-devel@lists.freedesktop.org, Cristiano Giuffrida , amd-gfx@lists.freedesktop.org, samba-technical@lists.samba.org, linux1394-devel@lists.sourceforge.net, drbd-dev@lists.linbit.com, linux-arch , linux-cifs@vger.kernel.org, kvm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-rdma@vger.kernel.org, linux-staging@lists.linux.dev, "Bos, H.J." , Jason Gunthorpe , intel-wired-lan@lists.osuosl.org, kgdb-bugreport@lists.sourceforge.net, bcm-kernel-feedback-list@broadcom.com, linux-media@vger.kernel.org, Kees Cook , Arnd Bergman , linux-pm@vger.kernel.org, intel-gfx@lists.freedesktop.org, Brian Johannesmeyer , linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christophe JAILLET , v9fs-developer@lists.sourceforge.net, linux-tegra@vger.kernel.org, Thomas Gleixner , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, linux-sgx@vger.kernel.org, Nathan Chancellor , Linus Torvalds , linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, linux-crypto@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mediatek@lists.infradead.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, Mike Rapoport Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 710EAC433F5 for ; Mon, 28 Feb 2022 13:27:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=jYrDtBnCTVCM38ObQxFwJhoqgZkBrAntf4cbmmX7AXk=; b=4YL1isZm9EhGe2 xOSg7emdTwJ/SAB1ermASlHQTCrSqYWAl7h6+MPVs2pRrM2NdWhRdTgRJvJXsenMURcFuQhBkFQiY 99zdKOWIpaLhbErm0cciCbvgBkEa0tUX9UiHF4kkv+s9m6aQzNFmEV0Fz6xtlmJN0avZqvSeTwkc5 PBZCt0c5vEopGStH1UHg6CTTMg7QoGAo5o+0gQcpYwN9q0DKr6InM1Qw6wRoA+gHgwgBOF3yBuXe7 DZIth22us+8slEafscDx+I93eu5SzNSclrINxL8apw+zUorkrK1DHQUSvEzWtIqOuZCEaGGErfWZd 1cp2EOxlU+hVa15X/kcg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nOg3E-00CYmw-8L; Mon, 28 Feb 2022 13:26:56 +0000 Received: from mx0a-00069f02.pphosted.com ([205.220.165.32]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nOfsN-00CU93-SB; Mon, 28 Feb 2022 13:15:45 +0000 Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9nQ1e021519; Mon, 28 Feb 2022 13:14:21 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efat1v6sk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Cc: Linus Torvalds , linux-arch , Thomas Gleixner , Arnd Bergman , Andy Shevchenko , Andrew Morton , Kees Cook , Mike Rapoport , "Gustavo A. R. Silva" , Brian Johannesmeyer , Cristiano Giuffrida , "Bos, H.J." , Christophe JAILLET , Jason Gunthorpe , Rasmus Villemoes , Nathan Chancellor , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-sgx@vger.kernel.org, drbd-dev@lists.linbit.com, linux-block@vger.kernel.org, linux-iio@vger.kernel.org, linux-crypto@vger.kernel.org, dmaengine@vger.kernel.org, linux1394-devel@lists.sourceforge.net, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, nouveau@lists.freedesktop.org, linux-rdma@vger.kernel.org, linux-media@vger.kernel.org, intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, linux-pm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-staging@lists.linux.dev, linux-usb@vger.kernel.org, linux-aspeed@lists.ozlabs.org, bcm-kernel-feedback-list@broadcom.com, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org, kvm@vger.kernel.org, linux-cifs@vger.kernel.org, samba-technical@lists.samba.org, linux-f2fs-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, kgdb-bugreport@lists.sourceforge.net, v9fs-developer@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, alsa-devel@alsa-project.org Subject: Re: [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR1001MB2365.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Proofpoint-ORIG-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220228_051543_949231_D2D199DB X-CRM114-Status: GOOD ( 22.94 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 72227C433FE for ; Tue, 1 Mar 2022 04:38:13 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 07D0410EB88; Tue, 1 Mar 2022 04:37:28 +0000 (UTC) Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by gabe.freedesktop.org (Postfix) with ESMTPS id 020E110E5DE; Mon, 28 Feb 2022 13:15:42 +0000 (UTC) Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 21S9nQ1e021519; Mon, 28 Feb 2022 13:14:21 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=qFDmxGdu7OTRhERHjL2WdcIJgwvCXSbQ1mrT98DsxHyhGv1awYmzokOXHaRjAZU5zjrF 3yQ0JH6uIA1bKugK+2yDWiaSZH1NrDZLmlpfpD0dZXYCTLg26gIBru0y3kqhAq2D6hbX gj49kjfSFFZuKWWDlMFQ4xadT2szePeeNxIQx1W4BHLIvgWnCba8Zm2rvLsPxYIIXmVW +0HK4ntoMmVbXormcxPLUar1c+RrMDJZLDWjgn5vQ9FLxuQOtdW8z7ict1zOXvoiOqgz gYKzO1AdnV/P/msEBCKdVeKcuCHwIW7qSn3wifmLKNPV+PJKf2gLLyEZ0TjJ8M/bUif9 3g== Received: from aserp3020.oracle.com (aserp3020.oracle.com [141.146.126.70]) by mx0b-00069f02.pphosted.com with ESMTP id 3efat1v6sk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:20 +0000 Received: from pps.filterd (aserp3020.oracle.com [127.0.0.1]) by aserp3020.oracle.com (8.16.1.2/8.16.1.2) with SMTP id 21SCukwt165301; Mon, 28 Feb 2022 13:14:19 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2048.outbound.protection.outlook.com [104.47.66.48]) by aserp3020.oracle.com with ESMTP id 3efc12yf7u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 28 Feb 2022 13:14:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K3TAkGH8TvF4AQ7wlB14nitaOm0IbHcl3x5wfSkHxXEOUqVXtgnJgjmyqfC9Z45MWNsR191iDOVHUkd3g9qqPjG5uePZv3gccwyMYmZ0eGGt+29RTtkCEjmejwkG7r4Gtak0xgvVXrYqXRhf/FEg/FzwhlTXK0JJ5ZsiUVVXeFWNITehXD9hGPH6IWoTKdXT4ZTuwPfYpuJ9O7FOD3CWUiPnmnCCHaFgYGVr0/q9JOkAHasNMpOuOnVX38Q+DA8v/aSJTWYcbFX3EyVLneWIo0GxL0TiDss1OoRxJ7CyQrA/Anv3k728HkM4y7A7vKwaqFMSu9gOcddTRkyCXTVHzw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=SvMdMzu1iYWdUbDUVPV4hYdGS0uAE0CypPM+9kDpTrFOIG4CEsQAhZKICoLhU0IfB7Pfjp2fsDMVWlBLo2Q/9y7t2SLJw8evYqInsQ+vuet0HhM8Jei5o0+k5aW6jNYGj15cSLqpvMaWCoYfhhCGypFFtzkpeS/gmGUAzKKvwilWBh1mPlsJ+r4CPmzdp0pIiki40KMgvEGTgob3qsA2MQyaHRhFq/3amN6eB5X3UWmyfVXkvsPY2LRuGdyOcduPXn4+A6XjqekNyu7HXVwRAmi3TMgledmzJXg4H6SLGOhwu1rStsBRPxz6StRb7eM8yNYd8zdjaFQo19iEfEwiCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NvferwtCX3rywbPFtcFbe7khL3mP+l5hGLqhdm467HU=; b=Cp7cfYsqS5EFSsYrMRFCUTGF/k5MQd/chu0oFBUC+nfEz9R6RGP75pM5dVmkUyc9e8fZvL2/qtlIgtGa+PfG21uVJmh1uoLefU85ORAR0l0Z0DoVLG9INL4ilwNs1UR6bvL5SHBDpEo/wjUHDH8r/Ylb+AfqCFPR93eYEJlk5+U= Received: from MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) by DM5PR10MB1241.namprd10.prod.outlook.com (2603:10b6:4:e::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.21; Mon, 28 Feb 2022 13:14:16 +0000 Received: from MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5]) by MWHPR1001MB2365.namprd10.prod.outlook.com ([fe80::2c3d:92b5:42b3:c1c5%4]) with mapi id 15.20.5017.027; Mon, 28 Feb 2022 13:14:15 +0000 Date: Mon, 28 Feb 2022 16:13:25 +0300 From: Dan Carpenter To: Jakob Koschel Message-ID: <20220228131325.GC2794@kadam> References: <20220228110822.491923-1-jakobkoschel@gmail.com> <20220228110822.491923-3-jakobkoschel@gmail.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220228110822.491923-3-jakobkoschel@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-ClientProxiedBy: CT2P275CA0026.ZAFP275.PROD.OUTLOOK.COM (2603:1086:100:a::14) To MWHPR1001MB2365.namprd10.prod.outlook.com (2603:10b6:301:2d::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-TrafficTypeDiagnostic: DM5PR10MB1241:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HBSAJMpVh1ar/Gd52v8X93Gq1tLyJua5/+JcFan12PFGyF6oeCPh6weqwOOk6ibPtKdytnYd8MXsD05YtM7LE2+j7SVEwPv5vPY2Qqto4qodst5K+0tXxtUe+E9kTYfGet66+q+pqSggWAZ9y+/Y/yI/bMdJWeNwfn0mpOZhAROTFN1K0howfk+AkMdwftCF6dm4ROSSm/lljscAW1Kn590Mnap8xaplF7rJJyaVsCsFGnW9jroNfOo301a+pDISpYWSDJ19Pr6UDwaGCMUvQh1myhtBhMHYHxGej62cISRWq7ZLy7xUenNMk8o7lQeLRDHZPlexLOMUR0ylwz7DdUV8soZlQqmfLlfJk+9ZwRhhs+xFZLrfR/y0VZ6nj2SlTfPlVyrYv+rguhwvLr41UqXiFwEMatMFhVbaANOSWr5Z1mc2Mg+qwBxl4Z1Xm0OB2lm7NN3HYKUXcXAWW+ooYSJUdwxLEvLDuZxNlwx6yTQ0hhS+7Czxd/Wpuu4AIlU2rCq0GylmjxK5LYFMs8V8jPgG13VDsLnyrFfJ8Lv3aTbocFLWkbjMgA08SLbNIElgCVvZmsRPbqLV1g0MTLs2QYLJYkA0Zph3tGYZtRoTKd/VSJzFCRaBQGhxJ7X9Wwio13Ds0IW1yRKNkDhw2LCOtuEX0tfbr1s0aUByq8Wm2r4u35R5pSr1rn9KeTSh5BZCyb8rrD4gz6mj8MIKN1SD/A== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR1001MB2365.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(7916004)(366004)(7416002)(5660300002)(7406005)(7366002)(8936002)(44832011)(4326008)(2906002)(66556008)(66476007)(8676002)(66946007)(33716001)(86362001)(33656002)(38350700002)(38100700002)(316002)(83380400001)(186003)(26005)(1076003)(6486002)(508600001)(54906003)(6916009)(52116002)(6506007)(6512007)(6666004)(9686003); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tnpFmoIVUBY9+A0nwXFCCOGbfHokH58MghExpEJudU3G0jSVueQRPOOPpTLg?= =?us-ascii?Q?hqhUmUSBkBxIL8j5GCoiEq/ADSwOryPkL6EfHFmFpoilF9ypf6r7pr9tEMSH?= =?us-ascii?Q?45K4KB4ObVEkqZFJhVtfZbeDKAIV0s3fzPsijOJhgtWjcNxTRdGusMpIxF0x?= =?us-ascii?Q?M0Pym2HJ27lj5u3Rv81JzOsBJoBVCQaIuN3VDLP39nlY8daip/tre+NSHPB8?= =?us-ascii?Q?f1jmYLqV9oVnZz7z6tKyLS/lbTyY2VYITX/YZAk+vYV09ey7TL6iF8ZstPCR?= =?us-ascii?Q?ORKR/QirszHg78/hQBWfwGMbTYLGhIbFZ2LV6PzsW5XT61e9IshKj+6L7CTv?= =?us-ascii?Q?6YEN50b4Ct27vfYseh7B3lvqtwBsucsoRQgoBsBXlIDT6EVLlXjq+1BilFhe?= =?us-ascii?Q?OjQweuTOT2wEvsbWFt1P9EoDOgMAkIBjNhGbaZ+Szu+CBOxkzbkQquxo3ZB3?= =?us-ascii?Q?GusFihnGcS8wBjkV2wE83KDHPc4M95LMXzPrD0zihmenGXffj6VvX7ELqrR/?= =?us-ascii?Q?g2SY4rigjPrs3OMtbUviUOEbP9Szl7gfrLSreWnLbUL9Kc+zCfrY0pkCk9Oo?= =?us-ascii?Q?9k6qEcjr0D4W3fzUYgRQ66fe9f2mxfS2p2wRsQv/YVoMidD555dcAZ3o7ecY?= =?us-ascii?Q?FiCe8K9mB8DlAkZvkqPk3nIi5J5WpzGU5+llJqA2qK5zRmqZYqmfwV+kZYHn?= =?us-ascii?Q?Xc+gCq8avNiFZwKLdTwquYeVPHeHcc1hrjxG6KZZg0grZ77FNbOZ/3LzYFXQ?= =?us-ascii?Q?8q7BZ7isAmmvXX2dZkD9J7I0+aAEeCCBRk0Xtln93PTSd72x5Vqium7+nYge?= =?us-ascii?Q?8rrRizK0GUpI8PmUjkYHTvNJF0iwk2WPedJMPWXm4N3eukNeNr0xeux5QnMG?= =?us-ascii?Q?XzRwoV2Z2OHIldtg+FK9suGbsOi8zyP8V3SVDkwTpFbHlYCFa9oELwVJcgs8?= =?us-ascii?Q?+BgvAWjE0AG2n7NbVCikt1yCTw4Dj6dqoWc3rtV1Y+6cgg3XPVaSK7Y1apLB?= =?us-ascii?Q?71RPUzKwtCw88+pFkm0ff10dSGtXaBLiubXjUf2P/nK4dGByUogvheFxToLr?= =?us-ascii?Q?pQxlxd9sWfK1LN1Zbsfv7bVh1EwiNzfkYWzR0VlVd8P1Q0EeZSR94ABzr8OQ?= =?us-ascii?Q?KnzEcezXP2Xq4I14KEYTGDkBbZJl5+UeXZDV+vg4xH/nkonCaqi5OFub8fNQ?= =?us-ascii?Q?TciWGo+Kaasw4Y/DW24XNFO+jplg0uwHujeQPR5OfQGq1gx+zPZt8vC0bOJc?= =?us-ascii?Q?tcdkWN1wqMHykaP1PxUeCmd3kxYkTS14z4QO/9j2/DT8KjeSUYVxKUAC9iAI?= =?us-ascii?Q?QWkMA4cmNSrJE2kjxC7Rnm9dm6Kcd19JhdmbbNDaNZSTyXvTjtjvGvd0xmzZ?= =?us-ascii?Q?QnY4gcyRKe9QgDn/ROAirh5052+DdA4fZxF9tRBnCPC/MKUOPLH4Epq4jyqP?= =?us-ascii?Q?CY+I7OTV2eTfki0qJDvO5HzxTSL7xgqNAneCFByktPUcw6+8uGXyt0Ehi2Wp?= =?us-ascii?Q?AB9v5+2ZKQhqGIF3t9iTKyTR2vWFPTHgBWBj4/FFOeerpJvjNy15ZvpXyIiq?= =?us-ascii?Q?7ds8PexWNrAwtUngu1Qs8k/Mr18qZpPHx2z3zW+h9UodZ6y4ORnL+JtZgG0I?= =?us-ascii?Q?WdqjUYdqVWrvAv8x5OMMA9yYc8YlhLgqRP2jwI67pZyNUUVtVaUns0I90ya2?= =?us-ascii?Q?C30LFw=3D=3D?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11330734-de4b-4304-c793-08d9fabc37f7 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 13:14:15.5197 (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: pO9kODICkk7VOql0luIhoT5cwQPYKZqHA2qqEqeASZWIMJ3m183W7XHISYMIjKu951+nXW+qVMXF1CfxUFnSmZ7v3YuMGcEX8mOQpDvyh2Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR10MB1241 X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10271 signatures=684655 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 phishscore=0 bulkscore=0 adultscore=0 spamscore=0 suspectscore=0 mlxlogscore=999 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2202280070 X-Proofpoint-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Proofpoint-ORIG-GUID: s7TGwM5vDCDJ_P1DjNSvTf6pMB3hx2z8 X-Mailman-Approved-At: Tue, 01 Mar 2022 04:37:20 +0000 Subject: Re: [Nouveau] [PATCH 2/6] treewide: remove using list iterator after loop body as a ptr X-BeenThere: nouveau@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Nouveau development list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alsa-devel@alsa-project.org, linux-aspeed@lists.ozlabs.org, "Gustavo A. R. Silva" , linux-iio@vger.kernel.org, nouveau@lists.freedesktop.org, Rasmus Villemoes , dri-devel@lists.freedesktop.org, Cristiano Giuffrida , amd-gfx@lists.freedesktop.org, samba-technical@lists.samba.org, linux1394-devel@lists.sourceforge.net, drbd-dev@lists.linbit.com, linux-arch , linux-cifs@vger.kernel.org, kvm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-rdma@vger.kernel.org, linux-staging@lists.linux.dev, "Bos, H.J." , Jason Gunthorpe , intel-wired-lan@lists.osuosl.org, kgdb-bugreport@lists.sourceforge.net, bcm-kernel-feedback-list@broadcom.com, linux-media@vger.kernel.org, Kees Cook , Arnd Bergman , linux-pm@vger.kernel.org, intel-gfx@lists.freedesktop.org, Brian Johannesmeyer , linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christophe JAILLET , v9fs-developer@lists.sourceforge.net, linux-tegra@vger.kernel.org, Thomas Gleixner , Andy Shevchenko , linux-arm-kernel@lists.infradead.org, linux-sgx@vger.kernel.org, Nathan Chancellor , Linus Torvalds , linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, tipc-discussion@lists.sourceforge.net, linux-crypto@vger.kernel.org, netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-mediatek@lists.infradead.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, Mike Rapoport Errors-To: nouveau-bounces@lists.freedesktop.org Sender: "Nouveau" On Mon, Feb 28, 2022 at 12:08:18PM +0100, Jakob Koschel wrote: > diff --git a/drivers/scsi/scsi_transport_sas.c b/drivers/scsi/scsi_transport_sas.c > index 4ee578b181da..a8cbd90db9d2 100644 > --- a/drivers/scsi/scsi_transport_sas.c > +++ b/drivers/scsi/scsi_transport_sas.c > @@ -1060,26 +1060,29 @@ EXPORT_SYMBOL(sas_port_get_phy); > * connected to a remote device is a port, so ports must be formed on > * all devices with phys if they're connected to anything. > */ > -void sas_port_add_phy(struct sas_port *port, struct sas_phy *phy) > +void sas_port_add_phy(struct sas_port *port, struct sas_phy *_phy) _phy is an unfortunate name. > { > mutex_lock(&port->phy_list_mutex); > - if (unlikely(!list_empty(&phy->port_siblings))) { > + if (unlikely(!list_empty(&_phy->port_siblings))) { > /* make sure we're already on this port */ > + struct sas_phy *phy = NULL; Maybe call this port_phy? > struct sas_phy *tmp; > > list_for_each_entry(tmp, &port->phy_list, port_siblings) > - if (tmp == phy) > + if (tmp == _phy) { > + phy = tmp; > break; > + } > /* If this trips, you added a phy that was already > * part of a different port */ > - if (unlikely(tmp != phy)) { > + if (unlikely(!phy)) { > dev_printk(KERN_ERR, &port->dev, "trying to add phy %s fails: it's already part of another port\n", > - dev_name(&phy->dev)); > + dev_name(&_phy->dev)); > BUG(); > } > } else { > - sas_port_create_link(port, phy); > - list_add_tail(&phy->port_siblings, &port->phy_list); > + sas_port_create_link(port, _phy); > + list_add_tail(&_phy->port_siblings, &port->phy_list); > port->num_phys++; > } > mutex_unlock(&port->phy_list_mutex); regards, dan carpenter