From: Jason Gunthorpe <firstname.lastname@example.org> To: Greg Kroah-Hartman <email@example.com> Cc: firstname.lastname@example.org, "Rafael J. Wysocki" <email@example.com> Subject: Re: [PATCH 01/10] driver core: Do not continue searching for drivers if deferred probe is used Date: Tue, 8 Jun 2021 09:16:54 -0300 [thread overview] Message-ID: <20210608121654.GX1002214@nvidia.com> (raw) In-Reply-To: <YL8RxPEMCDTXnPDg@kroah.com> On Tue, Jun 08, 2021 at 08:44:20AM +0200, Greg Kroah-Hartman wrote: > On Mon, Jun 07, 2021 at 09:55:43PM -0300, Jason Gunthorpe wrote: > > Once a driver has been matched and probe() returns with -EPROBE_DEFER the > > device is added to a deferred list and will be retried later. > > > > At this point __device_attach_driver() should stop trying other drivers as > > we have "matched" this driver and already scheduled another probe to > > happen later. > > > > Return the -EPROBE_DEFER from really_probe() instead of squashing it to > > zero. This is similar to the code at the top of the function which > > directly returns -EPROBE_DEFER. > > > > It is not really a bug as, AFAIK, we don't actually have cases where > > multiple drivers can bind. > > We _do_ have devices that multiple drivers can bind to. Are you sure > you did not just break them? I asked Cornelia Huck who added this code if she knew who was using it and she said she added it but never ended up using it. Can you point at where there are multiple drivers matching the same device? If multiple drivers are matchable what creates determinism in which will bind? And yes, this would break devices with multiple drivers that are using EPROBE_DEFER to set driver bind ordering. Do you know of any place doing that? > Why are you needing to change this? What is it helping? What problem > is this solving? This special flow works by returning 'success' when the driver bind actually failed. This is OK in the one call site that continues to scan but is not OK in the other callsites that don't keep scanning and want to see error codes. So after the later patches this becomes quite a bit more complicated to keep implemented as-is. Better to delete it if it is safe to delete. Jason
next prev parent reply other threads:[~2021-06-08 12:17 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-08 0:55 [PATCH 00/10] Allow mdev drivers to directly create the vfio_device Jason Gunthorpe 2021-06-08 0:55 ` [Intel-gfx] " Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 01/10] driver core: Do not continue searching for drivers if deferred probe is used Jason Gunthorpe 2021-06-08 5:51 ` Christoph Hellwig 2021-06-08 6:44 ` Greg Kroah-Hartman 2021-06-08 12:16 ` Jason Gunthorpe [this message] 2021-06-08 13:13 ` Greg Kroah-Hartman 2021-06-08 13:53 ` Jason Gunthorpe 2021-06-08 7:35 ` Greg Kroah-Hartman 2021-06-08 12:17 ` Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 02/10] driver core: Pull required checks into driver_probe_device() Jason Gunthorpe 2021-06-08 5:59 ` Christoph Hellwig 2021-06-08 12:21 ` Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 03/10] driver core: Flow the return code from ->probe() through to sysfs bind Jason Gunthorpe 2021-06-08 6:07 ` Christoph Hellwig 2021-06-08 23:53 ` Jason Gunthorpe 2021-06-08 6:47 ` Greg Kroah-Hartman 2021-06-08 12:30 ` Jason Gunthorpe 2021-06-08 13:16 ` Greg Kroah-Hartman 2021-06-08 14:03 ` Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 04/10] driver core: Don't return EPROBE_DEFER to userspace during " Jason Gunthorpe 2021-06-08 6:14 ` Christoph Hellwig 2021-06-08 7:37 ` Greg Kroah-Hartman 2021-06-08 0:55 ` [PATCH 05/10] driver core: Export device_driver_attach() Jason Gunthorpe 2021-06-08 6:19 ` Christoph Hellwig 2021-06-08 12:33 ` Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 06/10] vfio/mdev: Remove CONFIG_VFIO_MDEV_DEVICE Jason Gunthorpe 2021-06-08 0:55 ` [Intel-gfx] " Jason Gunthorpe 2021-06-08 6:20 ` Christoph Hellwig 2021-06-08 6:20 ` [Intel-gfx] " Christoph Hellwig 2021-06-11 12:40 ` Cornelia Huck 2021-06-11 12:40 ` [Intel-gfx] " Cornelia Huck 2021-06-14 12:35 ` Jason Gunthorpe 2021-06-14 12:35 ` [Intel-gfx] " Jason Gunthorpe 2021-06-14 12:35 ` Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 07/10] vfio/mdev: Allow the mdev_parent_ops to specify the device driver to bind Jason Gunthorpe 2021-06-08 6:21 ` Christoph Hellwig 2021-06-08 0:55 ` [PATCH 08/10] vfio/mtty: Convert to use vfio_register_group_dev() Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 09/10] vfio/mdpy: " Jason Gunthorpe 2021-06-08 0:55 ` [PATCH 10/10] vfio/mbochs: " Jason Gunthorpe 2021-06-08 6:22 ` [PATCH 00/10] Allow mdev drivers to directly create the vfio_device Christoph Hellwig 2021-06-08 6:22 ` [Intel-gfx] " Christoph Hellwig 2021-06-14 14:34 ` Kirti Wankhede 2021-06-14 14:34 ` [Intel-gfx] " Kirti Wankhede 2021-06-14 14:34 ` Kirti Wankhede 2021-06-14 14:36 ` Jason Gunthorpe 2021-06-14 14:36 ` [Intel-gfx] " Jason Gunthorpe 2021-06-14 14:36 ` Jason Gunthorpe
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=20210608121654.GX1002214@nvidia.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH 01/10] driver core: Do not continue searching for drivers if deferred probe is used' \ /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
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.