lkml.org 
[lkml]   [2014]   [Jul]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net-next 1/3 v4] pktgen: Fill the payload optionally with a pattern
On 08/07/14 19:28, David Miller wrote:
> From: Zoltan Kiss <zoltan.kiss@citrix.com>
> Date: Wed, 2 Jul 2014 20:54:13 +0100
>
>> @@ -1129,11 +1133,13 @@ static ssize_t pktgen_if_write(struct file *file,
>> i += len;
>>
>> if (node_possible(value)) {
>> + int j;
>> pkt_dev->node = value;
>> sprintf(pg_result, "OK: node=%d", pkt_dev->node);
>> - if (pkt_dev->page) {
>> - put_page(pkt_dev->page);
>> - pkt_dev->page = NULL;
>> + for (j = 0; j < MAX_SKB_FRAGS; ++j)
>> + if (pkt_dev->pages[j]) {
>> + put_page(pkt_dev->pages[j]);
>> + pkt_dev->pages[j] = NULL;
>> }
>> }
>
> I'm scratching my head with the bracing created here, and it's because
> the indentation is wrong:
>
> for () /* <-- no openning curly brace */
> if () {
> x;
> y;
> } /* <-- thus, wrong indentation */

Hm, checkpatch couldn't catch this, I'll report it to the maintainers.

Zoli



\
 
 \ /
  Last update: 2014-07-09 19:41    [W:0.048 / U:1.472 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site