From: Arthur Marsh <arthur.marsh@internode.on.net>
To: Christoph Hellwig <hch@infradead.org>,
Jiang Liu <jiang.liu@linux.intel.com>
Cc: Dario Ballabio <ballabio_dario@emc.com>,
"James E.J. Bottomley" <JBottomley@odin.com>,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Hannes Reinecke <hare@suse.de>,
"Ballabio, Dario" <dario.ballabio@emc.com>,
linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] eata: Convert eata driver as normal PCI and platform device drivers
Date: Wed, 2 Mar 2016 08:06:26 +1030 [thread overview]
Message-ID: <56D60B5A.3080209@internode.on.net> (raw)
In-Reply-To: <56D5D0F4.4060301@internode.on.net>
Arthur Marsh wrote on 02/03/16 03:57:
>
>
> Christoph Hellwig wrote on 01/03/16 17:22:
>> Hi Jiang.
>>
>> I'd love to see this patch in and abuse of the old PCI API gone.
>>
>> Did you resolve the problems Arthur saw with the previous iteratons
>> of the patch?
>>
>
> I applied Jiang Liu's patch of 1st March 2016 to a clean kernel
> 4.5.0-rc6 source, removed my workaround of removing and re-adding the
> eata module before mounting file-systems that are on disks attached to
> the DPT SCSI card using the eata driver, and was able to kexec from the
> new kernel successfully.
>
> Arthur.
I spoke too soon, without removing and re-inserting the eata module
before any filesystems on disks attached to the DPT controller were
mounted, I'd get the following messages, similar to ones previously
reported:
sd 0:0:6:0: tag#0 abort, mbox 1.
EATA0: abort, mbox 1 is in use.
sd 0:0:6:0: tag#0 reset, enter.
EATA0: reset, mbox 1 in reset.
EATA0: reset, board reset done, enabling interrupts.
EATA0: reset, interrupts disabled, loops 100415.
EATA0, reset, mbox 1 locked, DID_RESET, done.
EATA0: reset, exit, done.
and so on, finally hanging after printing "kexec_core: Starting new
kernel" (I have a photo of the messages if they're needed).
So I'm still using the new patch but have to continue to remove and
reinsert eata at start-up before any attempts to mount disks attatched
to the DPT SCSI controller.
Arthur.
next prev parent reply other threads:[~2016-03-01 21:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-01 3:02 [PATCH] eata: Convert eata driver as normal PCI and platform device drivers Jiang Liu
2016-03-01 3:10 ` kbuild test robot
2016-03-01 3:10 ` kbuild test robot
2016-03-01 3:16 ` kbuild test robot
2016-03-01 3:16 ` kbuild test robot
2016-03-01 6:52 ` Christoph Hellwig
2016-03-01 17:27 ` Arthur Marsh
2016-03-01 21:36 ` Arthur Marsh [this message]
2016-03-02 3:20 ` Jiang Liu
2016-03-02 6:52 ` Arthur Marsh
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=56D60B5A.3080209@internode.on.net \
--to=arthur.marsh@internode.on.net \
--cc=JBottomley@odin.com \
--cc=ballabio_dario@emc.com \
--cc=dario.ballabio@emc.com \
--cc=hare@suse.de \
--cc=hch@infradead.org \
--cc=jiang.liu@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
/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.