linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fs/eventpoll.c: fix spelling
@ 2011-01-30 21:42 Daniel Baluta
  2011-02-17 16:38 ` Jiri Kosina
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel Baluta @ 2011-01-30 21:42 UTC (permalink / raw)
  To: viro, trivial; +Cc: linux-fsdevel, linux-kernel, Daniel Baluta

eventpoll.c has wonderful comments but some annoying typos
sneaked in:
	* toepoll_ctl -> to epoll_ctl
	* rapresent -> represents
	* sructure -> structure
	* machanism -> mechanism
	* trasfering -> transferring

Signed-off-by: Daniel Baluta <daniel.baluta@gmail.com>
---
 fs/eventpoll.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/fs/eventpoll.c b/fs/eventpoll.c
index cc8a9b7..68bfc0b 100644
--- a/fs/eventpoll.c
+++ b/fs/eventpoll.c
@@ -62,7 +62,7 @@
  * This mutex is acquired by ep_free() during the epoll file
  * cleanup path and it is also acquired by eventpoll_release_file()
  * if a file has been pushed inside an epoll set and it is then
- * close()d without a previous call toepoll_ctl(EPOLL_CTL_DEL).
+ * close()d without a previous call to epoll_ctl(EPOLL_CTL_DEL).
  * It is possible to drop the "ep->mtx" and to use the global
  * mutex "epmutex" (together with "ep->lock") to have it working,
  * but having "ep->mtx" will make the interface more scalable.
@@ -145,11 +145,11 @@ struct epitem {
 
 /*
  * This structure is stored inside the "private_data" member of the file
- * structure and rapresent the main data sructure for the eventpoll
+ * structure and represents the main data structure for the eventpoll
  * interface.
  */
 struct eventpoll {
-	/* Protect the this structure access */
+	/* Protect the access to this structure */
 	spinlock_t lock;
 
 	/*
@@ -783,7 +783,7 @@ static struct epitem *ep_find(struct eventpoll *ep, struct file *file, int fd)
 
 /*
  * This is the callback that is passed to the wait queue wakeup
- * machanism. It is called by the stored file descriptors when they
+ * mechanism. It is called by the stored file descriptors when they
  * have events to report.
  */
 static int ep_poll_callback(wait_queue_t *wait, unsigned mode, int sync, void *key)
@@ -814,9 +814,9 @@ static int ep_poll_callback(wait_queue_t *wait, unsigned mode, int sync, void *k
 		goto out_unlock;
 
 	/*
-	 * If we are trasfering events to userspace, we can hold no locks
+	 * If we are transferring events to userspace, we can hold no locks
 	 * (because we're accessing user memory, and because of linux f_op->poll()
-	 * semantics). All the events that happens during that period of time are
+	 * semantics). All the events that happen during that period of time are
 	 * chained in ep->ovflist and requeued later on.
 	 */
 	if (unlikely(ep->ovflist != EP_UNACTIVE_PTR)) {
-- 
1.7.1


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

* Re: [PATCH] fs/eventpoll.c: fix spelling
  2011-01-30 21:42 [PATCH] fs/eventpoll.c: fix spelling Daniel Baluta
@ 2011-02-17 16:38 ` Jiri Kosina
  0 siblings, 0 replies; 2+ messages in thread
From: Jiri Kosina @ 2011-02-17 16:38 UTC (permalink / raw)
  To: Daniel Baluta; +Cc: viro, linux-fsdevel, linux-kernel

On Sun, 30 Jan 2011, Daniel Baluta wrote:

> eventpoll.c has wonderful comments but some annoying typos
> sneaked in:
> 	* toepoll_ctl -> to epoll_ctl
> 	* rapresent -> represents
> 	* sructure -> structure
> 	* machanism -> mechanism
> 	* trasfering -> transferring
> 
> Signed-off-by: Daniel Baluta <daniel.baluta@gmail.com>
> ---
>  fs/eventpoll.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/fs/eventpoll.c b/fs/eventpoll.c
> index cc8a9b7..68bfc0b 100644
> --- a/fs/eventpoll.c
> +++ b/fs/eventpoll.c
> @@ -62,7 +62,7 @@
>   * This mutex is acquired by ep_free() during the epoll file
>   * cleanup path and it is also acquired by eventpoll_release_file()
>   * if a file has been pushed inside an epoll set and it is then
> - * close()d without a previous call toepoll_ctl(EPOLL_CTL_DEL).
> + * close()d without a previous call to epoll_ctl(EPOLL_CTL_DEL).
>   * It is possible to drop the "ep->mtx" and to use the global
>   * mutex "epmutex" (together with "ep->lock") to have it working,
>   * but having "ep->mtx" will make the interface more scalable.
> @@ -145,11 +145,11 @@ struct epitem {
>  
>  /*
>   * This structure is stored inside the "private_data" member of the file
> - * structure and rapresent the main data sructure for the eventpoll
> + * structure and represents the main data structure for the eventpoll
>   * interface.
>   */
>  struct eventpoll {
> -	/* Protect the this structure access */
> +	/* Protect the access to this structure */
>  	spinlock_t lock;
>  
>  	/*
> @@ -783,7 +783,7 @@ static struct epitem *ep_find(struct eventpoll *ep, struct file *file, int fd)
>  
>  /*
>   * This is the callback that is passed to the wait queue wakeup
> - * machanism. It is called by the stored file descriptors when they
> + * mechanism. It is called by the stored file descriptors when they
>   * have events to report.
>   */
>  static int ep_poll_callback(wait_queue_t *wait, unsigned mode, int sync, void *key)
> @@ -814,9 +814,9 @@ static int ep_poll_callback(wait_queue_t *wait, unsigned mode, int sync, void *k
>  		goto out_unlock;
>  
>  	/*
> -	 * If we are trasfering events to userspace, we can hold no locks
> +	 * If we are transferring events to userspace, we can hold no locks
>  	 * (because we're accessing user memory, and because of linux f_op->poll()
> -	 * semantics). All the events that happens during that period of time are
> +	 * semantics). All the events that happen during that period of time are
>  	 * chained in ep->ovflist and requeued later on.
>  	 */
>  	if (unlikely(ep->ovflist != EP_UNACTIVE_PTR)) {

Not found in today's linux-next, so I have picked it up.

-- 
Jiri Kosina
SUSE Labs, Novell Inc.

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

end of thread, other threads:[~2011-02-17 16:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-30 21:42 [PATCH] fs/eventpoll.c: fix spelling Daniel Baluta
2011-02-17 16:38 ` Jiri Kosina

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).