Re: [Xen-devel] Re: Konrad devel/next-3.0, pci-passthrough not working: Error: pci: PCI Backend and pci-stub don't own device

[Sander Eikelenboom <linux@xxxxxxxxxxxxxx>]

Wednesday, June 22, 2011, 4:28:13 PM, you wrote:

>> >> >> module name: xen_pciback
>> >> >> driver:      pciback
>> >> >> 
>> >> >> Quite a mix of names, and different use of minus and underscore.
>> >> 
>> >> > Hmm, you are a right. I somehow I thought I backed up the patch that
>> >> > used the wrong name (well, not wrong but just incorrect).
>> >> 
>> >> So if I understand you correct you will revert the driver name from 
>> >> xen-pciback to pciback, so no xen change is needed ?
>> 
>> > So this patch:
>> > f3fc28d52f42220bcce8c2fed12aef2a88c9fadc
>> 
>> >     xen/pciback: revert the name of the PCI driver to 'pciback'.
>> 
>> > should have done the trick but I am not seeing it in the
>> > devel/xen-pciback-0.6.. Aha! I forgot to push the branch up to 
>> > git.kernel.org.
>> 
>> > Before I do that, can you cherry-pick that one please and tell
>> > me if it works? If it does I will stick 'Reported-by' and 'Tested-by'
>> > on the patch if you are OK with that.
>> 
>> And were exactly do i cherry pick that from ? :-) since you say you haven't 
>> pushed it :p

> <smacks his head>
>> If you can attach the patch i will test it right away, sticking the reported 
>> and tested-by is fine !

It does the trick, pci-passthrough works again with this patch !

Currently testing your "master branch (3.0-rc4)"  with the "devel/acpi-cpufreq" 
patches pulled it, together with this pciback patch.

Seems to be working OK, thx again !

--
Sander

> See attached or inline:

> commit f3fc28d52f42220bcce8c2fed12aef2a88c9fadc
> Author: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> Date:   Wed Apr 20 16:06:47 2011 -0400

>     xen/pciback: revert the name of the PCI driver to 'pciback'.
>     
>     At least until the toolstack catches up and it will be able
>     to handle 'xen_pciback'.
>     
>     This means we have these directories in sysfs:
>     
>     ./bus/pci/drivers/pciback
>     ./bus/xen-backend/drivers/xen-pciback
>     ./module/xen_pciback
>     
>     Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>

> diff --git a/drivers/pci/xen-pciback/pci_stub.c 
> b/drivers/pci/xen-pciback/pci_stub.c
> index d802261..aec214a 100644
> --- a/drivers/pci/xen-pciback/pci_stub.c
> +++ b/drivers/pci/xen-pciback/pci_stub.c
> @@ -848,7 +848,9 @@ static struct pci_error_handlers xen_pcibk_error_handler 
> = {
>   */
>  
>  static struct pci_driver xen_pcibk_pci_driver = {
> -       .name = DRV_NAME,
> +       /* The name should be xen_pciback, but until the tools are updated
> +        * we will keep it as pciback. */
> +       .name = "pciback",
>         .id_table = pcistub_ids,
>         .probe = pcistub_probe,
>         .remove = pcistub_remove,



-- 
Best regards,
 Sander                            mailto:linux@xxxxxxxxxxxxxx


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel