|
Hello,
On Sun, 10 Mar 2013, David Miller wrote:
> From: Julian Anastasov <ja@xxxxxx>
> Date: Sun, 10 Mar 2013 15:37:34 +0200 (EET)
>
> > The idea looks good, here is the implementation.
> > Can I use it in this form for next patchset versions?
>
> Yes, but with one minor change:
>
> > - if (unlikely(dst->flags & DST_NOCACHE)) {
> > + if (unlikely(dst->flags & DST_NOCACHE && !force)) {
>
> I keep forgetting the && vs. & operator precedence, and so
> do many other people, so please put parenthesis around
> the "dst->flags & DST_NOCACHE" expression, thanks.
Done, I'll wait a week for more feedback...
Regards
--
Julian Anastasov <ja@xxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
