[libvirt] [PATCH] util: Initialize hooks at daemon shutdown if no hooks defined

Osier Yang jyang at redhat.com
Thu Apr 28 06:50:12 UTC 2011


于 2011年04月28日 11:15, Eric Blake 写道:
> On 04/25/2011 04:28 AM, Osier Yang wrote:
>> We support to initialize the hooks at daemon reload if there is no
>> hooks script is defined, should we also support initialize the hooks
>
> s/should we/we should/
>
>> at daemon shutdown if no hooks is defined?
>
> s/\?/./
>
> (that is, frame this as a statement, not a question)
>
>>
>> To address bz: https://bugzilla.redhat.com/show_bug.cgi?id=688859
>> ---
>>   src/util/hooks.c |    3 ++-
>>   1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/util/hooks.c b/src/util/hooks.c
>> index a409d77..30e20ac 100644
>> --- a/src/util/hooks.c
>> +++ b/src/util/hooks.c
>> @@ -209,7 +209,8 @@ virHookCall(int driver, const char *id, int op, int sub_op, const char *extra,
>>        */
>>       if ((virHooksFound == -1) ||
>>           ((driver == VIR_HOOK_DRIVER_DAEMON)&&
>> -         (op == VIR_HOOK_DAEMON_OP_RELOAD)))
>> +         (op == VIR_HOOK_DAEMON_OP_RELOAD ||
>> +         op == VIR_HOOK_DAEMON_OP_SHUTDOWN)))
>>           virHookInitialize();
>
> ACK; looks like a reasonable fix for the mentioned bugzilla.
>

Thanks, applied with changing on the patch description.

Regards
Osier



More information about the libvir-list mailing list