Date: Tue, 13 Sep 2005 17:30:09 -0600
From: Robert Hancock <hancockr@shaw.ca>
Subject: Re: wait_event_interruptible_timeout problem
In-reply-to: <4MjuE-6Cb-37@gated-at.bofh.it>
To: linux-kernel <linux-kernel@vger.kernel.org>
Message-id: <43276101.3020104@shaw.ca>
MIME-version: 1.0
Content-type: text/plain; format=flowed; charset=ISO-8859-1
Content-transfer-encoding: 7bit
References: <4MjuE-6Cb-37@gated-at.bofh.it>
User-Agent: Mozilla Thunderbird 1.0.6 (Windows/20050716)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1437
Lines: 43

manomugdha biswas wrote:
>  Hi,
>  I have a kernel module (kernel 2.6) where I have
>  opened multiple tcp connections. when there is no
>  data
>  i want my process to sleep. For that i have added
>  the
>  following code.
>   
>  /* Initialise the wait q head */
>    init_waitqueue_head(&VNICClientWQHead);
>  
>  init_waitqueue_entry(&(currentMap->waitQ), current);
>  add_wait_queue(currentMap->sock->sk->sk_sleep,
>  &(currentMap->waitQ));
>  
>  /* here currentMap is a structure containing tcp
>  conenction info for my module. There is a currentMap
>  for each tcp connection */
>  
>  wait_event_interruptible_timeout(VNICClientWQHead,
>                                   0, HZ * 100000);
>  
>  I am not sure about the condition argument, 0. 

The condition should be an expression that returns true when whatever 
you are waiting for occurs - in this case when data is available. If you 
put the condition as 0 it will never wake up.

Also, why the huge timeout? If you just want to sleep forever, use 
regular wait_event_interruptible.

-- 
Robert Hancock      Saskatoon, SK, Canada
To email, remove "nospam" from hancockr@nospamshaw.ca
Home Page: http://www.roberthancock.com/


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