Linux Kernel Mentees Archive on lore.kernel.org
 help / color / Atom feed
* [Linux-kernel-mentees] [PATCH] include: linux: rculist_nulls: Change docbook comment headers
@ 2019-12-05 18:53 madhuparnabhowmik04
  2019-12-06  0:04 ` Paul E. McKenney
  0 siblings, 1 reply; 3+ messages in thread
From: madhuparnabhowmik04 @ 2019-12-05 18:53 UTC (permalink / raw)
  To: paulmck, josh, joel; +Cc: rcu, linux-kernel-mentees, linux-kernel

From: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>

This patch changes the docbook comment "head for your list"
to "head of the list".

Signed-off-by: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>
---
 include/linux/rculist_nulls.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/linux/rculist_nulls.h b/include/linux/rculist_nulls.h
index 517a06f36c7a..bea311c884b3 100644
--- a/include/linux/rculist_nulls.h
+++ b/include/linux/rculist_nulls.h
@@ -104,7 +104,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,
  * hlist_nulls_for_each_entry_rcu - iterate over rcu list of given type
  * @tpos:	the type * to use as a loop cursor.
  * @pos:	the &struct hlist_nulls_node to use as a loop cursor.
- * @head:	the head for your list.
+ * @head:	the head of the list.
  * @member:	the name of the hlist_nulls_node within the struct.
  *
  * The barrier() is needed to make sure compiler doesn't cache first element [1],
@@ -124,7 +124,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,
  *   iterate over list of given type safe against removal of list entry
  * @tpos:	the type * to use as a loop cursor.
  * @pos:	the &struct hlist_nulls_node to use as a loop cursor.
- * @head:	the head for your list.
+ * @head:	the head of the list.
  * @member:	the name of the hlist_nulls_node within the struct.
  */
 #define hlist_nulls_for_each_entry_safe(tpos, pos, head, member)		\
-- 
2.17.1

_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Linux-kernel-mentees] [PATCH] include: linux: rculist_nulls: Change docbook comment headers
  2019-12-05 18:53 [Linux-kernel-mentees] [PATCH] include: linux: rculist_nulls: Change docbook comment headers madhuparnabhowmik04
@ 2019-12-06  0:04 ` Paul E. McKenney
  2019-12-06  7:23   ` Madhuparna Bhowmik
  0 siblings, 1 reply; 3+ messages in thread
From: Paul E. McKenney @ 2019-12-06  0:04 UTC (permalink / raw)
  To: madhuparnabhowmik04; +Cc: joel, rcu, linux-kernel-mentees, josh, linux-kernel

On Fri, Dec 06, 2019 at 12:23:52AM +0530, madhuparnabhowmik04@gmail.com wrote:
> From: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>
> 
> This patch changes the docbook comment "head for your list"
> to "head of the list".
> 
> Signed-off-by: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>

I applied both with some updates to the subject line and commit log.
Could you please double-check for errors on my part?

							Thanx, Paul

> ---
>  include/linux/rculist_nulls.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/linux/rculist_nulls.h b/include/linux/rculist_nulls.h
> index 517a06f36c7a..bea311c884b3 100644
> --- a/include/linux/rculist_nulls.h
> +++ b/include/linux/rculist_nulls.h
> @@ -104,7 +104,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,
>   * hlist_nulls_for_each_entry_rcu - iterate over rcu list of given type
>   * @tpos:	the type * to use as a loop cursor.
>   * @pos:	the &struct hlist_nulls_node to use as a loop cursor.
> - * @head:	the head for your list.
> + * @head:	the head of the list.
>   * @member:	the name of the hlist_nulls_node within the struct.
>   *
>   * The barrier() is needed to make sure compiler doesn't cache first element [1],
> @@ -124,7 +124,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,
>   *   iterate over list of given type safe against removal of list entry
>   * @tpos:	the type * to use as a loop cursor.
>   * @pos:	the &struct hlist_nulls_node to use as a loop cursor.
> - * @head:	the head for your list.
> + * @head:	the head of the list.
>   * @member:	the name of the hlist_nulls_node within the struct.
>   */
>  #define hlist_nulls_for_each_entry_safe(tpos, pos, head, member)		\
> -- 
> 2.17.1
> 
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Linux-kernel-mentees] [PATCH] include: linux: rculist_nulls: Change docbook comment headers
  2019-12-06  0:04 ` Paul E. McKenney
