On Thu, Dec 15, 2016 at 09:34:33AM -0500, Brian Foster wrote: > FWIW, I was playing with this a bit more and managed to manufacture a > filesystem layout that this series doesn't handle too well. Emphasis on > "manufactured" because this might not be a likely real world scenario, > but either way the current code handles it fine. It does, although mostly by accident. I suspect with an even better manufcatured image you could also drive the current code to it's knees, e.g. only have one single block free in the first few AGs, and then a small number just higher than that in a higher AG. > I've attached a metadump of the offending image. mdestore it, mount and > attempt something like 'dd if=/dev/zero of=/mnt/file' on the root. The > buffered write looks like it's in a livelock, waiting indefinitely for a > writeback cycle that will never complete... Yeah, that's the loop that keeps going even if it can't allocate any blocks, which seems generally bogus. But even without that we'd get ENOSPC despite not having a reservations. Which is a little easier to debug, but just as wrong. The only good way out I can see is to not hand out any more reservations after we only nave nr_ags * xfs_bmap_worst_indlen(1) available. I'll see if I can come up with a patch for that. -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
