grub-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 1/2] Replace hook-based with flag-based approach for choosin


From: Vladimir 'phcoder' Serbinenko
Subject: Re: [PATCH 1/2] Replace hook-based with flag-based approach for choosing video mode
Date: Fri, 21 Aug 2009 14:54:32 +0200

On Mon, Aug 17, 2009 at 3:51 PM, Robert Millan<address@hidden> wrote:
> On Sun, Aug 16, 2009 at 08:51:06PM +0200, Vladimir 'phcoder' Serbinenko wrote:
>> After discussion on IRC it revealed that hook-based approach is
>> unpracticable because number of available modes grows exponentially
>> with every parameter available. Here is the patch to change to
>> flag-based approach
>
> Please can you ellaborate on this?  (if you still have a copy of the
> discussion, it'd be useful to post it)
>
There was no discussion on this point. But current design is to call a
hook with every available mode and see if it approves the mode.
Unfortunately it doesn't work as expected because some modes aren't
presented by driver unless explicitly requested. This may cause
appropriate mode to be missed. Making driver iterate through a lot of
modes will slow down mode switching. With flag-based approach driver
knows which mode upper layer wants and so can present appropriate one.
It has a drawback of limiting number of filtering possibilities but
currently no subsystem needs more than is done with this patch. If it
changes in the future interface can be extended to have new flags or
parameters

-- 
Regards
Vladimir 'phcoder' Serbinenko

Personal git repository: http://repo.or.cz/w/grub2/phcoder.git




reply via email to

[Prev in Thread] Current Thread [Next in Thread]