[vbox-dev] vhd format not stable?

[Huihong Luo]

Hi Alexander,
 
I checked out the fix, yes, it indeed fixed the ASMBitTest() crash bug.
 
However, there is another weird thing happening. With this newest fix, my vm starts to behaves differently, the audio drivers are no longer auto installed inside the xp vm. When I plugged back VBoxDDU.dll before the fix, then audio drivers are auto installed for the vm. I use exactly the same .vhd base image. The only diff is your fix on VBoxDDU.dll. Is there any chance something went wrong?
 
Thanks,
 
- Huihong


--- On Mon, 6/22/09, Alexander Eichner <Alexander.Eichner at Sun.COM> wrote:


From: Alexander Eichner <Alexander.Eichner at Sun.COM>
Subject: Re: [vbox-dev] vhd format not stable?
To: "Huihong Luo" <huisinro at yahoo.com>
Cc: vbox-dev at virtualbox.org, "Frank Mehnert" <Frank.Mehnert at Sun.COM>
Date: Monday, June 22, 2009, 12:25 PM


Hi Huihong,

the crash should be really fixed now. I also fixed the crash with
snapshots you reported in another mail. They will appear soon in the
public svn. Thanks again for your help!

Kind regards,
Alexander Eichner

Am Montag, den 22.06.2009, 08:59 -0700 schrieb Huihong Luo:
> Frank,
>  
> Right, that would be the correct fix.
>  
> I was wondering if there are other places that have similar issues.
> This problem would be very rare in happening, as the memory address
> would be still ok to be read even if it's less then DWORD in len.
>  
> Inside kernel mode, this becomes a severe problem, throwing invalid
> memory access crashes.
>  
> - Huihong
> 
> --- On Mon, 6/22/09, Frank Mehnert <Frank.Mehnert at Sun.COM> wrote:
> 
>         
>         From: Frank Mehnert <Frank.Mehnert at Sun.COM>
>         Subject: Re: [vbox-dev] vhd format not stable?
>         To: vbox-dev at virtualbox.org
>         Date: Monday, June 22, 2009, 3:30 AM
>         
>         Huihong,
>         
>         On Friday 19 June 2009, Huihong Luo wrote:
>         > I tested it again, but it still crashed.
>         >  
>         > I think I now found the cause, it's quite delicate, but you
>         should be able
>         > to fix it right away. 
>         > a8bd6ffd ff ff
>         ff ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??
>         >  
>         > The above is the bitmap when crashed. You can see the
>         beginnig there are
>         > only 3 bytes are valid, the 4th byte will cause crash. 
>         > But when you call
>         > ASMBitTest(const volatile void *pvBitmap, int32_t iBit)
>         >  
>         > it will the 1st 4-byte int, thus caused the crash.
>         
>         Good catch!
>         
>         > The easiest fix is to allocate 8 byte length more for the
>         bitmap, and zero
>         > fill them, that will definitely solve the issue. You can
>         also change
>         > ASMBitTest method impl. 
>         
>         I think a better fix would be to ensure that ASMBitTest isn't
>         called
>         with non-word-aligned memory addresses.
>         
>         Kind regards,
>         
>         Frank
>         -- 
>         Dr.-Ing. Frank Mehnert    Sun Microsystems, Inc.
>         www.sun.com
>         
>         
>         -----Inline Attachment Follows-----
>         
>         _______________________________________________
>         vbox-dev mailing list
>         vbox-dev at virtualbox.org
>         http://vbox.innotek.de/mailman/listinfo/vbox-dev
>         
> _______________________________________________
> vbox-dev mailing list
> vbox-dev at virtualbox.org
> http://vbox.innotek.de/mailman/listinfo/vbox-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.virtualbox.org/pipermail/vbox-dev/attachments/20090622/a4c64cc7/attachment.html>