Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp13341332rwb; Sun, 27 Nov 2022 04:18:42 -0800 (PST) X-Google-Smtp-Source: AA0mqf6l1uKu/mWojMHCxNPCuIf4FCQE+7uZVSjWtp5fWRCFuZdDiiSduXCcQ+MbUt7niE3auGHc X-Received: by 2002:a17:90a:e7c5:b0:219:252e:32c1 with SMTP id kb5-20020a17090ae7c500b00219252e32c1mr4617933pjb.182.1669551522603; Sun, 27 Nov 2022 04:18:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669551522; cv=none; d=google.com; s=arc-20160816; b=YbQCPivJz4RLyTCjQUd3DODg2wsePQ9YS3ktDXYXYUKxl7kIwxlF375E8Hf2yTaF+x iqJ9gmiNxdTRBZ7vkdQt42bunIPpTKGKkHkn3SYGA8oLms7ggjaEY0/9BzdB4bqJgCvw 6Pyeca4btmO+DROqkheT54qr6lKeO1psLbafk0VnuiNjjuV6rIHoXieEekY508EZtfIn cvVVFj95L0w8rItZ61Xv5djjkueW8ni8CsLBKG+xE2Fp50fiVWh6RRcRxSlZNipRkPLf 2tSHNINOEU/zsf7aDhh2gowTaP3nhwG3e/6jWxxZNK9vFujX+TNuWYPZny/sSX5gmIaT +olQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from :wdcironportexception:ironport-sdr:ironport-sdr:dkim-signature; bh=Lbbag9sGBwJUv0FpV7sSnmVqx6E1T75wA80KRzxNukI=; b=ZVsM9Nwl7/AHtfnShoZfmE7F8ZR8g5rLyFy6ZicytVPFzscn5wPB9qaHhPzeXLqmb4 YFfImvin14Fp7S0QuI/P3w5QL9j/74Z1kAMhJXzEDUHOLXukfqJAoap5WWtpdnLso8DI DO/cFA/P+7Rg6npqS/2eGvUX1pxmcX5rmS+cDekLLj3qvlWQ9NfuB7+cBtcL/s0uPt4e fYjVPmz0apDSqULhx9yns8j/V8pii3og47qn7vTIZqIt9XEpobtzWMHBkkwEz+d+BxXR 5Hi6khBWI79scigiLaDzi4RlFt9M2v0vKKHCFN4O5fxuUNwxjmlkbr+P8zWgxbVcVlkD fdVQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=d0kdRU7s; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=wdc.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id t12-20020a170902e84c00b001870a181f24si10060920plg.222.2022.11.27.04.18.30; Sun, 27 Nov 2022 04:18:42 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=d0kdRU7s; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=wdc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229681AbiK0MIl (ORCPT + 84 others); Sun, 27 Nov 2022 07:08:41 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53362 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229642AbiK0MIh (ORCPT ); Sun, 27 Nov 2022 07:08:37 -0500 Received: from esa4.hgst.iphmx.com (esa4.hgst.iphmx.com [216.71.154.42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 571A2F589; Sun, 27 Nov 2022 04:08:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1669550916; x=1701086916; h=from:to:cc:subject:date:message-id; bh=WsOMxY+mPL8jQ8iYAIC8w8tkwMDA68IYjaiBaloXufk=; b=d0kdRU7ssy4opxR7tuVZSXThA+doI0fGRtugjDgqwczXBhsSwqsA7wXD unTeTXkGypOAfi1UGEM/rdm9Jeb7f5BPjKukvaWj6Rc7NKN+HuCLipWEN 70bqTAccUJZ+qbjGVAdFldQjbvelcSx+5W3l6c5jzxPeVTEUYpJaNq3cc t/mIXMZUrTPOgrXdhuDp+JAII2WwKuoE7EgeC4qF1Q0WbgaQeF2tFQNhc 1r5UHN8Es1o5ZzR5QFhD0boYrSNkg2P508YFDzgBBxq3BPAxlnfdTsBsj eSC/zpypX+9RNL7SfPOiZPAZM/zeKw7HwiKQmZRhs4JhcmZ/oxKYASBq1 w==; X-IronPort-AV: E=Sophos;i="5.96,198,1665417600"; d="scan'208";a="215510875" Received: from uls-op-cesaip01.wdc.com (HELO uls-op-cesaep01.wdc.com) ([199.255.45.14]) by ob1.hgst.iphmx.com with ESMTP; 27 Nov 2022 20:08:35 +0800 IronPort-SDR: iaJd2Nw4Kw7krW7tJnTtai54dvUfJFAahhQlgxE4ozOuihy+ObwMQfgCxteqR+IxlU6DmHyPkb QqExMAb2OcxSQOtHetwqgKafveGlWixs9fTgSIWhd1qer1qHFgrb92mSFIhWbchz/e0/OBfoaG TMBfdGjCvGiMITex+xdTtGYCK7C8obSxwq6x8H88dYh/uFkdDUKLc4kB7jk1TmmLmLM6pytPex 7Ud/aEt543xkD64ZJ7MaDRyTg2YnNcYYrfI1kcPWU5RWBZHpnYz8Vn/F+qP4OSfyW/evp4mTHg NXM= Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 27 Nov 2022 03:27:18 -0800 IronPort-SDR: MUeIfeHbTxG4k+b7YqnOPpFiuVh8xYG4afDViF9r95qFJap4Q4eY890R6nAXJiBdMNfaJJdDGO aykeH8jPjZfsi/ebZ/iUD2UJHia+lj/TL4cEaO+NQS8Hvs5xiUeckM3qB4H7QGk0Xu54CLCOv1 LZtp/4qW2ECRD7LpUiQ0bWFed8kP6PvsNYkJPzeBNrrLfjNzCQ8ZEOubs7hsBLvNX9OKydI0p+ v6BNhMyYsYP0VSGAMZXcdk25jhR15/OiAsLPqFER5fDlNosOPNifQdFqtXLWSCuCxNRZnBsCnC +RM= WDCIronportException: Internal Received: from ilb001078.ad.shared (HELO ilb001078.sdcorp.global.sandisk.com) ([10.45.31.219]) by uls-op-cesaip01.wdc.com with ESMTP; 27 Nov 2022 04:08:33 -0800 From: Arthur Simchaev To: martin.petersen@oracle.com Cc: beanhuo@micron.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Arthur Simchaev Subject: [PATCH v4 0/4] ufs: core: Always read the descriptors with max length Date: Sun, 27 Nov 2022 14:08:26 +0200 Message-Id: <1669550910-9672-1-git-send-email-Arthur.Simchaev@wdc.com> X-Mailer: git-send-email 2.7.4 X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_PASS, SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org v3--v4: Add "Reviewed-by" to patch's commits Use kzalloc instead of kmalloc in drivers/ufs/core/ufshcd.c - patch 2/4 v2--v3: Based on Bean's comments: 1)Use kzalloc instead of kmalloc in ufshcd_set_active_icc_lvl - patch 2/4 2)Delete UFS_RPMB_UNIT definition - patch 2/4 3)Delete len description - patch 3/4 v1--v2: Fix argument warning in ufshpb.c Read any descriptor with a maximum size of QUERY_DESC_MAX_SIZE. According to the spec the device rerurns the actual size. Thus can improve code readability and save CPU cycles. While at it, cleanup few leftovers around the descriptor size parameter. Suggested-by: Bean Huo Arthur Simchaev (4): ufs:core: Remove redundant wb check ufs:core: Remove redundant desc_size variable from hba ufs: core: Remove len parameter from ufshcd_set_active_icc_lvl ufs: core: Remove ufshcd_map_desc_id_to_length function drivers/ufs/core/ufs_bsg.c | 7 +-- drivers/ufs/core/ufshcd-priv.h | 3 -- drivers/ufs/core/ufshcd.c | 100 ++++++++++------------------------------- drivers/ufs/core/ufshpb.c | 5 +-- include/ufs/ufshcd.h | 1 - 5 files changed, 26 insertions(+), 90 deletions(-) -- 2.7.4