On Tue, Jun 14, 2011 at 05:05:50PM -0500, Goldwyn Rodrigues wrote: > > Yes sure. I suppose you are refering to the patch attached. > I did not send it upstream because the upstream code had taken a > different approach and the patch was relevant to the SLES kernel > tree only. That, and "XFS: force log before waiting for a pinned buffer". They look similar, but different enough from things that we have in mainline that make me really curious if you a) were hitting the same things, and b) came to different conclusions under different workloads. Looking over it in more detail the bufd patch should be fully superceed by current mainline code, altough less so by the commit you quote, but rather by the LRU-ification of the xfs_buf code. Similarly the other one you quote isn't really related to what you have in tree, in fact you are still missing that case in the suse tree - the addition to xfs_buf_lock that you're adding only in one caller was added in commit ed3b4d6cdc81e8feefdbfa3c584614be301b6d39, but that was still missing the trylock case. Either way the mainline code only handles pinned and stale buffers, which from my understanding are what matters, but the changelog from Neil reads like he saw a case where even non-stale buffers might matter - except that it's not explained very well. -- 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
