On 10/17/2011 06:00 PM, Bryan Schumaker wrote: > On 10/17/2011 05:57 PM, J. Bruce Fields wrote: >> On Mon, Oct 17, 2011 at 05:55:49PM -0400, J. Bruce Fields wrote: >>> Bryan Schumaker noticed that we have a longstanding leak of an open >>> owner each time the first open with a new open owner fails. >>> >>> The following patches fix that, along with a few other less likely leaks >>> that occur if a memory allocation fails after a file is created. >> >> By the way, Bryan, I didn't try to write a real test case for that >> leak--would it be easy for you to rerun the test you were running before >> and verify whether it's gone? > > Sure, no problem. I'll let you know what I find. Looks like these patches worked. I never deleted more than one client structure during my test. - Bryan > > - Bryan >> >> --b. >> >>> >>> I intend to commit these to for-3.2. Any review welcomed. >>> >>> --b. >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
