diff options
| author | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2010-11-02 11:55:58 -0400 |
|---|---|---|
| committer | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2010-11-02 13:46:46 -0400 |
| commit | dcb8baeceaa1c629bbd06f472cea023ad08a0c33 (patch) | |
| tree | 72145e182cac0705e80553565356db24a41387e6 | |
| parent | be2f8373c188ed1f5d36003c9928e4d695213080 (diff) | |
xen/blkfront: cope with backend that fail empty BLKIF_OP_WRITE_BARRIER requests
Some(?) Xen block backends fail BLKIF_OP_WRITE_BARRIER requests, which
Linux uses as a cache flush operation. In that case, disable use
of FLUSH.
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Cc: Daniel Stodden <daniel.stodden@citrix.com>
| -rw-r--r-- | drivers/block/xen-blkfront.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c index 76b874a79175..4f9e22f29138 100644 --- a/drivers/block/xen-blkfront.c +++ b/drivers/block/xen-blkfront.c | |||
| @@ -656,6 +656,16 @@ static irqreturn_t blkif_interrupt(int irq, void *dev_id) | |||
| 656 | printk(KERN_WARNING "blkfront: %s: write barrier op failed\n", | 656 | printk(KERN_WARNING "blkfront: %s: write barrier op failed\n", |
| 657 | info->gd->disk_name); | 657 | info->gd->disk_name); |
| 658 | error = -EOPNOTSUPP; | 658 | error = -EOPNOTSUPP; |
| 659 | } | ||
| 660 | if (unlikely(bret->status == BLKIF_RSP_ERROR && | ||
| 661 | info->shadow[id].req.nr_segments == 0)) { | ||
| 662 | printk(KERN_WARNING "blkfront: %s: empty write barrier op failed\n", | ||
| 663 | info->gd->disk_name); | ||
| 664 | error = -EOPNOTSUPP; | ||
| 665 | } | ||
| 666 | if (unlikely(error)) { | ||
| 667 | if (error == -EOPNOTSUPP) | ||
| 668 | error = 0; | ||
| 659 | info->feature_flush = 0; | 669 | info->feature_flush = 0; |
| 660 | xlvbd_flush(info); | 670 | xlvbd_flush(info); |
| 661 | } | 671 | } |
