Re: [PATCH 2/3] kcov: prefault the kcov_area

From: Mark Rutland
Date: Fri May 04 2018 - 10:38:14 EST


On Fri, May 04, 2018 at 05:36:49PM +0300, Andrey Ryabinin wrote:
>
>
> 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.
>
> > +}
> > +

I'll remove the braces.

Can I consider that an ack, otherwise? :)

Thanks,
Mark.