| Date | Wed, 8 Jan 2014 14:08:02 +0000 | From | Zoltan Kiss <> | Subject | Re: [PATCH net-next v3 1/9] xen-netback: Introduce TX grant map definitions |
| |
On 08/01/14 01:29, David Miller wrote: >> +static inline int tx_dealloc_work_todo(struct xenvif *vif) > > Make this return bool. Done, also in the last patch.
>> + 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
|