Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp4142214rdb; Mon, 11 Dec 2023 09:59:44 -0800 (PST) X-Google-Smtp-Source: AGHT+IGZCJ9R49vCdd8Zhohx76UHqBEzZoncCpA/EMXF/a2/t92iT13uUDAzEwvZ9dwhYVhUtKcQ X-Received: by 2002:a05:6a20:918b:b0:18c:8afc:cdd2 with SMTP id v11-20020a056a20918b00b0018c8afccdd2mr2562934pzd.17.1702317584002; Mon, 11 Dec 2023 09:59:44 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702317583; cv=none; d=google.com; s=arc-20160816; b=qOl4tAIZy1ZARnLoH1Alq3AEDkWe6NTVey8Y74uTC9ZbTEQJwUaoh0TDxRYaMg2nHT 2n3hopxclcLpRs88CCpnfwzuJ+zygfxDI7Q2SvoRvevAwYBNKZ8MDahAk2ywC+ySNSdi 4VraYzW4hHRTJ8jfhmncZ9XgkTm94xDYdX4fmHpI1eOWm3sMM9JNXbb/WDoi0DkORRyZ d67xKBzefTuF0YuOVxcLvjBZPfxRuFvcG6srIzknkKc7MDC2GtAL8PNAilo9mWqwRtke 4g5mGmflV5Sy5j+cjbHXQXsF6eN3GC/0qzq0Dlv62C/wc4YHuGKgS/luvp1vqNjGSYSR J0dg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id; bh=8TG12YyuQcIlIUIhJpok1Z9XhgvnLDNR3xx7iZohyHA=; fh=kZU+eCw01JUAiNhzTjqJP9kH5mZSEqgBG9EmltBBod8=; b=LDLBaWkmVg/EIHKWQBBhsxiPhBK5+U7uYc2QTye7O4/YVroTvF+O5qzKGQpePUHuAN pWXwfzCgBjLVLZe1vw0aJHVcPAFbVHO9jifU9RFMyMpxugtMdjnsgt9wobuGM/bhEtX2 WnLOj+v7PSaTKPN1C6UiF8nX6jXiRTUcUJEup2JWgrJu/aDCAH+Z7VV4e1ukToOCB8uk vRnewype+8v9O8lzIyYt1sexe0AWPEhe6vVLnWNqi9dWwil7VI9id8ZpnMWzmJz7XkMA E67l+nyzNZyZ+BEZzxuihZJCzc+etXOEuiAasmgb7t8r/WR6XHpW1M5MkKcXJ6l7SMr1 A2cg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=acm.org Return-Path: Received: from groat.vger.email (groat.vger.email. [23.128.96.35]) by mx.google.com with ESMTPS id q5-20020a656845000000b0057745d87b50si6191474pgt.139.2023.12.11.09.59.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Dec 2023 09:59:43 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=acm.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id AF0358098FE8; Mon, 11 Dec 2023 09:59:41 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345128AbjLKR7Y (ORCPT + 99 others); Mon, 11 Dec 2023 12:59:24 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51194 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1345074AbjLKR7X (ORCPT ); Mon, 11 Dec 2023 12:59:23 -0500 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A38C9C3; Mon, 11 Dec 2023 09:59:29 -0800 (PST) Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-1d2e6e14865so21498905ad.0; Mon, 11 Dec 2023 09:59:29 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1702317569; x=1702922369; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=8TG12YyuQcIlIUIhJpok1Z9XhgvnLDNR3xx7iZohyHA=; b=F67ZyNzD63Qx4pluM/am8tEL13aLrpl+Qph7Hf8agm7nqr79KZE8QOUY0CM3gRro2V Q+yHzKs+wyaj4Fsl4wDFi49FvhF77MSVOZ/u8mQMhDwBaLj65fjDweSfxV4NOpl4AX7B SQK7W1aBTfcPYaU5/ORr4Q+3+sdJrqZeRKwhW0eC49PDKnNCTAmnqLlS26JxpwQLcXkl 71/ksIzPyspdYcqtIN78Pj0wPrjb2YXKxarCC2OFSqLBKTy7iHXpU7ewCHAEQpYpjnuI 5ZenvxsCKEZ5WWCPODa9F4yL+Vj6XDyHKCNcXXaooytPxwOQxh/uPRVz5uLLq5YROz0U 7oiQ== X-Gm-Message-State: AOJu0Yz2X+2fbjtc3oeycDDG35AIH2n+eEtxGsuYsMaPY49sHr/FxJJU /hcv1gdx4Fe4ue3poh43R18= X-Received: by 2002:a17:902:e810:b0:1d0:6ffd:6e60 with SMTP id u16-20020a170902e81000b001d06ffd6e60mr2382910plg.88.1702317568985; Mon, 11 Dec 2023 09:59:28 -0800 (PST) Received: from ?IPV6:2620:0:1000:8411:3431:681a:6403:d100? ([2620:0:1000:8411:3431:681a:6403:d100]) by smtp.gmail.com with ESMTPSA id w18-20020a170902e89200b001d0855ce7c8sm6979575plg.252.2023.12.11.09.59.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 11 Dec 2023 09:59:28 -0800 (PST) Message-ID: <0d59681d-2d7d-4459-b79c-c5f41f20b7a5@acm.org> Date: Mon, 11 Dec 2023 09:59:25 -0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] scsi: ufs: qcom: Perform read back after writing reset bit Content-Language: en-US To: Andrew Halaney , Andy Gross , Bjorn Andersson , Konrad Dybcio , Manivannan Sadhasivam , "James E.J. Bottomley" , "Martin K. Petersen" , Yaniv Gardi , Dov Levenglick Cc: Will Deacon , linux-arm-msm@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org References: <20231208-ufs-reset-ensure-effect-before-delay-v1-1-8a0f82d7a09e@redhat.com> From: Bart Van Assche In-Reply-To: <20231208-ufs-reset-ensure-effect-before-delay-v1-1-8a0f82d7a09e@redhat.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Mon, 11 Dec 2023 09:59:41 -0800 (PST) On 12/8/23 12:19, Andrew Halaney wrote: > The recommendation for ensuring this bit has taken effect on the > device is to perform a read back to force it to make it all the way > to the device. This is documented in device-io.rst [... ] There are more mb()'s that need to be replaced, namely the mb() calls in ufshcd_system_restore() and ufshcd_init(). Thanks, Bart.