On Mon, 7 Sep 2020 16:46:01 +0300 Moshe Shemesh wrote:

quoted

In that sense I don't like --live because it doesn't really say much.
AFAIU it means 1) no link flap; 2) < 2 sec datapath downtime; 3) no
configuration is lost in kernel or device (including netdev config,
link config, flow rules, counters etc.). I was hoping at least the
documentation in patch 14 would be more precise.  

Actually, while writing "no-reset" or "live-patching" I meant also no 
downtime at all and nothing resets (config, rules ... anything), that 
fits mlx5 live-patching.

However, to make it more generic,  I can allow few seconds downtime and 
add similar constrains as you mentioned here to "no-reset". I will add 
that to the documentation patch.

Oh! If your device supports no downtime and packet loss at all that's
great. You don't have to weaken the definition now, whoever needs a
weaker definition can add a different constraint level later, no?