[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel][PATCH] Dynamic modes support for PV xenfb (0 of 2)



On Wed, Jan 09, 2008 at 02:16:07PM -0700, Pat Campbell wrote:
> >>> On Wed, Jan 9, 2008 at 12:43 PM, in message 
> >>> <20080109194325.GN8732@xxxxxxxxxx>,
> "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote: 
> > On Tue, Jan 08, 2008 at 08:34:31AM -  0700, Pat Campbell wrote:
> >>   New xenstore virtual machine specific VNC attributes:
> >>     vncresizable-  pvfb: For those that don't need higher
> >>       resolutions in their guest. Default: 0.
> > 
> > Is there really any compelling need to make the resize stuff
> > a config option ?  IMHO it should just always be on by default.
> > If a user doesn't want to resize the guest, then they can simply
> > not run the xrandr tool in their guest.
> 
> My thinking was that some people would not want the extra
> 3MB memory hit caused by the larger frame buffer.  If no one
> else has any input as to the 5MB size I will remove that option.

That's a fair point. HOw about having a 'videoram=XXX' config
parameter instead, and have your new resize feature also inform
the guest of the desired RAM allocation. Then we can allow any
resolution that fits in the configured RAM.   A videoram param
is something that's a reasonably generic concept across full and
paravirt and different hypervisors so would fit into nicely to
the UI of management tools

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.