On 9/5/18 1:33 PM, Logan Gunthorpe wrote: > > > On 05/09/18 01:26 PM, Jens Axboe wrote: >> On 9/3/18 4:26 PM, Logan Gunthorpe wrote: >>> I personally agree with Christoph. But if there's consensus in the other >>> direction or this is a real blocker moving this forward, I can remove it >>> for the next version. >> >> It's a simple branch because the check isn't exhaustive. It just checks >> the first page. At that point you may as well just require the caller to >> flag the bio/rq as being P2P, and then do a check for P2P compatibility >> with the queue. > > Hmm, we had something like that in v4[1] but it just seemed redundant to > create a flag when the information was already in the bio and kind of > ugly for the caller to check for, then set, the flag. I'm not _that_ > averse to going back to that though... The point is that the caller doesn't necessarily know where the bio will end up, hence the caller can't fully check if the whole stack supports P2P. What happens if a P2P request ends up with a driver that doesn't support it? -- Jens Axboe
