Messages in this thread | | | Subject | Re: [PATCH] aoe: adjust ref of head for compound page tails | From | Ed Cashin <> | Date | Wed, 7 Aug 2013 19:48:15 -0400 |
| |
On Aug 7, 2013, at 7:41 PM, Ed Cashin wrote:
> It sounds like it's wrong to give block pages with a zero count, so why not just have aoe BUG_ON(compound_trans_head(bv->page->_count) == 0) until we're sure nobody does that anymore?
Ugh. I goofed the parens and such. I meant,
BUG_ON(compound_trans_head(bv->bv_page)->_count == 0)
... will catch the cases we think should not be occurring.
> If that idea makes sense to you, I will submit a new patch to follow the one under discussion.
-- Ed Cashin ecashin@coraid.com
| |