Re: [PATCH v3] kprobes: Disable Kprobe when ftrace arming fails

[Jiri Kosina]

On Mon, 23 Mar 2015, Ingo Molnar wrote:

> > > > wth is a 'universe' in this context?
> > > 
> > > We use the term "universe" to define whether the system or task uses
> > > original or patched functions. It is especially important for patches
> > > that modify semantic of functions. They need more complex consistency
> > > model. It defines when it is safe time for the system or task to start
> > > using the new functions (switch to the new universe).
> > > 
> > > In theory, different tasks might be in more universes if more patches are
> > > being applied. In practice, we deal with only two universes. The trick is
> > > that we allow to add new patch only when the whole system has switched
> > > to the previous one.
> > > 
> > 
> > Is this terminology documented anywhere upstream yet?
> 
> Even if it was documented (it isn't), 

There is no point in documenting it upstream, as no upstream code is using 
this "universe" notation. It should just be removed from original Petr's 
changelog and that's it.

> it's pretty weird terminology - please use clearer formulations, like 
> 'patched function' or 'unpatched function' or 'function with pending 
> patch'.

All your suggestions above unfortunately don't really reflect what is 
happening when lazy per-thread migration is applied.

> No need to redefine existing words in a weird fashion just to create the 
> appearance of being special ...

So which particular existing word(s) are you talking about here?

Thanks,

-- 
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/