[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180511174241.GC7344@localhost.localdomain>
Date: Fri, 11 May 2018 11:42:42 -0600
From: Keith Busch <keith.busch@...ux.intel.com>
To: Bjorn Helgaas <helgaas@...nel.org>
Cc: Jesse Vincent <jesse@...k.com>, Sagi Grimberg <sagi@...mberg.me>,
linux-pci@...r.kernel.org, Andrew Lutomirski <amluto@...il.com>,
linux-kernel@...r.kernel.org, linux-nvme@...ts.infradead.org,
Jens Axboe <axboe@...com>, Bjorn Helgaas <bhelgaas@...gle.com>,
Christoph Hellwig <hch@....de>
Subject: Re: Another NVMe failure, this time with AER info
On Fri, May 11, 2018 at 11:26:11AM -0600, Keith Busch wrote:
> I trust you know the offsets here, but it's hard to tell what this
> is doing with hard-coded addresses. Just to be safe and for clarity,
> I recommend the 'CAP_*+<offset>' with a mask.
>
> For example, disabling ASPM L1.2 can look like:
>
> # setpci -s <B:D.f> CAP_PM+8.l=0:4
My mistake above: CAP_PM is a different capability, not the intended
one. It looks like setpci doesn't even have a convenient symbol for the
L1 PM extended capability, so the hard-coded offsets are the only way
for this setting. Sorry about the mistake.
Powered by blists - more mailing lists