On 2010-10-11 21:13, Dan Carpenter wrote: > This should pass "buf" to bvec_kunmap_irq() instead of "bv". The api is > like kmap_atomic() instead of kmap(). > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/drivers/block/ps3disk.c b/drivers/block/ps3disk.c > index e9da874..03688c2 100644 > --- a/drivers/block/ps3disk.c > +++ b/drivers/block/ps3disk.c > @@ -113,7 +113,7 @@ static void ps3disk_scatter_gather(struct ps3_storage_device *dev, > memcpy(buf, dev->bounce_buf+offset, size); > offset += size; > flush_kernel_dcache_page(bvec->bv_page); > - bvec_kunmap_irq(bvec, &flags); > + bvec_kunmap_irq(buf, &flags); > i++; > } > } Thanks applied, that bug is all too common. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
