lkml.org 
[lkml]   [2022]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/4] kvm: Merge "atomic" and "write" in __gfn_to_pfn_memslot()
On Tue, Jun 28, 2022 at 02:52:04PM -0700, John Hubbard wrote:
> On 6/28/22 12:46, Peter Xu wrote:
> > I'd try to argu with "I prefixed it with kvm_", but oh well.. yes they're a
> > bit close :)
> >
> > >
> > > Yes, "read the code", but if you can come up with a better TLA than GTP
> > > here, let's consider using it.
> >
> > Could I ask what's TLA? Any suggestions on the abbrev, btw?
>
> "Three-Letter Acronym". I love "TLA" because the very fact that one has
> to ask what it means, shows why using them makes it harder to communicate. :)

Ha!

>
> As for alternatives, here I'll demonstrate that "GTP" actually is probably
> better than anything else I can come up with, heh. Brainstorming:
>
> * GPN ("Guest pfn to pfn")
> * GTPN (similar, but with a "T" for "to")
> * GFNC ("guest frame number conversion")

Always a challenge on the naming kongfu. :-D

One good thing on using TLA in macros, flags and codes (rather than simply
mention some three letters): we can easily jump to the label of any of the
flags when we want to figure out what it means, and logically there'll (and
should) be explanations of the abbrev in the headers if it's a good header.

Example: it's even not easy to figure out what GFP is in GFP_KERNEL flag
for someone not familiar with mm (when I wrote this line, I got lost!), but
when that happens we do jump label and at the entry of gfp.h we'll see:

* ... The GFP acronym stands for get_free_pages(),

And if you see the patch I actually did something similar (in kvm_host.h):

/* gfn_to_pfn (gtp) flags */
...

I'd still go for GTP, but let me know if you think any of the above is
better, I can switch.

Thanks,

--
Peter Xu

\
 
 \ /
  Last update: 2022-06-29 00:51    [W:0.065 / U:1.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site