linux-kernel - Re: [PATCH] rhashtable: Improve grammar

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3eb1b3a5-ecdc-4720-b9c2-16a18a3a5e2f@infradead.org>
Date: Fri, 29 Mar 2024 09:28:55 -0700
From: Randy Dunlap <rdunlap@...radead.org>
To: j.neuschaefer@....net, Thomas Graf <tgraf@...g.ch>,
 Herbert Xu <herbert@...dor.apana.org.au>
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] rhashtable: Improve grammar



On 3/29/24 09:26, Jonathan Neuschäfer via B4 Relay wrote:
> From: Jonathan Neuschäfer <j.neuschaefer@....net>
> 
> Change "a" to "an" according to the usual rules, fix an "if" that was
> mistyped as "in", improve grammar in "considerable slow" ->
> "considerably slower".
> 
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@....net>

Reviewed-by: Randy Dunlap <rdunlap@...radead.org>

Thanks.

> ---
>  include/linux/rhashtable.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/include/linux/rhashtable.h b/include/linux/rhashtable.h
> index 5b5357c0bd8c4..8463a128e2f48 100644
> --- a/include/linux/rhashtable.h
> +++ b/include/linux/rhashtable.h
> @@ -633,7 +633,7 @@ static inline struct rhash_head *__rhashtable_lookup(
>   * @params:	hash table parameters
>   *
>   * Computes the hash value for the key and traverses the bucket chain looking
> - * for a entry with an identical key. The first matching entry is returned.
> + * for an entry with an identical key. The first matching entry is returned.
>   *
>   * This must only be called under the RCU read lock.
>   *
> @@ -655,7 +655,7 @@ static inline void *rhashtable_lookup(
>   * @params:	hash table parameters
>   *
>   * Computes the hash value for the key and traverses the bucket chain looking
> - * for a entry with an identical key. The first matching entry is returned.
> + * for an entry with an identical key. The first matching entry is returned.
>   *
>   * Only use this function when you have other mechanisms guaranteeing
>   * that the object won't go away after the RCU read lock is released.
> @@ -682,7 +682,7 @@ static inline void *rhashtable_lookup_fast(
>   * @params:	hash table parameters
>   *
>   * Computes the hash value for the key and traverses the bucket chain looking
> - * for a entry with an identical key.  All matching entries are returned
> + * for an entry with an identical key.  All matching entries are returned
>   * in a list.
>   *
>   * This must only be called under the RCU read lock.
> @@ -699,7 +699,7 @@ static inline struct rhlist_head *rhltable_lookup(
>  }
>  
>  /* Internal function, please use rhashtable_insert_fast() instead. This
> - * function returns the existing element already in hashes in there is a clash,
> + * function returns the existing element already in hashes if there is a clash,
>   * otherwise it returns an error via ERR_PTR().
>   */
>  static inline void *__rhashtable_insert_fast(
> @@ -1130,7 +1130,7 @@ static inline int rhashtable_remove_fast(
>   *
>   * Since the hash chain is single linked, the removal operation needs to
>   * walk the bucket chain upon removal. The removal operation is thus
> - * considerable slow if the hash table is not correctly sized.
> + * considerably slower if the hash table is not correctly sized.
>   *
>   * Will automatically shrink the table if permitted when residency drops
>   * below 30%
> 
> ---
> base-commit: 4cece764965020c22cff7665b18a012006359095
> change-id: 20240329-misc-rhashtable-d7578e8f2ed5
> 
> Best regards,

-- 
#Randy