Messages in this thread | | | Date | Tue, 25 Jun 2013 11:24:39 +0200 | From | Robert Richter <> | Subject | Re: [PATCH v2 03/14] perf: Add persistent event facilities |
| |
On 25.06.13 09:44:01, Peter Zijlstra wrote: > Elsewhere in this series you use 'pers' to shorten things; it reads a > bit odd to me because 'pers' is the Dutch word for press (both meanings > transfer) but that is just something I'll have to live with isn't it ;-) > > As for tracepoint, it seems common to shorten that to tp; which always > reminds me of toilet paper, but I suppose more people suffer from that. > > Yielding: perf_add_pers_tp() > > which I read as adding pressed toilet paper.. a well :-)
Better we avoid this, don't wonat you misread the code. ;)
I also see 'pers_' not as an optimum since it could be mixed-up easily with 'perf_'. Maybe we take 'persist_' instead?
-Robert
| |