From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (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 A833A268C4A for ; Thu, 13 Mar 2025 15:13:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741878815; cv=none; b=bs8cQ8a25jBhNL0p+EIGI/JCzydhv8PL8cxhB31iiMpR4yvZJVthq95yHSVs+GEFNuP8rZzcSy25ROXI9MKdRM0/69zUbXUpEdyTXwjzuHkdzv8eYf4y3OLjf8X2ECOoYlvll/7wgnxbbfvaCavAOIq7aSqiGSCjJyt6S92Lnqc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741878815; c=relaxed/simple; bh=F6KcbIPUgfP6kX1LbBxUFczoKtPUcvCPuPG03lq65CM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=paOEPtjirygPhho95GllZBzbL8isZjSJwvAXK1ixy0ZOQ1z3nWZOlKGM4GhSm2buuXKP6D34B6gcQd/Mg9lqdJmi+TaQRFX9E2FmIHHudxnXRnVNBSzTaUTYz2Df8jqXPGk/kdvmrqQltc2q8nKf/bor7BXIOJawutVbxR2xrS8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=Jp1AtK2/; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="Jp1AtK2/" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1741878812; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=f5K4pCCizVTVFaCfJxsbcvtellHTlJx5Au+uaHTpI0k=; b=Jp1AtK2/x4hUighXSEhUtgNbwyfi3Tx7HeGxFFvY513S1RM+6YLTG03fkM+O5Wg9MuacLX pWI58iT/TtMYagc1leUEGzcl1403u1PIgKD3qyPPMrUGohiTeot51vMLW/hGXz0lF+L/gZ ekhKwhO35JpZeAkffAo+Gu/sxgBaRMo= Received: from mail-ej1-f71.google.com (mail-ej1-f71.google.com [209.85.218.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-97-rUCx6LZFPDa5B1YPO-9PCQ-1; Thu, 13 Mar 2025 11:13:28 -0400 X-MC-Unique: rUCx6LZFPDa5B1YPO-9PCQ-1 X-Mimecast-MFC-AGG-ID: rUCx6LZFPDa5B1YPO-9PCQ_1741878807 Received: by mail-ej1-f71.google.com with SMTP id a640c23a62f3a-ac2815aba51so85668066b.3 for ; Thu, 13 Mar 2025 08:13:28 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741878807; x=1742483607; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=f5K4pCCizVTVFaCfJxsbcvtellHTlJx5Au+uaHTpI0k=; b=gRXnVoA8LWVBZrmELiYv3re8yKCCAB4uZjOwtQRBSqSxVqxB8DiRdHO8cZjC4AE5hH YJuFZIFkQyKYHbDpFotHC3QVVT2o1lDdsIxWp2by7ZxyzUSBX6cJIeAUwBMvnkyuxt8K trvt6TO92Yi8AqgqVjeZHAA2R5xKAHiry+OzaX/IIaliW7ge0UYBkjVeJMAVdBQR86HY ys3L5eKstOz0NvsqZ20AX/u1c+3CzN0BzXPSx5EQymvYStOd+J9LdBbYKBT/XXh5aYio PNOPH5fnIEpNnBiAFupst/jT1FUgC8DWKLAqGEGT0aoVxTz1VK8swxowU0zxgYFoi3sO 4S9A== X-Forwarded-Encrypted: i=1; AJvYcCX7fyjmAtqkGIUECZKBbcJ+aP+IMGlahyCvU9cVJ/5WG9KeeubFsbVGF4pFKqFHVsYwARYuFYAVBLk=@vger.kernel.org X-Gm-Message-State: AOJu0YxxhkPN1i3KgQCyTI/UyRg5Vlx8xLHMj0u6De1HPiFiB4dW7dkx G3ZxPnqojk0LU1ekpSMNA/gTKgfHPt5P2i53sHBlP5a2lTbkPLlHEJ+S8WgWeHZIeVwH1WVN/Pd pG4ybKOyqxSdAiqCXM9GKXlNuX0eoP++cfjqG7pp7w1pE482rR+24iUtlqg== X-Gm-Gg: ASbGncsBxJuuKAm/2B6Bj5E448Cd5ydCZHORJPnwtqjGF/PZXU1tizFdrcJYMb8pJwB WiDpEmVHAmz77gCCfKclKMKi48iEbN5Gj7ehUzE+A8PtSCbbODUGI+HXIFCxNXy55ZbWrhz6KaG u+s5Jg8E0dxNuUGwNlGjCHUbQyoXT4Q2PH9CRcucHQ+IqAaznEF5ISMkaBYDneRl+MDWQ6YgEE8 8tgKOYU/cRRqmZBr0OqxcozbeCzbCU437+1qEBjFoOf2BJ+nLRgWvoyLr9hoIA1kiG8bjshe2DW WhPzrCkR+SJ5bpt6e671810Rj6qmLZsEZLWgBiarp+AsWrlCcbjdwkiVutziertUs4L6hydCEih 0vJ3Ydk8p9W/ItgBluuUmCh9SvNp/Zj/tPVPX6cheKisKEA6W+H1va1F2SCRKyswViA== X-Received: by 2002:a17:906:c381:b0:ac2:ff66:dd68 with SMTP id a640c23a62f3a-ac2ff66e08dmr593616666b.39.1741878806665; Thu, 13 Mar 2025 08:13:26 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHIfef2yMH96c0YIEDZ4YUlcCBGhBwi8aFPWDDZ+VDysqpAUegkeCUPWy1GYkWLR9SUagK39A== X-Received: by 2002:a17:906:c381:b0:ac2:ff66:dd68 with SMTP id a640c23a62f3a-ac2ff66e08dmr593612466b.39.1741878806264; Thu, 13 Mar 2025 08:13:26 -0700 (PDT) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ac314a40f61sm90305766b.134.2025.03.13.08.13.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 13 Mar 2025 08:13:25 -0700 (PDT) Message-ID: Date: Thu, 13 Mar 2025 16:13:24 +0100 Precedence: bulk X-Mailing-List: linux-ide@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Regression from 7627a0edef54 ("ata: ahci: Drop low power policy board type") on reboot (but not cold boot) To: Niklas Cassel Cc: Eric , Salvatore Bonaccorso , Mario Limonciello , Christoph Hellwig , Mika Westerberg , Damien Le Moal , Jian-Hong Pan , regressions@lists.linux.dev, linux-kernel@vger.kernel.org, stable@vger.kernel.org, linux-ide@vger.kernel.org, Dieter Mummenschanz References: <8b1cbfd4-6877-48ef-b17d-fc10402efbf7@grabatoulnz.fr> <689f8224-f118-47f0-8ae0-a7377c6ff386@grabatoulnz.fr> <9c4a635a-ce9f-4ed9-9605-002947490c61@redhat.com> <383d5740-7740-4051-b39a-b8c74b035ec2@redhat.com> <9ac6e1ab-f2af-4bff-9d50-24df68ca1bb9@redhat.com> Content-Language: en-US, nl From: Hans de Goede In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hi Niklas, On 13-Mar-25 1:48 PM, Niklas Cassel wrote: > Hello Hans, > > On Thu, Mar 13, 2025 at 11:04:13AM +0100, Hans de Goede wrote: >> >> I do agree with you that it is a question if this is another bad >> interaction with Samsung SATA SSDs, or if it is a general ATI SATA >> controller problem, but see below. > > (snip) > >> Right in the mean time Eric has reported back that the above patch fixes >> this. Thank you for testing this Eric, >> >> One reason why ATA_QUIRK_NO_NCQ_ON_ATI was introduced is because >> disabling NCQ has severe performance impacts for SSDs, so we did not want >> to do this for all ATI controllers; or for all Samsung drives. Given that >> until the recent LPM default change we did not use DIPM on ATI chipsets >> the above fix IMHO is a good fix, which even keeps the rest of the LPM >> power-savings. > > One slightly interesting thing was that neither the Maxtor or the Samsung > drive reported support for Host-Initiated Power Management (HIPM). > > Both drives supported Device-Initiated Power Management (DIPM), and we > could see that DIPM was enabled on both drives. > > We already know that LPM works on the Samsung drive with an Intel AHCI > controller. (But since the device does not report support for HIPM, even > on Intel, only DIPM will be used/enabled.) > > >> >> Right I think it is safe to assume that this is not a Samsung drive problem >> it is an ATI controller problem. The only question is if this only impacts >> ATI <-> Samsung SSD combinations or if it is a general issue with ATI >> controllers. But given the combination of DIPM not having been enabled >> on these controllers by default anyways, combined with the age of these >> motherboards (*) I believe that the above patch is a good compromise to >> fix the regression without needing to wait for more data. >> >> Regards, >> >> Hans >> >> *) And there thus being less users making getting more data hard. And >> alo meaning not having DIPM will impact only the relatively few remaining >> users > > I'm still not 100% sure with the best way forward. > > The ATI SATA controller reports that it supports ALPM (i.e. also HIPM). > It also reports support for slumber and partial, which means that it must > support both host initiated and device initiated requests to these states. > (See AHCI spec 3.1.1 - Offset 00h: CAP – HBA Capabilities, > CAP.PSC and CAP.SSC fields.) > > Considering that DIPM seems to work fine on the Maxtor drive, I guess your > initial suggestion of a Samsung only quirk which only disables LPM on ATI > is the best way? I have no objections against going that route, except that I guess this should then be something like ATA_QUIRK_NO_DIPM_ON_ATI to not loose the other LPM modes / savings? AFAIK/IIRC there still is quite some powersaving to be had without DIPM. > It seems that ATI and Samsung must have interpreted some spec differently > from each other, otherwise, I don't understand why this combination > specificially seems to be so extremely bad, ATI + anything other than > Samsung, or Samsung + anything other than ATI seems to work. Yes the most severe problems do seem to come from that specific mix, although the long list of other ATI controller quirks also shows those controllers are somewhat finicky. Regards, Hans