Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1626616 > unrolled thread

[PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND

Started byArnd Bergmann <arnd@arndb.de>
First post2017-04-19 19:20 +0200
Last post2017-04-20 06:00 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND Arnd Bergmann <arnd@arndb.de> - 2017-04-19 19:20 +0200
    Re: [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND Stefano Stabellini <sstabellini@kernel.org> - 2017-04-19 19:30 +0200
      Re: [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND Stefano Stabellini <sstabellini@kernel.org> - 2017-04-20 01:20 +0200
        Re: [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND Juergen Gross <jgross@suse.com> - 2017-04-20 06:00 +0200

#1626616 — [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND

FromArnd Bergmann <arnd@arndb.de>
Date2017-04-19 19:20 +0200
Subject[PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND
Message-ID<ty1qO-6bq-27@gated-at.bofh.it>
All Xen frontends need to select this symbol to avoid a link error:

net/built-in.o: In function `p9_trans_xen_init':
:(.text+0x149e9c): undefined reference to `__xenbus_register_frontend'

Fixes: d4b40a02f837 ("xen/9pfs: build 9pfs Xen transport driver")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 net/9p/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/9p/Kconfig b/net/9p/Kconfig
index 3f286f1bd1d3..e6014e0e51f7 100644
--- a/net/9p/Kconfig
+++ b/net/9p/Kconfig
@@ -24,6 +24,7 @@ config NET_9P_VIRTIO
 
 config NET_9P_XEN
 	depends on XEN
+	select XEN_XENBUS_FRONTEND
 	tristate "9P Xen Transport"
 	help
 	  This builds support for a transport for 9pfs between
-- 
2.9.0

[toc] | [next] | [standalone]


#1626630

FromStefano Stabellini <sstabellini@kernel.org>
Date2017-04-19 19:30 +0200
Message-ID<ty1Au-6ev-21@gated-at.bofh.it>
In reply to#1626616
On Wed, 19 Apr 2017, Arnd Bergmann wrote:
> All Xen frontends need to select this symbol to avoid a link error:
> 
> net/built-in.o: In function `p9_trans_xen_init':
> :(.text+0x149e9c): undefined reference to `__xenbus_register_frontend'
> 
> Fixes: d4b40a02f837 ("xen/9pfs: build 9pfs Xen transport driver")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Thank you for the fix!

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  net/9p/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/net/9p/Kconfig b/net/9p/Kconfig
> index 3f286f1bd1d3..e6014e0e51f7 100644
> --- a/net/9p/Kconfig
> +++ b/net/9p/Kconfig
> @@ -24,6 +24,7 @@ config NET_9P_VIRTIO
>  
>  config NET_9P_XEN
>  	depends on XEN
> +	select XEN_XENBUS_FRONTEND
>  	tristate "9P Xen Transport"
>  	help
>  	  This builds support for a transport for 9pfs between
> -- 
> 2.9.0
> 

[toc] | [prev] | [next] | [standalone]


#1626863

FromStefano Stabellini <sstabellini@kernel.org>
Date2017-04-20 01:20 +0200
Message-ID<ty73c-1b1-9@gated-at.bofh.it>
In reply to#1626630
Juergen, I have committed this patch to for-linus-4.12 and linux-next, I
hope that's OK.

Og Wed, 19 Apr 2017, Stefano Stabellini wrote:
> On Wed, 19 Apr 2017, Arnd Bergmann wrote:
> > All Xen frontends need to select this symbol to avoid a link error:
> > 
> > net/built-in.o: In function `p9_trans_xen_init':
> > :(.text+0x149e9c): undefined reference to `__xenbus_register_frontend'
> > 
> > Fixes: d4b40a02f837 ("xen/9pfs: build 9pfs Xen transport driver")
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> 
> Thank you for the fix!
> 
> Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
> > ---
> >  net/9p/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/net/9p/Kconfig b/net/9p/Kconfig
> > index 3f286f1bd1d3..e6014e0e51f7 100644
> > --- a/net/9p/Kconfig
> > +++ b/net/9p/Kconfig
> > @@ -24,6 +24,7 @@ config NET_9P_VIRTIO
> >  
> >  config NET_9P_XEN
> >  	depends on XEN
> > +	select XEN_XENBUS_FRONTEND
> >  	tristate "9P Xen Transport"
> >  	help
> >  	  This builds support for a transport for 9pfs between
> > -- 
> > 2.9.0
> > 
> 

[toc] | [prev] | [next] | [standalone]


#1626960

FromJuergen Gross <jgross@suse.com>
Date2017-04-20 06:00 +0200
Message-ID<tybqa-3PD-3@gated-at.bofh.it>
In reply to#1626863
On 20/04/17 01:10, Stefano Stabellini wrote:
> Juergen, I have committed this patch to for-linus-4.12 and linux-next, I
> hope that's OK.

Sure.


Juergen

> 
> Og Wed, 19 Apr 2017, Stefano Stabellini wrote:
>> On Wed, 19 Apr 2017, Arnd Bergmann wrote:
>>> All Xen frontends need to select this symbol to avoid a link error:
>>>
>>> net/built-in.o: In function `p9_trans_xen_init':
>>> :(.text+0x149e9c): undefined reference to `__xenbus_register_frontend'
>>>
>>> Fixes: d4b40a02f837 ("xen/9pfs: build 9pfs Xen transport driver")
>>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>>
>> Thank you for the fix!
>>
>> Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
>>> ---
>>>  net/9p/Kconfig | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/net/9p/Kconfig b/net/9p/Kconfig
>>> index 3f286f1bd1d3..e6014e0e51f7 100644
>>> --- a/net/9p/Kconfig
>>> +++ b/net/9p/Kconfig
>>> @@ -24,6 +24,7 @@ config NET_9P_VIRTIO
>>>  
>>>  config NET_9P_XEN
>>>  	depends on XEN
>>> +	select XEN_XENBUS_FRONTEND
>>>  	tristate "9P Xen Transport"
>>>  	help
>>>  	  This builds support for a transport for 9pfs between
>>> -- 
>>> 2.9.0
>>>
>>
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web