All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: "Martin Öhrling" <martin@xn--hrling-vxa.se>
Cc: Richie <listmail@triad.rr.com>, xen-devel <xen-devel@lists.xen.org>
Subject: Re: testing of Xen PCIbackend support for slot and bus reset
Date: Tue, 28 Oct 2014 10:04:36 -0400	[thread overview]
Message-ID: <20141028140436.GD21290@laptop.dumpdata.com> (raw)
In-Reply-To: <CABg547UGe3y22Y9fpBSMn3L767+vG394XUzGGgTQ4+0XK=mTVw@mail.gmail.com>

On Tue, Oct 28, 2014 at 11:58:41AM +0100, Martin Öhrling wrote:
> Any progress in testing the branch?
> 
> I can see one thing that could explain if the function isn't called. The
> reset file is only created if it doesn't exist. I know it's unlikely for a
> gpu, but have you verified that your driver didn't export a "reset"-file
> without these patches? As far as I can tell, the drivers reset function
> will be used instead of pcistub_reset_pci_dev() in the presence of a
> "reset"-file.

Use the 'do_flr' SysFS instead of the 'reset'. Once an PCI device
is 'owned' by PCIback the 'reset' is no longer routed to the GPU driver.

And the 'reset' does not do a bus-reset (whcih is what you need for
most GPUs).


> 
> /Martin
> 
> 2014-08-17 3:52 GMT+02:00 Richie <listmail@triad.rr.com>:
> 
> >  @Konrad
> >
> > I'm interesting in testing your latest iteration of bus reset support.
> > Actually I cloned your devel/xen-pciback.slot_and_bus.v2.1
> > <http://git.kernel.org/cgit/linux/kernel/git/konrad/xen.git/log/?h=devel/xen-pciback.slot_and_bus.v2.1>
> > branch and tested that but did not seem to reset the vga card.  I did
> > capture dmesg output which was nice and clean regarding resetting the
> > device but don't have it handy.
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xen.org
> > http://lists.xen.org/xen-devel
> >
> >

> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

  reply	other threads:[~2014-10-28 14:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-17  1:52 testing of Xen PCIbackend support for slot and bus reset Richie
2014-10-28 10:58 ` Martin Öhrling
2014-10-28 14:04   ` Konrad Rzeszutek Wilk [this message]
2014-10-28 15:06     ` Martin Öhrling
2014-10-28 15:20       ` Martin Öhrling

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=20141028140436.GD21290@laptop.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=listmail@triad.rr.com \
    --cc=martin@xn--hrling-vxa.se \
    --cc=xen-devel@lists.xen.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.