From: Maxim V. P. <mpa...@pa...> - 2012-09-26 10:15:05
|
Brian, > I had a couple nit-level comments/questions inline with the associated > patches and am hesitantly confident I understand the > fuse_get_user_pages() bits, but all in all this looks pretty good to me: > > Reviewed-by: Brian Foster <bf...@re...> Thanks a lot for review. Highly appreciated. > > As Avati mentioned, we're very interested in incorporating this > functionality. We've seen very good results in preliminary performance > testing. If this can get upstream relatively soon, we can probably > contribute some functional testing against the core bits soonish as > well. Thanks. Thanks for support. Glad to hear that the patches helps in your use-cases too. Btw, what do you, guys, think about making fuse_release() synchronous? This issue seemed to bite pretty many people and it was partially fixed for fuseblk use-case. Are you interested in improving fuse in this area? Thanks, Maxim |