[SCM] strace branch, master, updated. v4.6-333-g31972d5
Denys Vlasenko
dvlasenk at redhat.com
Sat Mar 17 17:01:20 UTC 2012
On 03/17/2012 01:49 PM, Dmitry V. Levin wrote:
> On Sat, Mar 17, 2012 at 12:28:50PM +0000, Denys Vlasenko wrote:
>> commit 31972d52b1059d8faca1c5f417c2db1a90b868ae
>> Author: Denys Vlasenko<vda.linux at googlemail.com>
>> Date: Sat Mar 17 13:23:00 2012 +0100
>>
>> Remove underscores from a few syscall names which have them
>
> I think this change is wrong.
>
>> Affected names are "_newselect", "_llseek", "_sysctl".
>> I see no apparent reason why they have leading underscores.
>> Moreover, some arches have underscored names and some have
>> non-underscored ones. This is not consistent.
>
> The naming consistency does not matter here. It is not our business to
> give syscalls consistent names. All we have to do about syscall names is
> just to follow the kernel. And the kernel still exports some names with
> leading underscores, e.g.
> $ git grep '[[:space:]]_' arch/x86/syscalls/syscall_32.tbl
> arch/x86/syscalls/syscall_32.tbl:140 i386 _llseek sys_llseek
> arch/x86/syscalls/syscall_32.tbl:142 i386 _newselect sys_select compat_sys_select
> arch/x86/syscalls/syscall_32.tbl:149 i386 _sysctl sys_sysctl compat_sys_sysctl
I reverted this change. Sorry.
--
vda
More information about the Strace-devel
mailing list