mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Cc: Christoffer Dall <christoffer.dall@linaro.org>,
	Yoder Stuart-B08248 <B08248@freescale.com>,
	Alex Williamson <alex.williamson@redhat.com>,
	Kim Phillips <kim.phillips@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"a.motakis@virtualopensystems.com"
	<a.motakis@virtualopensystems.com>,
	"agraf@suse.de" <agraf@suse.de>,
	Wood Scott-B07421 <B07421@freescale.com>,
	Sethi Varun-B16395 <B16395@freescale.com>,
	Bhushan Bharat-R65777 <R65777@freescale.com>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"santosh.shukla@linaro.org" <santosh.shukla@linaro.org>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>
Subject: Re: RFC: (re-)binding the VFIO platform driver to a platform device
Date: Thu, 3 Oct 2013 14:11:34 -0500	[thread overview]
Message-ID: <1380827494.12932.161.camel@snotra.buserror.net> (raw)
In-Reply-To: <20131003185434.GA26123@kroah.com>

On Thu, 2013-10-03 at 11:54 -0700, gregkh@linuxfoundation.org wrote:
> On Thu, Oct 03, 2013 at 01:33:27PM -0500, Scott Wood wrote:
> > What it looks like we do still want from the driver core is the ability
> > for a driver to say that it should not be bound to a device except via
> > explicit sysfs bind,
> 
> You can do that today by not providing any device ids in your driver
> structure, relying on the dynamic ids the driver core creates.

I'm not sure what you mean by dynamic ids, but if the driver doesn't
provide any match data, then driver_match_device() will return 0 and
bind_store() will fail.

> > and the ability for a user to say that a device should not be bound to
> > a driver except via explicit sysfs bind.
> 
> That's not going to happen, as how can the kernel know a specific device
> is going to want this, before it asks the drivers about it?

This would normally be set by the user prior to unbinding from a
different driver, though it would also be nice to be able to set it at
boot time (e.g. via the kernel command line).

> Or, just don't ever create a driver that matches that device, then rely
> on userspace to do the binding explicitly.

That breaks the normal use case where you want the device to be bound to
the normal driver without doing anything special.  And again,
driver_match_device() will cause the bind to fail.

-Scott




  reply	other threads:[~2013-10-03 19:11 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-01 18:38 Kim Phillips
2013-10-01 19:15 ` Scott Wood
2013-10-01 19:17   ` Scott Wood
2013-10-01 22:01     ` Kim Phillips
2013-10-01 21:59   ` Kim Phillips
2013-10-01 22:44     ` Scott Wood
2013-10-01 20:00 ` Greg Kroah-Hartman
2013-10-01 22:02   ` Kim Phillips
2013-10-02  1:53     ` Christoffer Dall
2013-10-02  2:35       ` Alex Williamson
2013-10-02 15:14         ` Christoffer Dall
2013-10-02 15:29           ` Alex Williamson
2013-10-02 18:25           ` Yoder Stuart-B08248
2013-10-02 18:32             ` Scott Wood
2013-10-02 18:43               ` Christoffer Dall
2013-10-02 20:04                 ` Kim Phillips
2013-10-02 20:13                   ` Christoffer Dall
2013-10-02 20:19                     ` Scott Wood
2013-10-02 20:14                 ` Scott Wood
2013-10-02 20:27                   ` Christoffer Dall
2013-10-02 20:39                     ` gregkh
2013-10-02 20:44                       ` Christoffer Dall
2013-10-02 20:37                 ` gregkh
2013-10-02 20:42                   ` Christoffer Dall
2013-10-02 21:08                   ` Scott Wood
2013-10-02 21:16                     ` gregkh
2013-10-02 21:35                       ` Scott Wood
2013-10-02 23:40                         ` gregkh
2013-10-03 18:33                           ` Scott Wood
2013-10-03 18:54                             ` gregkh
2013-10-03 19:11                               ` Scott Wood [this message]
2013-10-03 20:32                                 ` gregkh
2013-10-09 19:02                                   ` Yoder Stuart-B08248
2013-10-09 19:16                                     ` gregkh
2013-10-09 19:49                                       ` Scott Wood
2013-10-09 19:21                                     ` Scott Wood
2013-10-09 19:44                                       ` Yoder Stuart-B08248
2013-10-09 20:03                                         ` Scott Wood
2013-10-10  3:05                                           ` Kim Phillips
2013-10-10  8:01                                             ` Bhushan Bharat-R65777
2013-10-10 15:27                                               ` Scott Wood
2013-10-11  6:27                                                 ` [PATCH 1/4] driver core: Add new device_driver flag to allow binding via sysfs only Kim Phillips
2013-10-11  6:27                                                   ` [PATCH 2/4] driver core: platform: allow platform drivers to bind to any device Kim Phillips
2013-10-11  6:27                                                   ` [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only Kim Phillips
2013-10-11 20:43                                                     ` Scott Wood
2013-10-11 23:17                                                       ` Kim Phillips
2013-10-14 13:01                                                         ` Yoder Stuart-B08248
2013-10-14 17:13                                                           ` Scott Wood
2013-10-24 11:32                                                         ` Bhushan Bharat-R65777
2013-10-28 17:47                                                     ` Alex Williamson
2013-10-28 18:00                                                       ` Scott Wood
2013-10-28 18:09                                                         ` Scott Wood
2013-10-29  3:38                                                           ` Bhushan Bharat-R65777
2013-10-29  3:40                                                             ` Scott Wood
2013-10-29  3:52                                                               ` Bhushan Bharat-R65777
2013-10-29  4:29                                                                 ` Scott Wood
2013-10-29  4:31                                                                   ` Bhushan Bharat-R65777
2013-10-29  4:35                                                                     ` Scott Wood
2013-10-29  4:45                                                                       ` Bhushan Bharat-R65777
2013-10-29  4:54                                                                         ` Scott Wood
2013-10-29  6:39                                                                           ` Bhushan Bharat-R65777
2013-10-11  6:27                                                   ` [PATCH] VFIO: platform: allow the driver to bind to any device explicitly via sysfs Kim Phillips
2013-10-10  7:45                                           ` RFC: (re-)binding the VFIO platform driver to a platform device Bhushan Bharat-R65777
2013-10-10 13:43                                             ` Yoder Stuart-B08248
2013-10-10 15:23                                             ` Scott Wood
2013-10-10 15:25                                               ` Bhushan Bharat-R65777

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=1380827494.12932.161.camel@snotra.buserror.net \
    --to=scottwood@freescale.com \
    --cc=B07421@freescale.com \
    --cc=B08248@freescale.com \
    --cc=B16395@freescale.com \
    --cc=R65777@freescale.com \
    --cc=a.motakis@virtualopensystems.com \
    --cc=agraf@suse.de \
    --cc=alex.williamson@redhat.com \
    --cc=christoffer.dall@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=kim.phillips@linaro.org \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peter.maydell@linaro.org \
    --cc=santosh.shukla@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome