WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] linux: adjust Solarflare Kconfig additions

To: Jan Beulich <jbeulich@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] linux: adjust Solarflare Kconfig additions
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Thu, 21 Feb 2008 09:55:36 +0000
Delivery-date: Thu, 21 Feb 2008 01:56:11 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <47BD57C8.76E4.0078.0@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Ach0b+fYJo35WOBjEdyH3gAX8io7RQ==
Thread-topic: [Xen-devel] [PATCH] linux: adjust Solarflare Kconfig additions
User-agent: Microsoft-Entourage/11.3.6.070618
Oh, that's fine, so long as the existing defconfigs can be used without
requiring manual intervention.

 -- Keir

On 21/2/08 09:51, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> I don't think so, unless you require them to go through a 'make oldconfig'
> entirely unmodified. The patch just modifies ordering (and
> dependencies), so the present settings (all SFC options off) should work
> fine (with SFC off, the line
> 
> # CONFIG_XEN_NETDEV_ACCEL_SFC_BACKEND is not set
> 
> would be removed by a subsequent 'make oldconfig', and
> 
> # CONFIG_XEN_NETDEV_ACCEL_SFC_UTIL is not set
> 
> is also not being generated in 2.6.24 anymore - not sure about older
> kernel versions, though).
> 
> Jan
> 
>>>> Keir Fraser <Keir.Fraser@xxxxxxxxxxxx> 21.02.08 10:21 >>>
> Does this require changing our defconfig files?
> 
>  -- Keir
> 
> On 21/2/08 09:01, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:
> 
>> Cleanup drivers/xen/Kconfig after the Solarflare additions:
>> - placement of new items should not disturb menu hierarchy
>> - dependencies of XEN_NETDEV_ACCEL_SFC_BACKEND were missing
>> - use tabs for indentation
>> 
>> As usual, written and tested on 2.6.24.1 and made apply to the 2.6.18
>> tree without further testing.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
>> 
>> Index: head-2008-02-20/drivers/xen/Kconfig
>> ===================================================================
>> --- head-2008-02-20.orig/drivers/xen/Kconfig 2008-02-20 15:30:01.000000000
>> +0100
>> +++ head-2008-02-20/drivers/xen/Kconfig 2008-02-20 15:32:18.000000000 +0100
>> @@ -30,6 +30,9 @@ config XEN_XENBUS_DEV
>> def_bool y
>> depends on PROC_FS
>>  
>> +config XEN_NETDEV_ACCEL_SFC_UTIL
>> + tristate
>> +
>>  config XEN_BACKEND
>>          tristate "Backend driver support"
>>          default XEN_PRIVILEGED_GUEST
>> @@ -79,15 +82,11 @@ config XEN_NETDEV_PIPELINED_TRANSMITTER
>>  are unsure; or if you experience network hangs when this option is
>>  enabled; then you must say N here.
>>  
>> -config XEN_NETDEV_ACCEL_SFC_UTIL
>> -        tristate
>> -        default n
>> -
>>  config XEN_NETDEV_ACCEL_SFC_BACKEND
>> tristate "Network-device backend driver acceleration for Solarflare NICs"
>> -        depends on XEN_NETDEV_BACKEND
>> -        select XEN_NETDEV_ACCEL_SFC_UTIL
>> -        default m
>> + depends on SFC && SFC_RESOURCE && XEN_NETDEV_BACKEND
>> + select XEN_NETDEV_ACCEL_SFC_UTIL
>> + default m
>>  
>>  config XEN_NETDEV_LOOPBACK
>> tristate "Network-device loopback driver"
>> @@ -185,6 +184,12 @@ config XEN_NETDEV_FRONTEND
>>  dedicated device-driver domain, or your master control domain
>>  (domain 0), then you almost certainly want to say Y here.
>>  
>> +config XEN_NETDEV_ACCEL_SFC_FRONTEND
>> + tristate "Network-device frontend driver acceleration for Solarflare NICs"
>> + depends on XEN_NETDEV_FRONTEND
>> + select XEN_NETDEV_ACCEL_SFC_UTIL
>> + default m
>> +
>>  config XEN_GRANT_DEV
>> tristate "User-space granted page access driver"
>> default XEN_PRIVILEGED_GUEST
>> @@ -192,12 +197,6 @@ config XEN_GRANT_DEV
>>  Device for accessing (in user-space) pages that have been granted
>>  by other domains.
>>  
>> -config XEN_NETDEV_ACCEL_SFC_FRONTEND
>> - tristate "Network-device frontend driver acceleration for Solarflare NICs"
>> - depends on XEN_NETDEV_FRONTEND
>> -        select XEN_NETDEV_ACCEL_SFC_UTIL
>> - default m
>> -
>>  config XEN_FRAMEBUFFER
>> tristate "Framebuffer-device frontend driver"
>> depends on FB
>> 
>> 
>> 
>> 
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
> 
> 
> 



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