From: Roland Dreier <rdreier@cisco.com>
To: Lukas Hejtmanek <xhejtman@ics.muni.cz>
Cc: xen-devel@lists.xensource.com, general@lists.openfabrics.org
Subject: Re: [ofa-general] MTHCA driver for Linux
Date: Fri, 18 Jan 2008 12:36:00 -0800 [thread overview]
Message-ID: <adatzlax45r.fsf@cisco.com> (raw)
In-Reply-To: <20080118170117.GH4136@ics.muni.cz> (Lukas Hejtmanek's message of "Fri, 18 Jan 2008 18:01:17 +0100")
> If I keep ib_mthca driver in kernel while shutting down the DomU, the next
> start of the DomU resets the machine.
>
> Trivial fix is possible: either to rmmod ib_mthca before shutting down the
> DomU or set .shutdown section to the same value as the .remove section in
> pci_driver structure.
>
> Are you willing apply a patch that sets .shutdown in the mainline of IB driver
> in Linux? Or is it something that should be fixed by Xen guys?
I would like to understand the underlying problem before blindly
setting the .shutdown method of the ib_mthca PCI driver section. The
mthca driver should be able to handle the hardware being in an
arbitrary state when it is reloaded -- that is why it resets the
adapter very early during initialization. Do you have any idea what
is going wrong in the case where the machine resets?
Very few other PCI drivers have a .shutdown method, and I don't know
of any that just duplicate the .remove method. So rather than just
having a bandaid for mthca that probably leaves the same problem for
every other driver, I would prefer to understand the problem first,
and if it is indeed something specific to mthca, then fix the
underlying issue in mthca with a simpler shutdown method.
I guess one way to debug this would be to delete operations from
mthca_remove_one() one by one (starting from the end of the function),
and each time try restarting your domU after doing rmmod ib_mthca.
When you reach the really necessary thing, then you'll see the reset.
- R.
next prev parent reply other threads:[~2008-01-18 20:36 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-18 17:01 [ofa-general] MTHCA driver for Linux Lukas Hejtmanek
2008-01-18 20:36 ` Roland Dreier [this message]
2008-01-18 20:48 ` Lukas Hejtmanek
2008-01-18 21:38 ` Roland Dreier
2008-01-18 22:50 ` Lukas Hejtmanek
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=adatzlax45r.fsf@cisco.com \
--to=rdreier@cisco.com \
--cc=general@lists.openfabrics.org \
--cc=xen-devel@lists.xensource.com \
--cc=xhejtman@ics.muni.cz \
/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.