[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [PATCH v3 1/8] target-ppc: Add helper for KVM_PPC_RTAS_DE

From: Alexander Graf
Subject: Re: [Qemu-ppc] [PATCH v3 1/8] target-ppc: Add helper for KVM_PPC_RTAS_DEFINE_TOKEN
Date: Tue, 27 Aug 2013 10:36:33 +0200

On 27.08.2013, at 06:10, Benjamin Herrenschmidt wrote:

> On Tue, 2013-08-27 at 03:48 +0200, Andreas Färber wrote:
>> Also, QEMU is definitely not the only project that has higher
>> acceptance
>> criteria than patch-works-for-the-patch-author. :)
> There's a difference between high acceptance criteria and systematic
> bike shed painting including in some case request to turn reasonably
> meaningful identifiers into something that nobody would get :-)

The name doesn't tell at all what the function is doing. This is even true for 
the kernel ioctl, but that one is in now, so it's there to stay. Heck, I even 
had to look up the API documentation to know whether this sets an RTAS to be 
handled in-kernel or in-user-space.

Just come up with a good name and all is well. Or are you enjoying to complain 
on every patch review I do now?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]