Messages in this thread | | | Date | Thu, 4 Nov 2021 07:08:49 -0400 | Subject | Re: [RFC PATCH v3 0/3] Introduce BPF map tracing capability | From | Jamal Hadi Salim <> |
| |
On 2021-11-04 06:59, Jamal Hadi Salim wrote: > On 2021-11-03 13:12, Joe Burton wrote: >> That's a good point. Since the probe is invoked before the update takes >> place, it would not be possible to account for the possibility that the >> update failed. >> >> Unless someone wants the `pre update' hook, I'll simply adjust the >> existing hooks' semantics so that they are invoked after the update. >> As discussed, this better suits the intended use case. >> > > If the goal is to synchronize state between two maps (if i understood > correctly the intent) then it is more useful to go post-update. >
To complete that thought: Only positive results are interesting. For example if the command was to delete an entry which doesnt exist there is no point in reporting that (or is there?). OTOH, a successful delete is useful...
cheers, jamal
| |