While I agree that the L2ARC logic needs some work, I'm not convinced that this is the right direction, and the change is too risky to take as-is.
With sufficient testing, I would consider a simpler semantic change like "evict blocks in L2ARC first". That would probably require separating the lists into one for blocks in L2ARC (HDR_HAS_L2HDR) and one for blocks not in L2ARC. For now, I'm going to close this PR. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/189#issuecomment-324477018 ------------------------------------------ openzfs-developer Archives: https://openzfs.topicbox.com/groups/developer/discussions/T122a323c2623f54c-Mff1117ee8ed21833f970a72a Powered by Topicbox: https://topicbox.com