On Tue, 2008-08-12 at 14:04 +0100, David Woodhouse wrote: > > Or just match the check before -EOPNOTSUPP with bio_has_data(), > > since it only applies to a barrier that carries data. > > Like this, you mean? Empty barriers don't get to there? Seems to work, although I'm somewhat dubious about it. Still, if you're happy that it's correct and you really prefer it that way, then I can commit it. Anything else I need to address before you pull the tree? Any comment on the BLKDISCARD ioctl? I've left that one using the non-barrier version since it's waiting for it anyway, and shouldn't be happening concurrently with anything else. (http://,git://} git.infradead.org/users/dwmw2/discard-2.6.git -- David Woodhouse Open Source Technology Centre David.Woodhouse@xxxxxxxxx Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html