From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============1996110365832387072=="
MIME-Version: 1.0
From: Sergey Senozhatsky <sergey.senozhatsky at gmail.com>
Subject: Re: [Powertop] [PATCH] devices: Fix processing /proc/net/dev file
Date: Mon, 15 Oct 2012 07:59:21 -0700
Message-ID: <20121015145921.GC3140@swordfish.minsk.epam.com>
In-Reply-To: 1350312990.1507.11.camel@leonhard
To: powertop@lists.01.org
List-ID: <powertop@lists.01.org>

--===============1996110365832387072==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

On (10/15/12 23:56), Namhyung Kim wrote:
> 2012-10-15 (=EC=9B=94), 07:47 -0700, Sergey Senozhatsky:
> > On (10/15/12 23:33), Namhyung Kim wrote:
> > > @@ -114,7 +114,7 @@ static void do_proc_net_dev(void)
> > >  			c2 =3D c;
> > >  			val =3D strtoull(c, &c, 10);
> > >  			i++;
> > > -			if (i =3D=3D 1 || i =3D=3D 10)
> > > +			if (i =3D=3D 2 || i =3D=3D 10)
> > >  				pkt +=3D val;
> > >  =

> > >  		}
> > > -- =

> > =

> > or `i++' after `if (i =3D=3D 1 || i =3D=3D 10)'.
> =

> But if we moved `i++', it should be `if (i =3D=3D 1 || i =3D=3D 9)'.
> =


yeah, right :)

	-ss

--===============1996110365832387072==--