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] xen: fbdev frontend needs xenbus frontend

To: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen: fbdev frontend needs xenbus frontend
From: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Date: Thu, 25 Feb 2010 12:48:08 -0500
Cc: Bastian Blank <waldi@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 25 Feb 2010 10:06:25 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4B86B562.4030009@xxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20100224135828.GA32009@xxxxxxxxxxxxxxxxxxxxxxx> <20100225124047.GA9040@xxxxxxxxxxxxxxxxxxx> <4B86B562.4030009@xxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.19 (2009-01-05)
On Thu, Feb 25, 2010 at 09:37:38AM -0800, Jeremy Fitzhardinge wrote:
> On 02/25/2010 04:40 AM, Konrad Rzeszutek Wilk wrote:
>> On Wed, Feb 24, 2010 at 02:58:28PM +0100, Bastian Blank wrote:
>>    
>>> The fbdev (and kbddev) frontend needs the xenbus frontend, but does not
>>> define a dependency.  Make the fbdev frontend select the xenbus
>>> frontend.
>>>      
>> Isn't this the same as ba69ea42f8ba8286cbe0e939bd1ce781b7905b84? If so
>> (and if the above mention commit isn't in xen/next)
>> shouldn't you also make this change for drives/input/Kconfig for
>> XEN_KBDDEV_FRONTEND?
>>    
>
> Did that end up in .33?  Should we put it in -stable too?


I took a look at the code and I think Bastian's is OK. The -stable
has this:

152 config XEN_KBDDEV_FRONTEND
153         tristate "Xen virtual keyboard and mouse support"
154         depends on XEN_FBDEV_FRONTEND
155         default y

So it would end up selecting the xenbus frontend as well with Bastian's
patch.

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

<Prev in Thread] Current Thread [Next in Thread>