linux-kernel - Re: [Xen-devel] [PATCH] xen-pciback: Add MODULE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <201411261718.sAQHIsgd027807@aserz7022.oracle.com>
Date:	Wed, 26 Nov 2014 12:18:48 -0500
From:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
To:	"Luis R. Rodriguez" <mcgrof@...not-panic.com>
Cc:	xen-devel@...ts.xenproject.org, linux-kernel@...r.kernel.org,
	Ian Campbell <ian.campbell@...rix.com>
Subject: Re: [Xen-devel] [PATCH] xen-pciback: Add MODULE_ALIAS for pciback.


On Nov 26, 2014 11:39 AM, "Luis R. Rodriguez" <mcgrof@...not-panic.com> wrote:
>
> On Wed, Aug 20, 2014 at 1:26 PM, Ian Campbell <ian.campbell@...rix.com> wrote: 
> > On Wed, 2014-08-20 at 13:20 -0400, Konrad Rzeszutek Wilk wrote: 
> >> On Wed, Aug 20, 2014 at 06:18:52PM +0100, Ian Campbell wrote: 
> >> > On Wed, 2014-08-20 at 12:40 -0400, Konrad Rzeszutek Wilk wrote: 
> >> > > The rest of the Xen device drivers use an module alias 
> >> > > to load devices when they shop up in XenBus. 
> >> > 
> >> > "show". 
> >> > > 
> >> > >  MODULE_LICENSE("Dual BSD/GPL"); 
> >> > >  MODULE_ALIAS("xen-backend:pci"); 
> >> > > +MODULE_ALIAS("xen:pci"); 
> >> > 
> >> > Isn't that xen-backend:pci already the right thing for a backend device? 
> >> > xen: is for frontends, I thought. 
> >> 
> >> Oh, you are right. Cool! Thanks! 
> > 
> > The patch turned out to be even more trivial than expected ;-) 
>
> Is this what we expected to be pending work for the item "device 
> hotplug (MODULE_ALIAS)" on the upstream TODO list? 
>
> http://wiki.xenproject.org/wiki/XenParavirtOps#Upstream_delta_details 
>
> This was simply to just auto load that driver when needed right? 
>

Right which it does now.

> Also as for actual PCI device hotplug support: 
>
> http://wiki.xen.org/wiki/Xen_PCI_Passthrough#Hotplug 
>
> I don't think we need a delta for that do we? 

Nope. This one is all done.
>
> Luis