[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/6] blkdebug: store list of active rules
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 4/6] blkdebug: store list of active rules |
Date: |
Tue, 03 Jul 2012 18:03:19 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Il 03/07/2012 17:40, Kevin Wolf ha scritto:
>> > case ACTION_INJECT_ERROR:
>> > - vars->inject_errno = rule->options.inject.error;
>> > - vars->inject_once = rule->options.inject.once;
>> > - vars->inject_immediately = rule->options.inject.immediately;
>> > + if (!injected) {
>> > + QSIMPLEQ_INIT(&s->active_rules);
>> > + injected = true;
>> > + }
> Does this mean that the next event will invalidate the active rules?
> Currently it only does so if a rule for the new event exists.
No, the point of having the injected bool is exactly to keep the
currently active rules unless a rule exists for the new event. We
cannot just do the QSIMPLEQ_INIT in blkdebug_debug_event, that would be
the behavior you describe.
Paolo