On 08/01/14 01:29, David Miller wrote:

quoted

+static inline int tx_dealloc_work_todo(struct xenvif *vif)

Make this return bool.

Done, also in the last patch.

quoted

+		wait_event_interruptible(vif->dealloc_wq,
+					tx_dealloc_work_todo(vif) ||
+					 kthread_should_stop());

Inconsistent indentation.  You should make the arguments line up at
exactly the first column after the openning parenthesis of the function
call.

Done, thanks.

Zoli