15.01.2019 18:55, Eric Blake wrote: > On 1/15/19 9:52 AM, Vladimir Sementsov-Ogievskiy wrote: > >>>>> -nbd_opt_meta_request(const char *context, const char *export) >>>>> "Requesting to set meta context %s for export %s" >>>>> +nbd_opt_meta_request(const char *optname, const char *context, const >>>>> char *export) "Requesting %s %s for export %s" >>>> >>>> Hmm, you forget nbd_opt_lookup() >>> >>> Where? The updated trace point at [2] has nbd_opt_lookup() for >>> determining optname. >> >> Your morning is my evening) >> >> Yes, it's ok, and in the next patch too. But you usually trace both number + >> lookup-string, >> and here only string. Is there a reason to? > > I didn't see any added value in tracing the number, since the trace > point can only happen on one of two opt values (the number is more > important when the opt value can be anything, including one that does > not have a corresponding string name compiled in). >
OK, agreed. -- Best regards, Vladimir