On 9/9/21 11:45 AM, Jan Beulich wrote: > On 03.09.2021 21:06, Daniel P. Smith wrote: >> SILO implements a few XSM hooks to extended the decision logic beyond >> what is defined in the dummy/default policy. For each of the hooks, it >> falls back to the dummy/default policy. The fall back is done a slight >> round-about way. This commit makes the direct call to the default policy's >> logic, xsm_default_action(). > > Again it's not clear to me what you're finding wrong here. The way it's > done is not as direct as it could be, but going through the extra layer > allows the functions to document things at the same time. You lose not > only that documentation, but also ...
It is only for six calls, thus I figured the slight overhead would be worth cutting out the indirection. If now one is worried about the extra indirection, than I can adjust to call the default's handlers.
