Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp5110853rdb; Sat, 30 Dec 2023 06:25:37 -0800 (PST) X-Google-Smtp-Source: AGHT+IEBiHUlD2HiOxEh4ANsf8CIux5JHNqxODLydsdkgEt2rQVEmLvPGAl9H+xuZXj+dVmaxJHh X-Received: by 2002:a17:907:710f:b0:a27:9365:ef68 with SMTP id zr15-20020a170907710f00b00a279365ef68mr342558ejb.127.1703946337224; Sat, 30 Dec 2023 06:25:37 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703946337; cv=none; d=google.com; s=arc-20160816; b=znHXREvMl3gHWerRpQ9RbEKT3KDJ9kWMGD8bxFAIsZqMovTjkmr5an0a6qfR71loK7 JOhuevaA81ULNJcYmyivGzdLoPKe0uxQRAivW0bwsf2zGlU4NWWNqdTDoDY96PBOZFHM 7sKRD/VEQ8Dub+QmINd8YSy3qcFlew87bu67vNdt1mRG/ogJY9XNqT5ApPqCcPnWciR4 JULvgOw1eb0GxDurSLiPhiF6OdRHXIXzEQPRseAFYMcptA4B6N6OVIpQ15K2Uy9HrF/L /5TBCbePn3w29s/R2qsE+Nzl8JHa4YJ1kS1qYhZetaDkFms2sD161ugTY3x/ZNqeSWvC q9Hg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=gLoEdxPxVcrk2HhDtU3I82ELAUxLV/CTHNEShmI5eAM=; fh=yt//x1K0AElDRB7CQejL+12OXOUb3bqF/gF4Mhj/YP4=; b=vYKDTjuB4b1PiuWzNbLGOJMetBHgBSUaGMWumep/2npJhVtedLbYUeRka/Zl1kjSL2 vbs6LV6aaon+IMP1fyyMXEuOxZ9G/oDggTRYQDd/Q3T6ap7ASpMw5MJEb0M5OKWSq/8P RwHgze/Fd2FNMCF30rpprNASWUD5geQxXSdiuhAtQzvCRPFp2hax5x+p1mI7DepVEeTl dZd9s7V2WrM0RQFy8pWJsIwzELRTFvdi90n/Bd0iOoH0qsupVgcCgbbCGoaP9Vc+w2Zq yAZQF0oMyP9Zq2wIBjzqvpkbM0/iI60nkgQGPcmIs+l+ER1usLIxxrvXKACjkLC3S76k UQ2Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=JrZgkkLc; spf=pass (google.com: domain of linux-kernel+bounces-13451-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13451-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id an2-20020a17090656c200b00a26882a7d69si8433555ejc.932.2023.12.30.06.25.37 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 30 Dec 2023 06:25:37 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-13451-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=JrZgkkLc; spf=pass (google.com: domain of linux-kernel+bounces-13451-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13451-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id F307C1F21B15 for ; Sat, 30 Dec 2023 14:25:36 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 12EFA9465; Sat, 30 Dec 2023 14:25:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="JrZgkkLc" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.web.de (mout.web.de [212.227.15.3]) (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 B70F08C17; Sat, 30 Dec 2023 14:25:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1703946312; x=1704551112; i=markus.elfring@web.de; bh=dI9cCClD6JexG1woOmdgPnUw06082v8DX+zQoFVxay0=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From: In-Reply-To; b=JrZgkkLcSZEPJI2sUZwLyvAFvdiwnzUUMNrTO+464QkuRNRbHs0x0k+4okj/iV3n RleY5H0MRaRdI6RTzbBQHRRvq+7MvBUeqsOSc3NJXUEJfGPYNPcyuRMb2WY6DScgH XZ8+l1Dry/BirwDYtwVnvLrn/cwZ4DPbhnL7YUV/pjpOIJ6EPuUbFu6vBWEHBvPTI FIwl0YQOSBl6WkizinXiUky2nwsom+ojXYDpACegt7/BzGmx5+IcyrJbh33PV29t7 FSYRJmASTws7Fljq7ADYjU6fgk8hFPRl/HYKXoZSyvi1aRYiAhzVTKhNQZscmWSGk OKiHxIUdn5h86cX3UA== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.87.95]) by smtp.web.de (mrweb005 [213.165.67.108]) with ESMTPSA (Nemesis) id 1N9cLR-1r7xFt1Rhd-0159S3; Sat, 30 Dec 2023 15:25:12 +0100 Message-ID: <38dfe7b6-6d8b-4056-9943-12197c80f4d7@web.de> Date: Sat, 30 Dec 2023 15:25:11 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: scsi: ses: Move a label in ses_enclosure_data_process() Content-Language: en-GB To: James Bottomley , linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org, "Martin K. Petersen" Cc: LKML References: <4616e325-e313-4078-9788-dd1e6e51b9e0@web.de> <9d24844f30604f969ac10da456801f594ce72f2d.camel@linux.ibm.com> <4018ab9225ecaf18501e54114a94217a58a8a57f.camel@linux.ibm.com> From: Markus Elfring In-Reply-To: <4018ab9225ecaf18501e54114a94217a58a8a57f.camel@linux.ibm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:GNJw0eiTnvRz1Z7tZBuB+o4pb2XmsXkWWTZiR+2n3Tvnb6BFqlp pAi4/57LDG+7T+iqF45/wO7HVSzok2K91AmYGWbueThOhNsTbMdd/XS1h4jVX+bIJiSfBee y+TE/KR2g1a0mx/hHl1Eng1SxE4DnQBlK2d9/1y9QbHRIag5zCKiw3Jf7vcfHBza0ZeL3Ot NOU7r/Ul+GWy3MtQJA0Zw== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:Dv4K7cuIcss=;f9YSP876OXo4u8npqQbrBWqzcnR Aa5pg1EAca/6Vvw6iVYzRo07mMqjCV5x3Bx9SCn/KZqfb39Kszo7sk4UtLK8hjOciIl72hIje HJirzLhbASjwNDjaY64AzE37v7kGcO/p72Rs3YkY1+6Jswx4s+zg4XjNLLBIt0jQ+NC/6qlwz fq5WwDbZY3uaRC/EkUETlx+emjPRzeFrUvi2wRXM/jVF+merCQIpchW4IRHQO7PSRlKfDfFdC QGqJVvkHUITnTjaFlT3E8ugMaWh1moO+Zjl3JSe/nHfEXt7GHCUk3IhwIneT/eUbPmaVOzKil v7A+X0ArDLhikEGDWBq8IAFf20mV8D7S2T5wlZKX8sOioLL1JW2ZzM8EKkLvgy//X25PoERwX eBMQYEOAX/46J938bbBNs/hYkl3WvK8ZGNEB6BGPsm4sZ9EVKOG1dWtYbWMpBHwLNc855f7K1 VZPx/qXMhXmuzqy9NO2zbcyF8upj0pFsMO2DEjGsvgE8N0nFs46+KfimnfmjA/LwI0l9phEax Ras33D5O/u9iJHjtNsK+sEN6yCoy8O645a/T2tkfSe09zyuymbg0cysAZzz8+JOg26yt2RZqe 3vyRGw0kZeO66qGVTlEbZBR+sInbuXB96+umk4LiuVHYrOvmS1CIIf6JqgYXRm1La+4m+A40q yUpB+v7fvphXCWVCutulLBcr+MpPSLztGboAF0sqDMK0gE1lMsXTBmWCQDy1BytrNKXyuFF6s WF4FFcn/4aZ40Xi9mOuslFbeUCuKQFm9WLVEIomBlFTWijuBk0acIDudd5ATdfs7+ykUEjlnz ZQ9DQcGKgjMFZX79MySJA2zQ4lMED7uUNtzKWTupva5+14t5+nnae61oQU0LJMZtDQlDyXSoD 2zADWO5ELDdUyeRfCEzvTfGPKReLBwbtNZkQ8Vb+xNkOKSoyoY63VIQyeNGI6AT7QdZkNg2fb 0kNrI4d/cH5rQd99QHOYQYMY9cM= >> If you would dare to follow advice from goto chains in a strict way, >> I imagine that you can tend to stress the attention for more useful >> data processing a bit more than such a redundant function call. > > It's about maintainability and simplicity. Eliminating kfree(NULL) > doesn't simplify most code, I find it easy to avoid such a call in the affected and concrete function implementation. > it just makes the exit paths more complex Where is undesirable software complexity here in the repositioning of the label =E2=80=9Csimple_populate=E2=80=9D before the statement =E2=80= =9Cbuf =3D NULL;=E2=80=9D? Regards, Markus