Received: by 2002:ab2:5d18:0:b0:1ef:7a0f:c32d with SMTP id j24csp61634lqk; Sat, 9 Mar 2024 00:12:30 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCV/Nn4FQAgspU5nRnVLcJd9DmnUHy4sHkmeTOlFO2n7C54VG96j+wduhMCcn5geiyvaeSIiVaNDZIXeuS7gM/ZnL4wtd7RgHCKE31QLDw== X-Google-Smtp-Source: AGHT+IH7L2aOjtop5sMoBhwJ9fxfROPbaAUJ735vhRtq28UYBMDO8QhmISMe0pClllbEPbdUa3C3 X-Received: by 2002:a05:6a20:b22a:b0:1a1:800d:8c61 with SMTP id eh42-20020a056a20b22a00b001a1800d8c61mr492993pzb.52.1709971950280; Sat, 09 Mar 2024 00:12:30 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709971950; cv=pass; d=google.com; s=arc-20160816; b=gbcb1M9sBr903DO70M4w3dFD28vSN3dM/cRJWTusvfttO3c40W/uydJI60pGgk3AmS HWhLybTw/Lz3eAbkNfsMOvyVJbPY2c7JuIAzenSbHUVFuuZIz1vaxvnt9z7WEuYIBVes ro4H2RQwQ2IeFEqPtTFsm8h9080Eu8bbBJ3ZYi4HPJUGA9VK6pRS9tcJOGSgH4ItfKKQ zwLXT8nfBePXoQrCePMTEyGzGTNTwuNzQDmAqLDf9h3R7oPjWifg30dA1uIbrMa35a+U KAmEw9DRDk4obFPferdFhRSiASBfGHRYPwm+uRBCVd/VFRLI4sRLJWEQgFSqeYXgdMRL mtQQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:wdcironportexception:ironport-sdr:ironport-sdr:dkim-signature; bh=xyrcLJRx10XaVu1nF27Rv4q5kBbpYZb/RWK0jzI/P4w=; fh=M/KanMlWJ9O2I0hMJPl483UULCHuHOYUPdaxVvdB79E=; b=iHDYa00zSXa+MmS3fs1GZonM4uBYEx85oe+0vWYVz7E97BNsq1xjcfe1zRqNT0g3bc LaIrlgB9rs9ZYSg3Tj00tyQKwkdToTkDXffwOavC4gAp6xW3zMCC/HFZx6mo8EcCXRdz hMqd52tuDFn927jE57/75dLcpOTicNJDGiymQ5dZYoaCpd7kS7wdJvRX85GaaTDOZFEt rzMW3NCpoSQcC0k4u2TRgz1Ay6L8BR89O8HMIonr/uEsDDPa3EDBCPjCO4dAFt6RgBWS LNo6moEwxPoeObRFmVoCGJroaIdfbhfc56FyeOn78GW7nGVDgHpVMCUx2IJBMeQWeZ5c dF/A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@wdc.com header.s=dkim.wdc.com header.b=mtckFTT6; 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+bounces-97805-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97805-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=wdc.com Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id f22-20020a17090ace1600b0029bc1e34e56si1186554pju.81.2024.03.09.00.12.30 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 09 Mar 2024 00:12:30 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-97805-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@wdc.com header.s=dkim.wdc.com header.b=mtckFTT6; 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+bounces-97805-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97805-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=wdc.com 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 sv.mirrors.kernel.org (Postfix) with ESMTPS id EAA0A281E31 for ; Sat, 9 Mar 2024 08:12:29 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 772CA374DD; Sat, 9 Mar 2024 08:12:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=wdc.com header.i=@wdc.com header.b="mtckFTT6" Received: from esa4.hgst.iphmx.com (esa4.hgst.iphmx.com [216.71.154.42]) (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 E5994249F5; Sat, 9 Mar 2024 08:12:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=216.71.154.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709971931; cv=none; b=C1yvr/9TbEADU74Z9PQ4392og1Q4o2UCE5MIojhY90S8Of0ko9k2R2zPksRxL61Mn8DCadfMY4wovbWS1Dt4Lp7zWV7RB+NegvsRt/NChM4ePaVbWhZapmM9yE0LLmLt5QSP+qpXoYBtWrMP+T51co5IMz2uPsk5+1VK1jtfs54= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709971931; c=relaxed/simple; bh=YUXiHY4k8KdxfRkpMRa4zrfX6fzO4umIBuPK9q+tKlE=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=IaIuvAzq0sphS2sk9XC+DQEE2SyTM7D2UOgKX+y1e6ZkgiwunIlCz6/EbkrhY0Xzyr2Pt0ETsr+F1ZI6ls87SmH9LmQ1POUx133s5K2A0ca+hXfncB99DkoXgn4v4P79VqMto7rn4OforJyb1VeS9OCvloGg0Ebyd9wVGOo6pcI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wdc.com; spf=pass smtp.mailfrom=wdc.com; dkim=pass (2048-bit key) header.d=wdc.com header.i=@wdc.com header.b=mtckFTT6; arc=none smtp.client-ip=216.71.154.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=wdc.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1709971930; x=1741507930; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=YUXiHY4k8KdxfRkpMRa4zrfX6fzO4umIBuPK9q+tKlE=; b=mtckFTT6nYEuitlaINCOORYAJOvtgPAAdpgCrCPhcsFEkW6AMiCWMBoC J+scBnWhPm8iEo3B/HWl2ag5RcpcwhNdnSUGVNpR/Y5YHkkCtWP4H0y7Y vidVxIgSp9OqdJva4t0k69tREMyuWC6S8EzSymzcIXbw0klfDlaQqviZm nvy+Pjb04GenNVCBL8am781doqzJGv3MAOMqe1mreh3GMg0kkToxj9KLK HWpPAV7xdtG8IhOYIq0U3A3pM3VLXqcA6lofrLOx/Yz4tzTbfRqedsMTX zae8M0mYnMa4DE3SmGIUp2w2NMadlVgOXm+WHOQupzgde9RRArSWuJTYX A==; X-CSE-ConnectionGUID: NvKoeE8dSyO2eCb06znFxw== X-CSE-MsgGUID: Zuiiz2TyRIa70X024KqZpQ== X-IronPort-AV: E=Sophos;i="6.07,112,1708358400"; d="scan'208";a="10715102" Received: from uls-op-cesaip02.wdc.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 09 Mar 2024 16:11:07 +0800 IronPort-SDR: jSC2buZsMmFz+I5Tn+6SLJpkng26O2S7da8W9BYkAhvcYr86IgKLG24qr5RUME5OiDxX4dqMPj bzkHcBOYsKhQ== Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 08 Mar 2024 23:14:33 -0800 IronPort-SDR: xo0xJQxqbFnR1B0uE3yFCV1Jf1+WGqrMjq4vctoJz5Hd8fWZNDKLSNkqkJ9cP48N7/iAVnAYzi FL15C8bINO1g== WDCIronportException: Internal Received: from 87h6l33.ad.shared (HELO BXYGM33.ad.shared) ([10.225.32.8]) by uls-op-cesaip01.wdc.com with ESMTP; 09 Mar 2024 00:11:06 -0800 From: Avri Altman To: "James E . J . Bottomley" , "Martin K . Petersen" Cc: Bart Van Assche , Bean Huo , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Avri Altman Subject: [PATCH v3 0/4] Re-use device management code fragments Date: Sat, 9 Mar 2024 10:10:58 +0200 Message-ID: <20240309081104.5006-1-avri.altman@wdc.com> X-Mailer: git-send-email 2.42.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit v2->v3: - 2/4 - Clarify commit log (Bean) - 4/4 - pass cmd_type to ufshcd_prepare_req_desc_hdr (Bean) v1->v2: - Attend Bart's comments Device management commands are constructed for query commands that are being issued by the driver, but also for raw device management commands originated by the bsg module, and recently, by the advanced rpmb handler. Thus, the same code fragments, e.g. locking, composing the command, composing the upiu etc., appear over and over. Remove those duplications. Theoretically, there should be no functional change. Avri Altman (4): scsi: ufs: Re-use device management locking code scsi: ufs: Re-use exec_dev_cmd scsi: ufs: Re-use compose_dev_cmd scsi: ufs: Re-use compose_devman_upiu drivers/ufs/core/ufshcd.c | 204 ++++++++++++++++---------------------- include/ufs/ufshci.h | 2 +- 2 files changed, 87 insertions(+), 119 deletions(-) -- 2.42.0