[ovs-dev] [PATCH 1/2] odp-utils: Refactor slow_path_reason parse and format functions.

[Ben Pfaff]

On Tue, Nov 13, 2012 at 05:32:56PM -0800, Pravin B Shelar wrote:
> These functions are used in next patch.
> 
> Signed-off-by: Pravin B Shelar <pshelar at nicira.com>

I don't really understand how one should interpret SLOW_MAX.  Also, it's
not really a member of the enumeration, so probably a #define would be
better.

It looks like something like:
        #define SLOW_ALL ((1 << 6) - 1)
has a more obvious interpretation and would avoid the extra switch cases
for the enum.

I may have other comments later; I didn't read the whole patch yet.
(But if Jesse approves it, that's fine.)