@ 2019-12-06  7:23   ` Madhuparna Bhowmik
  0 siblings, 0 replies; 3+ messages in thread
From: Madhuparna Bhowmik @ 2019-12-06  7:23 UTC (permalink / raw)
  To: paulmck; +Cc: Joel Fernandes, rcu, linux-kernel-mentees, josh, linux-kernel

[-- Attachment #1.1: Type: text/plain, Size: 2144 bytes --]

On Fri, Dec 6, 2019 at 5:34 AM Paul E. McKenney <paulmck@kernel.org> wrote:

> On Fri, Dec 06, 2019 at 12:23:52AM +0530, madhuparnabhowmik04@gmail.com
> wrote:
> > From: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>
> >
> > This patch changes the docbook comment "head for your list"
> > to "head of the list".
> >
> > Signed-off-by: Madhuparna Bhowmik <madhuparnabhowmik04@gmail.com>
>
> I applied both with some updates to the subject line and commit log.
> Could you please double-check for errors on my part?


 Yes, I checked, it seems alright.

Thank you,
Madhuparna


>                                                         Thanx, Paul
>
> > ---
> >  include/linux/rculist_nulls.h | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/include/linux/rculist_nulls.h
> b/include/linux/rculist_nulls.h
> > index 517a06f36c7a..bea311c884b3 100644
> > --- a/include/linux/rculist_nulls.h
> > +++ b/include/linux/rculist_nulls.h
> > @@ -104,7 +104,7 @@ static inline void hlist_nulls_add_head_rcu(struct
> hlist_nulls_node *n,
> >   * hlist_nulls_for_each_entry_rcu - iterate over rcu list of given type
> >   * @tpos:    the type * to use as a loop cursor.
> >   * @pos:     the &struct hlist_nulls_node to use as a loop cursor.
> > - * @head:    the head for your list.
> > + * @head:    the head of the list.
> >   * @member:  the name of the hlist_nulls_node within the struct.
> >   *
> >   * The barrier() is needed to make sure compiler doesn't cache first
> element [1],
> > @@ -124,7 +124,7 @@ static inline void hlist_nulls_add_head_rcu(struct
> hlist_nulls_node *n,
> >   *   iterate over list of given type safe against removal of list entry
> >   * @tpos:    the type * to use as a loop cursor.
> >   * @pos:     the &struct hlist_nulls_node to use as a loop cursor.
> > - * @head:    the head for your list.
> > + * @head:    the head of the list.
> >   * @member:  the name of the hlist_nulls_node within the struct.
> >   */
> >  #define hlist_nulls_for_each_entry_safe(tpos, pos, head, member)
>      \
> > --
> > 2.17.1
> >
>
[-- Attachment #1.2: Type: text/html, Size: 3598 bytes --]

<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 6, 2019 at 5:34 AM Paul E. McKenney &lt;<a href="mailto:paulmck@kernel.org">paulmck@kernel.org</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Dec 06, 2019 at 12:23:52AM +0530, <a href="mailto:madhuparnabhowmik04@gmail.com" target="_blank">madhuparnabhowmik04@gmail.com</a> wrote:<br>
&gt; From: Madhuparna Bhowmik &lt;<a href="mailto:madhuparnabhowmik04@gmail.com" target="_blank">madhuparnabhowmik04@gmail.com</a>&gt;<br>
&gt; <br>
&gt; This patch changes the docbook comment &quot;head for your list&quot;<br>
&gt; to &quot;head of the list&quot;.<br>
&gt; <br>
&gt; Signed-off-by: Madhuparna Bhowmik &lt;<a href="mailto:madhuparnabhowmik04@gmail.com" target="_blank">madhuparnabhowmik04@gmail.com</a>&gt;<br>
<br>
I applied both with some updates to the subject line and commit log.<br>
Could you please double-check for errors on my part?</blockquote><div> </div><div> Yes, I checked, it seems alright. </div><div><br></div><div>Thank you,</div><div>Madhuparna</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                                                        Thanx, Paul<br>
<br>
&gt; ---<br>
&gt;  include/linux/rculist_nulls.h | 4 ++--<br>
&gt;  1 file changed, 2 insertions(+), 2 deletions(-)<br>
&gt; <br>
&gt; diff --git a/include/linux/rculist_nulls.h b/include/linux/rculist_nulls.h<br>
&gt; index 517a06f36c7a..bea311c884b3 100644<br>
&gt; --- a/include/linux/rculist_nulls.h<br>
&gt; +++ b/include/linux/rculist_nulls.h<br>
&gt; @@ -104,7 +104,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,<br>
&gt;   * hlist_nulls_for_each_entry_rcu - iterate over rcu list of given type<br>
&gt;   * @tpos:    the type * to use as a loop cursor.<br>
&gt;   * @pos:     the &amp;struct hlist_nulls_node to use as a loop cursor.<br>
&gt; - * @head:    the head for your list.<br>
&gt; + * @head:    the head of the list.<br>
&gt;   * @member:  the name of the hlist_nulls_node within the struct.<br>
&gt;   *<br>
&gt;   * The barrier() is needed to make sure compiler doesn&#39;t cache first element [1],<br>
&gt; @@ -124,7 +124,7 @@ static inline void hlist_nulls_add_head_rcu(struct hlist_nulls_node *n,<br>
&gt;   *   iterate over list of given type safe against removal of list entry<br>
&gt;   * @tpos:    the type * to use as a loop cursor.<br>
&gt;   * @pos:     the &amp;struct hlist_nulls_node to use as a loop cursor.<br>
&gt; - * @head:    the head for your list.<br>
&gt; + * @head:    the head of the list.<br>
&gt;   * @member:  the name of the hlist_nulls_node within the struct.<br>
&gt;   */<br>
&gt;  #define hlist_nulls_for_each_entry_safe(tpos, pos, head, member)             \<br>
&gt; -- <br>
&gt; 2.17.1<br>
&gt; <br>
</blockquote></div></div><div hspace="streak-pt-mark" style="max-height:1px"><img alt="" style="width:0px;max-height:0px;overflow:hidden" src="https://mailfoogae.appspot.com/t?sender=abWFkaHVwYXJuYWJob3dtaWswNEBnbWFpbC5jb20%3D&amp;type=zerocontent&amp;guid=2898abe1-2c32-4cb4-a621-66d629dc236d"><font color="#ffffff" size="1">ᐧ</font></div>

[-- Attachment #2: Type: text/plain, Size: 201 bytes --]

_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-05 18:53 [Linux-kernel-mentees] [PATCH] include: linux: rculist_nulls: Change docbook comment headers madhuparnabhowmik04
2019-12-06  0:04 ` Paul E. McKenney
2019-12-06  7:23   ` Madhuparna Bhowmik

Linux Kernel Mentees Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-kernel-mentees/0 linux-kernel-mentees/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-kernel-mentees linux-kernel-mentees/ https://lore.kernel.org/linux-kernel-mentees \
		linux-kernel-mentees@lists.linuxfoundation.org
	public-inbox-index linux-kernel-mentees

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.linuxfoundation.lists.linux-kernel-mentees


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git