[Vm-dev] AsyncFilePlugin 32/64 bit clean patches (updated)
David T. Lewis
lewis at mail.msen.com
Fri May 16 01:03:43 UTC 2008
This an update of the 32/64 bit clean changes for AsyncFilePlugin
that I originally posted November 2007. This update adds the platform
support changes for Windows and OS X. These are not tested, but
consist of straightforward changes to type declarations and removal
of unneeded casts, so I expect no problems.
I'm adding the Windows and Mac OS patches in hopes that this will
make it easier to adopt the plugin changes in VMMaker and Subversion
in a coordinated way. On Linux, the AsyncFilePlugin compiles cleanly
and works on all combinations of 32/64 bit VM and image. Windows and
Mac OS will hopefully be problem free, and there is no RiscOS version
of the plugin.
Note that VmUpdates-1007-dtl.cs is included in this collection of
patches, but this should really be considered part of the base 32/64
bit clean VM changes, which currently consist of:
The diff and change set files are:
Fix object pointer arithmetic for #asOop: in plugins.
to handle address arithmetic for 32bit/64bit image and host.
These changes make the current values of #bytesPerWord and #baseHeaderSize
visible to plugins to enable clean address calculations on 64bit machines.
VMMBytesPerWord-dtl conflicts with VMMakerCppTranslation-dtl.
This patch merges the two. Ignore this if you are not using
These changes fix address calculation problems on 64 bit machines and
enable the plugin to work on all combinations of 32bit/64bit image and
host. Corresponding changes are required in the external support code,
which will affect all platforms (hence a coordinated change of VMMaker
and SVN sources is required).
Patch for AsynchFilePlugin header. Affects all platforms.
Patch for unix AsynchFilePlugin support code. Similar patches will be
required for other platforms.
The Windows changes compile, but I have not tested the result. The
changes just involve removing type casts and minor changes from int
to sqInt, so I expect that the result will be fine.
Added changes for Mac OS. These are not tested, but straightforward
enough that they should work without further modification.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 8836 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20080515/d1da8e2c/asynchfilepatches-dtl.zip
More information about the Vm-dev