Re: [Bochs-developers] [Patch] [BIOS] support BCV for rombios

[Akio Takebe <takebe_akio@jp.fujitsu.com>]

[-- Attachment #1: Type: text/plain, Size: 751 bytes --]

Sebastian Herbszt wrote:
> Akio Takebe wrote:
>> Hi,
>>
>>>> +   /* jump to BCV function entry pointer */
>>>> +   mov  bp, sp   ;; Call ROM BCV routine using seg:off on stack
>>>> +   db   0xff     ;; call_far ss:[bp+0]
>>>> +   db   0x5e
>>>> +   db   0
>>>> +   cli           ;; In case expansion ROM BIOS turns IF on
>>>> +   add  sp, #2   ;; Pop offset value
>>>> +   pop  cx       ;; Pop seg value (restore CX)
>>>> +   ja   no_bev
>>>
>>> ja? not jmp?
>>>
>> Oops. yes jmp.
> 
> Can you please resend a fixed version?
> 
> Thanks
> Sebastian
> 
> 
Sorry, and thank you for your review. I updeted our patch.

Signed-off-by: Akio Takebe <takebe_akio@jp.fujitsu.com>
Signed-off-by: Kouya Shimura <kouya@jp.fujitsu.com>

Best Regards,

Akio Takebe

[-- Attachment #2: bcv_bochs.2.patch --]
[-- Type: text/x-diff, Size: 1219 bytes --]

Index: bios/rombios.c
===================================================================
RCS file: /cvsroot/bochs/bochs/bios/rombios.c,v
retrieving revision 1.222
diff -u -r1.222 rombios.c
--- bios/rombios.c	15 Dec 2008 06:05:52 -0000	1.222
+++ bios/rombios.c	18 Dec 2008 06:15:52 -0000
@@ -10251,6 +10251,30 @@
   mov  ax, 2[bx]
   cmp  ax, #0x506e
   jne  no_bev
+
+  mov  ax, 0x16[bx] ;; 0x16 is the offset of Boot Connection Vector
+  cmp  ax, #0x0000
+  je   no_bcv
+
+  ;; Option ROM has BCV. Run it now.
+  push cx       ;; Push seg
+  push ax       ;; Push offset
+
+  ;; Point ES:DI at "$PnP", which tells the ROM that we are a PnP BIOS.
+  mov  bx, #0xf000
+  mov  es, bx
+  lea  di, pnp_string
+  /* jump to BCV function entry pointer */
+  mov  bp, sp   ;; Call ROM BCV routine using seg:off on stack
+  db   0xff     ;; call_far ss:[bp+0]
+  db   0x5e
+  db   0
+  cli           ;; In case expansion ROM BIOS turns IF on
+  add  sp, #2   ;; Pop offset value
+  pop  cx       ;; Pop seg value (restore CX)
+  jmp   no_bev
+
+no_bcv:
   mov  ax, 0x1a[bx] ;; 0x1A is also the offset into the expansion header of...
   cmp  ax, #0x0000  ;; the Bootstrap Entry Vector, or zero if there is none.
   je   no_bev

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel