On Thursday 30 April 2009 14:26:06 Daniel P. Berrange wrote:
On Wed, Apr 29, 2009 at 01:14:05PM +0200, Pritesh Kothari wrote:
Hi All,
PATCH 1/1: contains changes in version detection and moving a global variable in the global struct, so that no more global vairables exists.
+ if (uVersion >= 2001052 && uVersion < 2002051) { + DEBUG0("VirtualBox API version: 2.2"); + driver = &vbox22Driver; + } else if (uVersion >= 2002051 && uVersion < 2005051) { + DEBUG0("VirtualBox API version: 2.5"); + driver = &vbox25Driver; + } else { + DEBUG0("Unsupport VirtualBox API version"); + }
This bit seems to cause a link error for the driver, because nothing is defined the vbox25Driver symbol. I guess you have another patch pending to add vbox_V2_5.c ?
I just #if 0'd the 'else if' block to make it link
Thanks for #if0'ing it, sorry forgot it. yes i have a patch for 2.5 but much of work is still underway due to the recent added functionality in VirtualBox, post it soon once it is stable enough. Regards, Pritesh