[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2389e204-1b78-00a8-d69d-0ca4969be1fd@virtuozzo.com>
Date: Fri, 4 May 2018 17:36:49 +0300
From: Andrey Ryabinin <aryabinin@...tuozzo.com>
To: Mark Rutland <mark.rutland@....com>, linux-kernel@...r.kernel.org
Cc: akpm@...ux-foundation.org, dvyukov@...gle.com, mingo@...hat.com,
peterz@...radead.org
Subject: Re: [PATCH 2/3] kcov: prefault the kcov_area
On 05/04/2018 04:55 PM, Mark Rutland wrote:
>
> +static void kcov_fault_in_area(struct kcov *kcov)
> +{
> + unsigned long stride = PAGE_SIZE / sizeof(unsigned long);
> + unsigned long *area = kcov->area;
> + unsigned long offset;
> +
> + for (offset = 0; offset < kcov->size; offset += stride) {
> + READ_ONCE(area[offset]);
> + }
Usually we don't use {} for a single statement blocks.
> +}
> +
Powered by blists - more mailing lists