On Wed, Nov 09, 2011 at 02:05:50PM +0100, Ramshankar wrote: > Thought I'd give a status update. > > I've integrated the following patches (with some minor > changes/additions) after testing them out a bit. > > vbox.panic-params.patch > vbox.stat-ttl.patch > vbox.dir-seek.patch > > I'm giving more priority to the patches that address bugs. Patches > that improve performance will also be integrated but after the > bug-fix ones are done.
Thanks! I completely understand. As you've probably already figured out, the above plus the vbox.dir-stat.patch and vbox.alloc-size.patch are basically bug fixes; the vbox.pre-symlink-refactor.patch and vbox.symlink.patch are an enhancement (i.e. symlink support); the rest of the patches are purely performance and related refactoring. Cheers, Ronald > On 11/ 6/11 07:45 AM, Life is hard, and then you die wrote: > >On Sat, Nov 05, 2011 at 04:29:32AM -0700, Life is hard, and then you die > >wrote: > >>On Tue, Jan 18, 2011 at 10:45:16AM +0100, Frank Mehnert wrote: > >>> > >>>On Sunday 16 January 2011 11:50:06 Life is hard, and then you die wrote: > >>>>Attached are some patches to improve the performance of shared-folders > >>>>in Solaris guests. Together they speed up things between 2 to 4 times: > >>>>operations like untarring and deleting whole directory tree are > >>>>typically 2 to 2.5 times faster, and when running builds that include > >>>>things like building Solaris packages I see a speedup of 3 to almost 4. > >>>>The bulk of the speedup comes from the second (file open) patch. > >>>> > >>>>These patches assume my previously sent patches have been applied. > >>>> > >>>>Licensed under MIT/public-domain. > >>> > >>>Thank you very much for this code contribution! We will revise it a bit > >>>later as our colleague which maintains this code is currently not > >>>available. > >> > >>I don't want to appear pushy, but since I've seen that some work has > >>been done in recent weeks on the Solaris shared-folders code, I'm > >>wondering if you might have time to review the patches. I rebased them > >>onto svn rev 39192 (and updated to VB 4.1.6) and they appear to still > >>work fine on Solaris 10u8 (after disabling the mmap support - see > >>https://www.virtualbox.org/ticket/9856). > >> > >>For your convenience I've attached the rebased patches (in the order > >>they should be applied), though all in one email instead of spread > >>over 4 separate emails like last Jan. > > > >Argh! I just realized that the symlink deletion issue was fixed a > >while ago, so the hack in vbox.symlink.patch is not necessary anymore > >(yay!). Since updating this patch causes some subsequent patches to > >not apply cleanly anymore, I'm just re-sending the full list of > >patches (i.e. please ignore those I sent yesterday). > > > > > > Cheers, > > > > Ronald > > > > _______________________________________________ vbox-dev mailing list [email protected] https://www.virtualbox.org/mailman/listinfo/vbox-dev
