From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Daniel P. Berrange" Subject: Re: [PATCH][IOEMU] Dynamic modes support for PV xenfb (2 of 2) Date: Sat, 12 Jan 2008 15:46:35 +0000 Message-ID: <20080112154635.GA12159@redhat.com> References: <47833544.3E48.0018.0@novell.com> <47833544.3E48.0018.0@novell.com> <87ejcqhtku.fsf@pike.pond.sub.org> <47872348.3E48.0018.0@novell.com> <87lk6vscyh.fsf@pike.pond.sub.org> Reply-To: "Daniel P. Berrange" Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <87lk6vscyh.fsf@pike.pond.sub.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Markus Armbruster Cc: xen-devel@lists.xensource.com, Pat Campbell List-Id: xen-devel@lists.xenproject.org On Sat, Jan 12, 2008 at 08:46:46AM +0100, Markus Armbruster wrote: > "Pat Campbell" writes: > > >>>> On Thu, Jan 10, 2008 at 3:18 AM, in message > > <87ejcqhtku.fsf@pike.pond.sub.org>, Markus Armbruster > > wrote: > >> "Pat Campbell" writes: > >> > >>> Attached patch adds multiple frame buffer size support to the xenfb PV > >>> backend QEMU xenfb. Backend sets feature- resize and handles the > >>> resize frame buffer event. > >>> > >>> Corresponding frontend LINUX patch is required for functionality but this > >>> patch is not dependent on it, preserving backwards compatibility. > [...] > >> By the way, the feature negotiation only covers whether the frontend > >> is permitted to resize, not acceptable resolutions. > > > > True. Acceptable resolutions are what fits within a 5MB framebuffer > > and has a width no greater than 1280. > > Is this enough for all eternity? Screen resolutions continue to > grow... If it's not, then how can we ensure that we can enlarge it > easily while maintaining backwards compatibility? > > Perhaps the backend could publish the maximum resolution it can > accept, instead of just a flag that it can accept resize events. Maximum resolutions are not well defined because they vary according to your aspect ratio - normal vs widescreen. I increasingly like the like of having a 'videoram' parameter in the backend config, so the admin can set as large a value as they feel neccessary, and frontend then set whatever resolutions they like within the constraint of that value. Regards, 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 -=|