<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: Times New Roman; font-size: 12pt; color: #000000'><br> Hi Dave,<br><br> Thanks for the patch! I'll take a look at it in detail soon (at first glance it looks very reasonable). However, I'm not going to commit anything unless I can run at least the most basic verification that the code does something useful. So what can I use as a testcase so check that the modified code does what it's supposed to?<br><br> Regards,<br> Michal<br><br>----- Original Message -----<br>From: stope19@gmail.com<br>To: vbox-dev@virtualbox.org<br>Sent: Sunday, February 15, 2015 10:07:41 AM GMT +01:00 Amsterdam / Berlin / Bern / Rome / Stockholm / Vienna<br>Subject: [vbox-dev] Patch to PC BIOS file: \src\VBox\Devices\PC\BIOS\floppy.c<br><br><div dir="ltr"><div>Hi,</div><div> </div><div>I'd like to contribute a patch to the file: '\src\VBox\Devices\PC\BIOS\floppy.c'. This patch is submitted under the MIT License.</div><div>The patch provides an implementation for 2 BIOS entry points that are - in the current source tree - stubbed out as 'not available'. The 2 entry points are:</div><div> </div><div>Int 13/AH=17h - FLOPPY DISK - SET DISK TYPE FOR FORMAT (AT,PS)<br>Int 13/AH=18h - FLOPPY DISK - SET MEDIA TYPE FOR FORMAT (AT model 3x9,XT2,XT286,PS)</div><div> </div><div>As mentioned, these are not currently implemented - so I see little risk to existing software. The patch was developed on a Win32 XP platform, the usual dev tools and Open Watcom 1.9</div><div> </div><div>The motivation for this patch is a requirement to get an old (early 90's) database system to load in VirtualBox. It uses these entry points not to perform format operations, but to setup the floppy for subsequent IO operations. Why it does this I cannot say...</div><div> </div><div>The patch is in the attached text file, and was created against the current file image from the source repository.</div><div> </div><div>Regards,</div><div>Dave</div><div> </div><div><br></div></div>
</div></body></html>