Re: [PATCH v2 7/7] dm snapshot: use bufio prefetch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 13 2014 at  6:45pm -0500,
Mikulas Patocka <mpatocka@xxxxxxxxxx> wrote:

> 
> 
> On Mon, 13 Jan 2014, Mike Snitzer wrote:
> 
> > On Mon, Jan 13 2014 at  5:00pm -0500,
> > Mikulas Patocka <mpatocka@xxxxxxxxxx> wrote:
> > 
> > > No.
> > > 
> > > This changed patch inefficiently loops in bufio_prefetch_chunks for each 
> > > buffer that is read.
> > 
> > Yeah, I overlooked the importance of preserving prefetch_area (which
> > explains the unlikely you had).  But my intent was to make your code
> > less "special"; your if (DM_PREFETCH_CHUNKS) do { } while() code to
> > avoid the extra indentation is pretty ugly.
> > 
> > Given your current code, DM_PREFETCH_CHUNKS is always 12 so why not just
> > remove the check?
> 
> If someone ever changes it to a variable, the condition is there avoid 
> unneeded call to dm_bufio_prefetch. The compiler optimizes out the 
> constant expression, so it doesn't matter that it's there.

How about move the conditional inside the do { } ?

I really dislike the style you've used on this, is there anywhere else
in the kernel that does this?

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel




[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux