Re: [PATCH 1/3] context_tracking: New context tracking susbsystem

[Frederic Weisbecker]

2012/11/6 Gilad Ben-Yossef <gilad@xxxxxxxxxxxxx>:
> On Sat, Nov 3, 2012 at 6:09 PM, Frederic Weisbecker <fweisbec@xxxxxxxxx> wrote:
>> diff --git a/arch/Kconfig b/arch/Kconfig
>> index 366ec06..3855e06 100644
>> --- a/arch/Kconfig
>> +++ b/arch/Kconfig
>> @@ -300,15 +300,15 @@ config SECCOMP_FILTER
>>
>>           See Documentation/prctl/seccomp_filter.txt for details.
>>
>> -config HAVE_RCU_USER_QS
>> +config HAVE_CONTEXT_TRACKING
>>         bool
>>         help
>> -         Provide kernel entry/exit hooks necessary for userspace
>> +         Provide kernel/user boundaries probes necessary for userspace
>>           RCU extended quiescent state. Syscalls need to be wrapped inside
>
> A minor nit pick: if whole point of the patch is to turn an RCU
> specific mechanism to a generic one
> that RCU happens to use, then the text needs to reflect that. How about:
>
> "Provide kernel/user boundaries probes necessary for subsystems that
> need it, such as userspace
> RCU extended quiescent state. "

Good point! I'm fixing this.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/