From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: Suman Tripathi <stripathi@apm.com>,
olof@lixom.net, tj@kernel.org, arnd@arndb.de
Cc: linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
mlangsdo@redhat.com, jcm@redhat.com, patches@apm.com
Subject: Re: [PATCH v2] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.
Date: Mon, 02 Feb 2015 20:07:53 +0300 [thread overview]
Message-ID: <54CFAEE9.30002@cogentembedded.com> (raw)
In-Reply-To: <1422878229-20488-2-git-send-email-stripathi@apm.com>
Hello.
On 02/02/2015 02:57 PM, Suman Tripathi wrote:
> This patch addresses the issue with ATA_CMD_SMART pio mode
> command for enumeration and device detection with ATA devices.
> The X-Gene AHCI controller has an errata in which it cannot clear
> the BSY bit after the PIO setup FIS. The dma state machine enters
> CMFatalErrorUpdate state and locks up. It is the same issue as
> in the commit 2a0bdff6b958d1b2523d2754b6cd5e0ea4053016(fix the
You forgot a space before '('.
> dma state machine lockup for the IDENTIFY DEVICE PIO mode
> command).
Actually, the summary is "ahci_xgene: fix the dma state machine lockup for
the IDENTIFY DEVICE PIO mode command.", i.e. yours is missing a prefix.
[...]
> Signed-off-by: Suman Tripathi <stripathi@apm.com>
> Reported-by: Mark Langsdorf <mlangsdo@redhat.com>
MBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.
Date: Mon, 02 Feb 2015 20:07:53 +0300 [thread overview]
Message-ID: <54CFAEE9.30002@cogentembedded.com> (raw)
In-Reply-To: <1422878229-20488-2-git-send-email-stripathi@apm.com>
Hello.
On 02/02/2015 02:57 PM, Suman Tripathi wrote:
> This patch addresses the issue with ATA_CMD_SMART pio mode
> command for enumeration and device detection with ATA devices.
> The X-Gene AHCI controller has an errata in which it cannot clear
> the BSY bit after the PIO setup FIS. The dma state machine enters
> CMFatalErrorUpdate state and locks up. It is the same issue as
> in the commit 2a0bdff6b958d1b2523d2754b6cd5e0ea4053016(fix the
You forgot a space before '('.
> dma state machine lockup for the IDENTIFY DEVICE PIO mode
> command).
Actually, the summary is "ahci_xgene: fix the dma state machine lockup for
the IDENTIFY DEVICE PIO mode command.", i.e. yours is missing a prefix.
[...]
> Signed-off-by: Suman Tripathi <stripathi@apm.com>
> Reported-by: Mark Langsdorf <mlangsdo@redhat.com>
MBR, Sergei
next prev parent reply other threads:[~2015-02-02 17:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-02 11:57 [PATCH v2] Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command Suman Tripathi
2015-02-02 11:57 ` [PATCH v2] ahci_xgene: " Suman Tripathi
2015-02-02 17:07 ` Sergei Shtylyov [this message]
2015-02-02 17:07 ` Sergei Shtylyov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54CFAEE9.30002@cogentembedded.com \
--to=sergei.shtylyov@cogentembedded.com \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=jcm@redhat.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=mlangsdo@redhat.com \
--cc=olof@lixom.net \
--cc=patches@apm.com \
--cc=stripathi@apm.com \
--cc=tj@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.