[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Preboot support
|
From: |
Yoshinori K. Okuji |
|
Subject: |
Re: [PATCH] Preboot support |
|
Date: |
Sat, 11 Apr 2009 19:16:05 +0900 |
|
User-agent: |
KMail/1.9.10 |
On Saturday 11 April 2009 08:48:07 phcoder wrote:
> Hello, here is the preboot hooks support. Apply on top of my bootmove
> patch. They are very useful for patches like sendkey (my old patch that
> I'll rediff), badram, acpi (2 patches in separate threads) or drivemap
Some comments about the design.
- register/unregister is more appropriate than add/remove.
- preboot is ambigous. preboot_hook is better.
- Using an int value for the priority is quetionable. Very often, this style
of priority system leads to chaos, because everyone picks up arbitrary
numbers randomly. I prefer to define enums with a careful analysis.
Regards,
Okuji
- [PATCH] Preboot support, phcoder, 2009/04/10
- Re: [PATCH] Preboot support,
Yoshinori K. Okuji <=
- Re: [PATCH] Preboot support, phcoder, 2009/04/11
- Re: [PATCH] Preboot support, phcoder, 2009/04/12
- Re: [PATCH] Preboot support, Yoshinori K. Okuji, 2009/04/14
- Re: [PATCH] Preboot support, phcoder, 2009/04/15
- Re: [PATCH] Preboot support, John Stanley, 2009/04/15
- Re: [PATCH] Preboot support, Vladimir 'phcoder' Serbinenko, 2009/04/26
- Re: [PATCH] Preboot support, Vladimir 'phcoder' Serbinenko, 2009/04/27