>>> On 05.02.18 at 13:22, <andrew.coop...@citrix.com> wrote:
> On 05/02/18 08:57, Jan Beulich wrote:
>>>>> On 02.02.18 at 17:58, <andrew.coop...@citrix.com> wrote:
>>> To match all our other emulation handling.
>>>
>>> No functional change.
>>>
>>> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
>>> ---
>>> CC: Jan Beulich <jbeul...@suse.com>
>>> ---
>>> xen/arch/x86/pv/Makefile | 2 +-
>>> xen/arch/x86/pv/{ro-page-fault.c => emul-ro-page-fault.c} | 2 +-
>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>> rename xen/arch/x86/pv/{ro-page-fault.c => emul-ro-page-fault.c} (99%)
>> When this file was introduced, iirc I had specifically asked to drop
>> the pointless emul- prefix. If you want to make things consistent
>> again, please instead drop the emul- prefixes of the other files.
>
> No.
>
> First of all, this file is the most recent to come into existence,
> around 3 months after the others.
Right - it was too late for me to realize the needlessly long names
in those earlier code movement patches.
> The point of naming things in a consistent fashion is for the benefit of
> humans, and having the emulation related functionality logically grouped
> is a benefit, not a detriment.
They're all quite well grouped now already by being in pv/. Otherwise
do you mean to also change e.g. gpr_switch.S to emul-gpr_switch.S?
Besides the plain cosmetic aspect, shell command completion is of
more use when there aren't many files with similar prefixes in a single
directory.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
