All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arthur Marsh <arthur.marsh@internode.on.net>
To: Jiang Liu <jiang.liu@linux.intel.com>,
	James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	Bjorn Helgaas <bhelgaas@google.com>,
	Hannes Reinecke <hare@suse.de>,
	Ballabio Dario <dario.ballabio@emc.com>,
	Christoph Hellwig <hch@infradead.org>,
	Dario Ballabio <ballabio_dario@emc.com>,
	linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
	linux-scsi@vger.kernel.org, x86@kernel.org
Subject: Re: [RFT v3] eata: Convert eata driver as normal PCI and platform device drivers
Date: Sat, 26 Sep 2015 15:57:03 +0930	[thread overview]
Message-ID: <56063AB7.9030309@internode.on.net> (raw)
In-Reply-To: <560390A1.5060503@internode.on.net>



Arthur Marsh wrote on 24/09/15 15:26:
>
>
> Jiang Liu wrote on 24/09/15 13:58:
>
>> Hi James,
>>     Thanks for review. How about the attached patch which addresses
>> the three suggestions from you?
>> Thanks!
>> Gerry
>
> I've applied the patch, rebuilt the kernel and verified that it allows
> unloading of the eata module and reloading it, as well as a successful
> kexec.
>
> Regards,
>
> Arthur.

After some more thorough testing I've encountered an ongoing problem 
trying to use kexec with filesystems mounted with the eata driver.

If I boot up and have the eata driver loaded but no filesystem check or 
mounting of filesystems on the disk attached to the DPT2044W controller, 
then attempt a kexec reboot I get the reboot pausing after the 
"synchronizing scsi cache" messages and getting the errors that I have 
included as pictures in my previous reports.

If I do a normal boot which includes eata being loaded, the disk 
attached to the DPT2044W controller having its filesystems checked and 
mounted, then attempt a kexec reboot, I get the reboot pausing after the 
"synchronizing SCSI cache" messages as before.

If I un-mount the filesystems on the disk attached to the DPT2044W 
controller after start-up and try a reboot I get the same problem.

If I do modprobe -r eata after un-mounting the filesystems on the disk 
attached to the DPT2044W controller after a start-up kexec *works fine*.

If I do:

start-up
un-mount filesystems on disk attached to DPT2044W controller
modprobe -r eata
modprobe eata
fsck -a of filesystems on disk attached to DPT2044W controller
mount filesystems

then a kexec reboot works fine.

I did some more experimenting and found a workaround:

I was unable to blacklist the eata module but if I did:

modprobe -r eata
modprobe eata

in a cron job before the fsck and mount commands then
I could then perform a kexec reboot successfully.

I also verified that if I did:

modprobe -r eata

after eata was loaded on boot-up without any fsck or mounting of 
filesystems on the disk attached to the DPT2044W controller using the 
eata the kexec reboot worked fine.

In summary:

if eata is loaded kexec reboot will fail unless a modprobe -r eata is 
done either manually or by a cron job.

if a modprobe -r eata has been done, then even if I modprobe eata and 
fsck and mount filesystems, kexec reboot works.

Any suggestions for further tests or checks welcome.

Arthur.



  reply	other threads:[~2015-09-26  6:27 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-06  6:28 eata fails to load on post 4.2 kernels Arthur Marsh
2015-09-06  9:04 ` Arthur Marsh
2015-09-06 11:37   ` Arthur Marsh
2015-09-06 20:31     ` Arthur Marsh
2015-09-07  3:06       ` Jiang Liu
     [not found]         ` <55ED2DFA.6030502@internode.on.net>
2015-09-07  9:24           ` Fwd: " Arthur Marsh
2015-09-08  5:19           ` Jiang Liu
2015-09-08  6:32             ` Arthur Marsh
2015-09-08  6:37               ` Jiang Liu
2015-09-08  7:26               ` [Bugfix] PCI, x86: Correctly allocate IRQs for PCI devices managed by non-PCI drivers Jiang Liu
2015-09-08  9:03                 ` Arthur Marsh
2015-09-08  9:44                   ` Jiang Liu
2015-09-08 16:27                 ` Bjorn Helgaas
2015-09-08 16:49                   ` Jiang Liu
2015-09-09 19:11                     ` Bjorn Helgaas
2015-09-10  8:58                       ` Jiang Liu
2015-09-14  3:08                       ` [Bugfix 0/3] Convert eata driver to a normal PCI device driver Jiang Liu
2015-09-14  3:08                         ` [Bugfix 1/3] eata: Use IDA to manage eata board IDs Jiang Liu
2015-09-14  8:08                           ` Hannes Reinecke
2015-09-14  8:08                             ` Hannes Reinecke
2015-09-14  3:08                         ` [Bugfix 2/3] eata: Implement PCI driver to manage eata PCI devices Jiang Liu
2015-09-14  8:17                           ` Hannes Reinecke
2015-09-14  8:31                             ` Jiang Liu
2015-09-14  3:08                         ` [Bugfix 3/3] eata: Enhance eata driver to support PCI device hot-removal Jiang Liu
2015-09-14  8:21                           ` Hannes Reinecke
2015-09-14  8:31                             ` Ballabio, Dario
2015-09-14  8:33                               ` Jiang Liu
2015-09-16 13:42                             ` Christoph Hellwig
2015-09-17  6:49                               ` Jiang Liu
2015-09-18 15:08                               ` Arthur Marsh
2015-09-22  7:30                               ` [RFT v3] eata: Convert eata driver as normal PCI and platform device drivers Jiang Liu
2015-09-22 20:27                                 ` Hannes Reinecke
2015-09-22 22:25                                 ` Arthur Marsh
2015-09-22 22:45                                   ` James Bottomley
2015-09-22 23:36                                     ` Arthur Marsh
2015-09-23  5:24                                       ` Jiang Liu
2015-09-23 10:44                                         ` Arthur Marsh
2015-09-23 14:40                                           ` James Bottomley
2015-09-24  4:28                                             ` Jiang Liu
2015-09-24  5:56                                               ` Arthur Marsh
2015-09-26  6:27                                                 ` Arthur Marsh [this message]
2015-10-03  8:11                                                   ` Jiang Liu
2015-10-03 11:14                                                     ` Arthur Marsh
2015-10-05  8:29                                                     ` Arthur Marsh
2015-09-14 16:01                         ` [Bugfix 0/3] Convert eata driver to a normal PCI device driver Arthur Marsh
2015-09-15  2:31                           ` Jiang Liu
2015-09-15  7:19                             ` Arthur Marsh
2015-09-16  5:07                               ` Jiang Liu
2015-09-16  7:37                                 ` Arthur Marsh
2015-09-16  8:21                                   ` Jiang Liu
2015-09-16 11:29                                     ` Arthur Marsh
2015-09-09 19:04                 ` [Bugfix] PCI, x86: Correctly allocate IRQs for PCI devices managed by non-PCI drivers Arthur Marsh
2015-09-10  7:50             ` eata fails to load on post 4.2 kernels Arthur Marsh
2015-09-10  8:13               ` Jiang Liu
2015-09-10  9:00                 ` 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=56063AB7.9030309@internode.on.net \
    --to=arthur.marsh@internode.on.net \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=ballabio_dario@emc.com \
    --cc=bhelgaas@google.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-pci@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=tglx@linutronix.de \
    --cc=x86@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.