"KVM: x86/mmu:" for the scope. And explicitly state what the patch does, not what the symptom is, especially since "duplicate" can be an adjective, noun, or a verb depending on context. And as an almost nit, state that it's a comment so that it's obvious from the shortlog that the patch should have no functional impact. On Thu, Sep 15, 2022, liujing wrote: > in "static int FNAME(walk_addr_generic)" function, > there is an explanation that uses two to's, which is redundant code Again, explicitly state what the patch actually does. All that said, a patch has already been submitted for this[*]. No need to send a v2, I'll grab the previous patch. Thanks! [*] https://lore.kernel.org/all/20220831125217.12313-1-yuanjilin@xxxxxxxxxx
