OpenBIOS August 2016

openbios@openbios.org

10 participants
26 discussions

Start a nNew thread

[Report] MorphOS

by Natalia Portillo

4 years, 1 month

[Report] Mac OS X 1.0 Server, Mac OS X 1.1 Server, Mac OS X 1.2 Server

by Natalia Portillo

Hi all, I've been asked by Mark to report my testings here. Tested with qemu-for-ppc-2.8 commit 8b0262d0bcb7c697f2db8d6cd731cb478937902a. Command line: qemu-system-ppc -M mac99 -m 256 -prom-env 'boot-args=-v' -bios foo/openbios-qemu-vga.elf -boot d -prom-env 'auto-boot?=false' -hda foo.qc2 -cdrom MacOSX.iso -nographic Welcome to OpenBIOS v1.1 built on Jul 31 2016 05:52 0 > boot cd:%BOOT >> Not a bootable ELF image >> switching to new context: 4+:interpret: exception -13 caught >> interpret : update-key-map " get-key-map" " kbd" open-dev $call-method drop gKeyMap 4 0 do swap dup @ >r 4+ swap dup dup @ r> or swap ! 4+ loop ; update-key-map gKeyMap 1+ c@ not h# 40 and if 0 stdout ! then failed with error ffffffed Rhapsody Kernel Loader <02:12:14 990220> sl-debug:interpret: exception ffffffed caught >> interpret sl-debug failed with error 00000003 Debug Flag = 00000000 interpret: exception ffffffdf caught >> interpret to DIH value &CL DIH to my-self 1 encode-int " AAPL,boot-display" property &CL 0 100 " set-colors" DIH $CM " width" GMP ?dup 0= if 280 then " height" GMP ?dup 0= if 1E0 then 2dup to Dheight to Dwidth " linebytes" GMP ?dup 0= if Dwidth then " depth" GMP ?dup 0= if 8 then over value rowbytes active-package DIH ihandle>phandle to active-package : rect-setup >r >r rowbytes * + frame-buffer-adr + r> -rot r> ; : DRAW-RECTANGLE rect-setup 0 ?do 2dup 4 pick move 2 pick rowbytes d+ loop 3drop ; : FILL-RECTANGLE rect-setup 0 ?do dup 3 pick 3 pick fill rowbytes + loop 3drop ; : READ-RECTANGLE rect-setup >r swap r> 0 ?do 2dup 4 pick move rowbytes 3 pick d+ loop 3drop ; to active-package failed with error 00000100 call-method fill-rectangle: exception ffffffdf >> call-method fill-rectangle failed with error ffffffdf interpret: exception ffffffdf caught >> interpret drawImage failed with error 00000108 interpret: exception ffffffde caught >> interpret dev /packages/obp-tftp failed with error 00000003 load isn't unique. ?no dev call-method fill-rectangle: exception ffffffdf >> call-method fill-rectangle failed with error ffffffdf call-method draw-rectangle: exception ffffffdf >> call-method draw-rectangle failed with error ffffffdf EXIT 0 > OS X Server 1.2=== Welcome to OpenBIOS v1.1 built on Jul 31 2016 05:52 0 > boot cd:,%BOOT >> Not a bootable ELF image >> switching to new context: or === Welcome to OpenBIOS v1.1 built on Jul 31 2016 05:52 0 > boot cd:9,\\:tbxi >> switching to new context:

4 years, 1 month

Re: [OpenBIOS] [PATCH] mac: Don't set APPL, address properties on NewWorld

by Benjamin Herrenschmidt

On Thu, 2016-08-04 at 13:56 +0200, Howard Spoelstra wrote: > Great, that fixes the "regression". One thing I noticed while booting > with only the Leopard iso is that Qemu/OpenBios make the Leopard > system profiler show two ATA devices, both containing a QemuDVD > drive. Perhaps that is the reason a hard disk is not recognised? Right, I'll look into that later. I have a mostly working PMU now, which speeds up 10.5 boot significantly. Getting that to work with OS 9 required some fiddling .. Last remaining problem with the video ndrv is that MacsBug doesn't work with it and I can't seem to figure out why so far. I've added pages support etc... to no avail. There is *something* we do that upsets it, everything else works (and MacsBug works fine without the driver, on just a dumb fb). Cheers, Ben. > On Thu, Aug 4, 2016 at 3:16 AM, Benjamin Herrenschmidt > <benh(a)kernel.crashing.org> wrote: > > They are created by the MacOS ROM and in some cases MacOS X to > > contain > > the *virtual* addresses of the BARs as mapped by the kernel. > > > > Having these already there with the wrong values causes the hangs > > with MacOS X 10.5 and the video driver. > > > > The reason is that the IONDRVFramebuffer code in OS X 10.5 won't > > replace > > those properties if they already exist, so the driver ends up > > accessing > > the addresses put in there by OpenBIOS which aren't valid mapped > > virtual > > addresses under OS X. > > > > Signed-off-by: Benjamin Herrenschmidt <benh(a)kernel.crashing.org> > > --- > > drivers/pci.c | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/pci.c b/drivers/pci.c > > index 0cdf0bf..9813086 100644 > > --- a/drivers/pci.c > > +++ b/drivers/pci.c > > @@ -1029,10 +1029,9 @@ static void ob_pci_add_properties(phandle_t > > phandle, > > } > > > > pci_set_assigned_addresses(phandle, config, num_bars); > > - > > - if (is_apple()) { > > + > > + if (is_apple() && is_oldworld()) > > pci_set_AAPL_address(config); > > - } > > > > PCI_DPRINTF("\n"); > > } > > > > > > -- > > OpenBIOS http://openbios.org/ > > Mailinglist: http://lists.openbios.org/mailman/listinfo > > Free your System - May the Forth be with you > > >

4 years, 1 month

[Qemu-ppc] [PATCH] ppc: Add MacOS VGA driver ROM

by Programmingkid

On Jul 27, 2016, at 6:10 AM, qemu-ppc-request(a)nongnu.org wrote: > Message: 1 > Date: Wed, 27 Jul 2016 18:27:33 +1000 > From: Benjamin Herrenschmidt <benh(a)kernel.crashing.org> > To: qemu-ppc(a)nongnu.org, David Gibson <david(a)gibson.dropbear.id.au> > Cc: qemu-devel(a)nongnu.org, Mark Cave-Ayland > <mark.cave-ayland(a)ilande.co.uk>, Alexander Graf <agraf(a)suse.de> > Subject: [Qemu-ppc] [PATCH] ppc: Add MacOS VGA driver ROM > Message-ID: <1469608053.5978.132.camel(a)kernel.crashing.org> > Content-Type: text/plain; charset="UTF-8" > > The project is at https://github.com/ozbenh/QemuMacDrivers > > This adds a native MacOS driver in ROM (which can be picked up > by MacOS once OpenBIOS has been updated if Mark accepts > the patches I sent him) which allows MacOS classic > (tested 9.2.1) and MacOS X (tested 10.1.4 and 10.4) > to properly use QEMU std VGA (10.1.x doesn't work at all > without it, the others get the ability to change resolution > and color depth). > > Building the ROM is tricky and requires CodeWarrior for MacOS > so I include a pre-built binary. > > Signed-off-by: Benjamin Herrenschmidt <benh(a)kernel.crashing.org> > --- > hw/ppc/mac_newworld.c | 8 ++++++++ > pc-bios/qemu_vga.ndrv | Bin 0 -> 12808 bytes > 2 files changed, 8 insertions(+) > create mode 100644 pc-bios/qemu_vga.ndrv > > diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c > index 7d25106..5029345 100644 > --- a/hw/ppc/mac_newworld.c > +++ b/hw/ppc/mac_newworld.c > @@ -511,6 +511,13 @@ static int core99_kvm_type(const char *arg) > return 2; > } > > +#define MAC_CORE99_COMPAT \ > + { \ > + .driver = "pci-vga", \ > + .property = "romfile", \ > + .value = "qemu_vga.ndrv", \ > + }, > + > static void core99_machine_class_init(ObjectClass *oc, void *data) > { > MachineClass *mc = MACHINE_CLASS(oc); > @@ -520,6 +527,7 @@ static void core99_machine_class_init(ObjectClass *oc, void *data) > mc->max_cpus = MAX_CPUS; > mc->default_boot_order = "cd"; > mc->kvm_type = core99_kvm_type; > + SET_MACHINE_COMPAT(mc, MAC_CORE99_COMPAT); > } > > static const TypeInfo core99_machine_info = { > diff --git a/pc-bios/qemu_vga.ndrv b/pc-bios/qemu_vga.ndrv > new file mode 100644 > index 0000000000000000000000000000000000000000..0b1549d1c5978f87aed2b6e496a23fbaf18dbea4 > GIT binary patch > literal 12808 > zcmcIq4RBl4mA;amWZ5{0p=Q^?KY3{!Mof&y6hj}_b>&0>89PD20;gS=wQRj4+L0xV > zBqz%95<NR9O=j7tGYM^grZ^3-VJ6FXrqIG9-~>W<nW0&j9d^hpKcNAdodrr4*ph_g > z?RW0GlAmOWm~N}lnRo8F=bn4+x#ym9-+lMC_~yk4b=|tero=E~%=Yhxe^QO-%+JB> > zR50)}iOt&?%Yh@oB*{`&<z_4g9%i#y>vAIPp8(S(v=cuYx=rNd=boq|hHn5+&(H?K > znEe$|<~jk-{m^gzcSRm!%n3YK_e~4k0eqe)UsFV{p)xwS8E^~WZvYX%7QlAELx4ko > z!+?JSd;mCSXSVr(s{uCxdI2GT2H4F<0K<^n+pbuCtM0hPrvJoSrT@{pP+#C(=1WF4 > zs>yyexiK=Vrurg7Y;|Zvy+d6eNu`sUsT^3*Lyx;6VKr{$TM>#4tI=R&L`||lJRa?i > zM&rYwG}@-X8cVbO&9UJP$#^XC_iE4WtmpO>@sW{GEZma<l8CBlmED=3=Bzs$_KhY( > z=}0^ljCT*E;?Yr5_N(chSXxacM-%CpUVLgu^uuyWu8cv`9^Z<1VskOg7fB`JDYcl~ > z6N{wtv_LYRP?PD+x2h#+{ZOB+jD;hiSpTL_;<_7`qwUH}V%7TOig;{YWc?j#C_E5P > zh6{)T$w*q|)VuoogckYH2rDvFtyVXcBKzXAsKE`YIafY4HJpqj$XdR*NrHK~$4Z#( > z+UHKOs@E>qnC%Jpt<G%lg6DuG;1F*6<I`itClfa2-m}=4(zN=F_Om6KiTy00y>)#? > zGdeQLkew;x_RIXt_B~G{tLh%JNubpYOxE124YO7);&^<uF3H(#r!)E(OMq{U7Ov0u > z89SwisBC{<lwYN?qBG`ajw=Q2$V@x?DnUCqLv!sEv^}OQSEDFzC!Xx_Pczx~&u8>w > z)Q`g^>NaFw1pPhG_Y+-m3VL968|SE~YcEmv15sCJ_U+8|?N)T-#K5VFd%feVr3E&| > zvs%G=!kFu*Z*6-;yPqw&T<>PJ+F(VyCOI<Nwj~+e#?~r}SwM3;9zP7buhyA7F~%BT > zZ+YSvYo2(Qd3M=Y%dW+&ZC5vIr+Pj|`;E`?bA<Wn_-*Bsm=|BKVH`Wy#Msm##z=>W > zMa;8z5o-ptc1$xVd*Ht@7xL?*i?Ws3vybxfTU2gA<mnz~Zs>i4+M<mDXn~Bjj(0E? > zw8ebYWMBF;qa3C2HSM3sVoc%s%{1nYN#g|CCZNk&{dvZq%MEBJi*|LEb`i8|C~Bvm > z-F;@eix*5i7rm@)soxBpYEkz~(zVyP_%8aW_Orj<i8}JfHw67<YExiS*r}w=(2o4t > zVz*BqcW2ISI_#o1F&A`{Fm^Y}RVdF#xf<mf>W6#{ZAQ!WxBFe=%lsT=xt>@z4jbAm > z&YnI`cC=v}jmjVrj>QkQp<PaA(f%mLNNsC-jSKJl*}?~uA<XwMlQ56h@_966_~A=l > z@}*q+>0`zhc^%%qKC|ru)*xhgAnQSl_bLx#3%keIE6R|gGsp_purg@;9y&fHo`*<p > zKdX5Tv`G_}BEa{Uxa$evJ50QS=$gQZKVjmP`+$c{TzVS#S|K}N(rPe1<}>lCn}91O > zUQIGuOx*oCa9QA=1Ftpl%ZSHK@^XGmxYf3YVULowd45@D+xvOjJlbn~xkcD!8RA<F > zZ1YsHZOlIBUxz+FGjZoU;Ac#{VlnVHO?)2lyejA+;KxneeHQpLChjDkJ_LO~#`95= > zHoph>UK6jn9QgevK5rFpUGO{xJZ<9D#J|DBYt{h2+r%#?I|}<cNiShvXE$(RU#_pP > zZ^dswmnm)`c9XuszEvlIlYKw7*!Sad_AT^XDcozEzef1_EI%#cP4^n-E})+ikbNoM > zaqzxYj@P;dXV$v33vKr4S?2K|-tkyHXuOB9)wMeegW@G<h=Er^?p)WVGzpnK%6R3J > z9%dfJ!E6%}<Rh(`vT8pnH+HLz8jlmwlghaCEcE$^V(bLsCvQ8=8ankvN|8nr8g`C| > ziHE#8^LFZym?EXu37QUCU8f#a6=`Ey(C&t;9i4hKp-7vOg0=y)bf>-%e94rcHKXld > zr=HrZNU>o-qg>EOI^gM7#%;QealuAZRyy@HD-~(goucf4e#43skwv*qw5t(M?mJe0 > z$;G-k<-DSO$w9=w;Fbj!<wgV=W~<S%EOW4dxw7xrkt?JVup#;Db(F_xk3su_?5{t~ > z4E&R;3-tX~J-$}%y_6nXg&r64dLXwLPY6A1lmoLLoX_d8u#_G=$A^p$#k!c(?nS;j > zvMfV;u;4{k7sdY_4yJfm-hU}hXLJYig64*ARAWsLEv(lwR>T*`K60A&W1ZTOF9HKc > zUEihrs^804aI#L(hV4(H&aE9-meC&;wCxznc4qgpZ#>&!V4t|?jMqD69Or&%V;hin > z2KgQaI_KkSk^X$XG|X3nWxnb$U(-1|Ip-laj%6=Wj)88|p9BW<uch10j<=y((^u=3 > zvz-jvX@iLE%=o?LsLSN^o{2@|L(XR8NvvIi;@nrnBF@_d-ecu>XRq^Fb4alt`I*bQ > z2eL-R{*4Jc=l6=abwhqK2f5@2%|dPxavKjWVlK=0Phk9t)eo)m3-%CO>-nB(2lj=Y > zLf^~j{X^u==XPNKiGAa>#d=%w?;P~D+T&r!Rbda?6m&0^`!)5o%AJDTc^J>Nd!ci& > z+&@9zoSly&-#snnkMAK76KBb{>Rse_zeD>`TEmz2JC_r@J#*xRKcB~*v$Q|YVNS@G > zT1;PZ^-}JI?O2QO1wUKRh<fCV1*?i}X&q}RpAs^$PdK-w$iHa5>OgDJ7;6u*lUu+3 > zT+y0xZGr5?x%FS<zdzlPWj5LmV9qKK58c{eeI_Wmx`M1CYwi*F`&=4)EU3G>v_Ym| > z+;;G`BX8D{4S?4HZ-BmSD3ct@lg+S&Lp*JGHlU1Hig%hpE|X&Q36$$}yDh3AhEx9& > zL$>Hw*>DbXMRwaG?20|l*`+jB9vqipJcxU&0&5iSa4ZTM?St;7{>7LS%a{nKG5Js@ > z0q-Qg7H!*r+XUVW+%E7si~QX9kpIi?r)H0b&)H+W#up=H=ByoiIEFcMc*l6YBOA)x > zhM0!D4WTR6p7ba5J>6^k@ed{X=Ke_i@HIUL`GsrSbrNzP%*mC}-kOJs*L!f#)djmR > zfZc2Gq}*Oh;{<*=Xm!A0U#3(3!gEYK$LT5Bj-gEhc&QET4{1+FzULOa3&c~VC*ONm > z-y22D7k*^)8vpxNiS?=Ii<mFya3Kd8YpCBT#8b?VpYMg6n4NqEG#B`4Cz|LzqR<zt > zv9^4Cs91{r?<vuLGv3!+7$2>{oWIq~rLWR{SK;k7KL0#7M;^#EV*r0Y^I$Ah$U{7y > z@6E#Jv&;e7Ao&^22xg6KZn3Z8`{;A5#MoLe4zFq33!mp~`)9Ie41H4mv*vU1Z<^0s > zpJTZ`F`s>Z=K0*&Ykc;PCHnW_%!Ssrts8z1T>=yrE$f_OO$Gd@rgISQI*#Ee_bJAH > zj3=#!f-{vl`qlyHVk^>x@?B>M-|~RQ%sKgTUoh<IBHQ>`<?D0txoPY99nPZ6cT<XY > z6~F2=J}u6Fq(gMFk-5kZE&jb$-@@!2FB%5wa=Kj3by>vvv?kaKah~Q;lXy;PFyHAb > zuM_X@v`$F2xlS5q`|aV<=Vz02?nXLqdl8Pp_p22rdX4{ismONZM+aM)jEl5(I*#&O > zQ_}*wq3quF4(rn&VF8>$+7z6b^7*B6K4x5)Xf)$B+HZia<O9{3L$=O;fX1SXF|V?p > z`TWd1&oU;;18qtZ-oYV9w=plCZajT>(z$FWo|ON_ry9*1jJlb3m~vyLIQa9*sg6_m > zF@M-={Nc47jm2xn_DX^8HWvG??T|8T7&E$8tSQvpc0Y8p!H=sZoA8dyW9qq^f-2to > zF#nM4n!rAVo>LCQQ}an@lvJKCo;CCY-xZxY^K*O~`z!IhO4MDA^LbiluBmM}--Im% > zUQM{7vWnpFnTy)?h5sSwQj3%4oCZ8856PI9H-J-ag733efXkrkz>h-zoGwkuBtJuS > zO&vgv((%#3?DL!ad4M(36S`iFcK2iL+9$TM`QYPYfNo8KpV~TdZGk%lZf)OJsy%q+ > zGBQlNZ7a~h-Eqn|L1$;>{34&v;SahK=h%Z+)$ZNKyna@7C!NQl-K=*-Yo6ifM3UL= > zA5Qajjnq!`dCB)Yh5CJrH3SFUo!R&8we&3kvF8}cn94e_r>p#p(Bsmw#vyCFDN766 > zYbS?!`%!97=TVR~b}8MaE${jrKX<!%JR9Wa{|{(bH<*(u><e}V@70@>A-8|vxYRYE > zs&o$X(ZGw+jh*mk51Tp)-@)09dkFN`MQnNo@vBa&=ds_d)jKk~v)F$Q;v=SE+!VZ5 > z>Z17uer-_WxC8j&;8x&6?sPW%pPB3r@LdS!K9D)K`yJ*3c;Sz*5%xOuh$-k(+hVW9 > z*@>hz@HkngJjQInNu0UtW33*E;+64+`9$0pa{mZzZy*`LNvR7t-w)`++N{7c0DUO; > z%)}FMCNh^ZF5y>`N{9sz(;{Sd;C}e;xA5&MfbUwt939`F{OG4|B82#c*akkYK5pY< > zQ(nOLfnPIxb8?ReKgjEo+v8dNk<S-m4)*TN$YX8v9)!8z?_3^c<{ZeXBw4WU_b3i= > z-ZI}UX2~9M{{ro7Xh(f0z^~K>>0R#a@1ooaXhDA-Wtz2NzT2oz%uAhu?=Lu`98$JN > zWk0KUmikAIX%#xrT)coYBl_-84g3`FeZu~i^m{($p~@PxiHPwikn@0L{P@P?dYaDx > z^ugS=eU<E7Ou*P46nwK{K#qs@mE0B%*0RHqACDVnGZ#w54$*E(v}1b<+tupBtex_S > zpE*Ag^PL}msrjaNTkVl0SeK4}rFUGM3E-^8@$w}0w3>8F!aO73WxFWP6ZMi=rt|s2 > zJyyXU*Sf!<zV`9H{>AL;9L~rbFXLVOqu^HLG!uU}n9bv_4;=NjW{)~D$j@xx6!23V > zw+9D(tpR$s5c>nzDnHKLUPbw>+fDwjyUe^xdk1>ICOgm?hM(002XW3>!CG-f&pb{} > ztL3!yW*nFRjmDR=oi)DE-bec>#xsOChk3;LYCvm5+z|c1hYk%)!gWvMTllcO7JDRo > zAFo_O-=F!}?|rB7-SGV9kS}3f;~VL@rNA8n+a=7yxrM-O1MhJhHfMH<m7v>zJ5Bzx > z)YfJ4oc%B06(-Nww}9hYG|4<m-!3Wz{t|HND+KzpyAQLz-47!Mn)8>B3x};|w?${% > zG=JLj%lJ25YKJl6`+=S3o0+`}rw5Q{@$x{pov&Bq5No?O{#_m4adYSK;Cq66wn&b< > zI2Y#5=;{45w~j=d!aB8Y5q>{w%}sY3X9WKQ&0X*zoZTG8S^AsTXF5(n*YV14V-Jb< > zJ);u2%A%`{zVmyrPHQ!X<HwNU<=?T#D{rGcC*sCQ$fIx5Z^93KLH5b>;PHvva-_kD > zc0WUIZema33~3sCkM0O#p1tUw-etzw9(*gRrZE6-&?Q#mVH`(Xs~muh{~4f@Y<!C$ > z{~|i>FR%vy8^qX&_7KWrfS{+|jMXG-0=TDvm*pMs^Pr>Rd5SscJD~A%w70wVqCSXw > z8`%r2742&&@1Z}h*`ML1JvPR<i^h%TPBAALocAaAc|UARYh*6D&5{!)`FAcIJ_);{ > z-;*?V+Hl1Uuo0b2G*3c@K_|0Mp`Q9wzzh1zg0CGq=KVGocgB!ECLl+zhSB<v%PY-$ > zD4@$_=xt@_QW<(n8G1t*x>APTS%%(ThVCvyXJzQ#GW6y$^qMmC+A?&18G2nAx~~kq > zx)dEt;4k#6oPzu7j7gJ#U4U^w8W3j8@f6@uKqtThs0G*<qx*pNHvqo`+y__-@B!)o > zmjR%w^eo^>Z2vw4oB;d^Z~(9q5CeQ4-~}vY%w+&h0;U0v0icI#3jjK}dI8M<C$0wm > z4saX*ot+N@eh5ec?g4ZGz~{srK?(XvkSC$v$|Zmr#@wd>F93dl@n-P+5x@<=SlvSa > zETzha013bV;5NXwF%U$e+^_x&VIyhv@90^*MjaWwYki1c@gqXf&jw53H-BF1B^Zm^ > zl_=U`;pE1Rk#L+u29u#-^=5f#s)|`}vQe|WfI|R`--Z>>jsV^Oj=5uffI+}^Ktf!A > z!-c*`D2l6kD@MX>C>~0NZ&{9Og-JD)l9#qA(Y3N5t_md*YM3tdne-}?8+Qpyu!B0A > ze`PE*6jgC6a+Nv~Pj2o{U=VUrO^+sHYFJ(x#$foJN;v|hP>~vqrZLDo{SGw|MH5og > zWV{88dEHG?|0u2`rqr++o?*jn!*ntpB?A9R@C6BeD#?HrfCBIV0)V>#8vx}08ej+D > zcoFO=D(C7aL5J-*|1(0LbSOQVGP~wJ+zkIjpHB4YM4wLd=|rDS^yx&OP91<go#?{} > zc}~d7w|fR<^yPfjq&eRNeg^Ov%1rQKTsi*#*R$Hxzc&(%;sR>k<aB3qro=C6&J<Tf > zab<L-czZ&PQOWA9e1dOPQ|VAN?Z=(gDDJ>o>TgYk5*s4JxagWxagCKOs7BNjvyy$0 > zq&l1~Q;DmnrP|yY!Ufu%*t&QhF6S0VyF;ZOHmaN>Nd8$+4SBq#keCc@4usYhazY_! > zxw(*0O%q?C{v@)P5nh{!C(|iEmfxmOQe82cOvRH0vLOE}YJJJjrmv_*pM|x=-w)TE > zHHQ4{*@Di|zH|k&qRYy~%(H0V3&c0!y7a6*A-YTrwMnY$aE}>_nO_(dSDlM{n;X~i > zP)Ze%s#pM|$KX@A7hGJkDjqI05!A)xBymf-R29*1<2n*sUmyr0nZ@0|DUu%EFpCWN > ztJO>zSHbBncL|MnstJUWp^+4pJhSVF?Wd5UWZ-KNdqAUfg;dFLmlw||<X9a~#2o%a > z+5Gu>`U5?G+vEGr(zf=DeAlX#tG=@|EUy}>Vt2+;rcWX=_xR++RZI+9PO0fM21tQ9 > z!~a!mbv!Mvi;u>_h}?1%OH5uYkEXyRht+i<MDf-tCK8~$@w(QwHpu7IX}SY1r{i)e > zGBO&4JIZ%0?*sGdxNKIX%<J{C=k|Vit^D0<T8Kf8p}!=*Xs^OIz)yjPx-(-|Yb!{w > zxlGoz>KRI<BI}WBt__(94lA>}JJ8o6ifnoJ9ms9p%J2?y%ry3V#YhAq(2=#7oDlcm > S3pms9xI7eDPqe?>@BaW(s=rMD > > literal 0 > HcmV?d00001 Your patch does successfully work with Mac OS 10.4.0, Mac OS 9.2.1, and Mac OS 9.0.4. Great job :) Reviewed-by: John Arbuckle <programmingkidx(a)gmail.com>

4 years, 1 month

Re: [OpenBIOS] [PATCH 0/2] Add legacy virtio-blk driver to OpenBIOS

by Mark Cave-Ayland

On 01/08/16 16:06, Artyom Tarasenko wrote: > Good job, Mark! > So, now it's possible to boot the PPC and SPARC64 systems from virtio > drives, right? > > Artyom That's correct, with one caveat - it seems that the drivers in some of the test Debian 9.0 images don't like QEMU advertising both modern and legacy modes: ./qemu-system-sparc64 -drive file=/home/build/src/qemu/image/sparc64/debian-9.0-sparc64-NETINST-1.iso,if=virtio,media=cdrom,index=0 -nographic -bios /home/build/src/openbios/openbios.git/openbios/obj-sparc64/openbios-builtin.elf.nostrip -m 256 OpenBIOS for Sparc64 open isn't unique. close isn't unique. Configuration device id QEMU version 1 machine id 0 kernel cmdline CPUs: 1 x SUNW,UltraSPARC-IIi UUID: 00000000-0000-0000-0000-000000000000 Welcome to OpenBIOS v1.1 built on Aug 1 2016 18:06 Type 'help' for detailed information Trying disk:a... No valid state has been set by load or init-program 0 > boot virtio0:d (cut) [ 22.223263] ffe2f1d0: ttyS0 at MMIO 0x1fe020043f8 (irq = 5, base_baud = 115387) is a 16550A [ 22.223905] Console: ttyS0 (SU) [ 22.277730] console [ttyS0] enabled [ 22.297898] mousedev: PS/2 mouse device common for all mice [ 22.321023] rtc-m48t59 rtc-m48t59.0: rtc core: registered m48t59 as rtc0 [ 22.323433] ledtrig-cpu: registered to indicate activity on CPUs [ 22.332493] NET: Registered protocol family 10 [ 22.356833] mip6: Mobile IPv6 [ 22.357582] NET: Registered protocol family 17 [ 22.358693] mpls_gso: MPLS GSO support [ 22.364318] registered taskstats version 1 [ 22.367031] zswap: loaded using pool lzo/zbud [ 22.378680] rtc-m48t59 rtc-m48t59.0: setting system clock to 2016-08-01 20:14:14 UTC (1470082454) [ 23.527600] random: systemd-udevd urandom read with 0 bits of entropy available [ 26.788885] ne2k-pci.c:v1.03 9/22/2003 D. Becker/P. Gortmaker [ 26.901425] ne2k-pci 0000:00:04.0 eth0: RealTek RTL-8029 found at 0x1fe02008000, IRQ 6, 52:54:00:12:34:56. [ 48.346114] NMI watchdog: BUG: soft lockup - CPU#0 stuck for 23s! [systemd-udevd:57] [ 48.347490] Modules linked in: virtio_pci(+) virtio_ring ne2k_pci(+) virtio 8390 [ 48.349833] CPU: 0 PID: 57 Comm: systemd-udevd Not tainted 4.3.0-1-sparc64 #1 Debian 4.3.3-2 [ 48.351288] task: fffff8000f4275e0 ti: fffff8000f55c000 task.ti: fffff8000f55c000 [ 48.352552] TSTATE: 0000009911001600 TPC: 0000000010024274 TNPC: 0000000010024278 Y: 00000000 Not tainted [ 48.355228] TPC: <vp_reset+0x14/0x40 [virtio_pci]> [ 48.356043] g0: 0000000000b04400 g1: 000001ff04800014 g2: fffffffffffb0000 g3: 0000000000000000 [ 48.357446] g4: fffff8000f4275e0 g5: 0000000000332ed0 g6: fffff8000f55c000 g7: 0000000000000001 [ 48.358836] o0: 0000000000000000 o1: 000000001000ad08 o2: fffff8000f55f380 o3: fffff8000f41a880 [ 48.360203] o4: fffff8000f41a888 o5: fffff8000f5ad2c0 sp: fffff8000f55ea41 ret_pc: 0000000000764384 [ 48.361666] RPC: <dev_set_name+0x24/0x40> [ 48.362418] l0: 0000000080000000 l1: 0000001111001607 l2: 0000000010025168 l3: 0000000000000400 [ 48.363788] l4: 0000000000000000 l5: 0000000000000001 l6: 0000000000000000 l7: 0000000000000008 [ 48.365198] i0: fffff8000f516c00 i1: 000000001000ad08 i2: 0000000000000000 i3: 00000000000000d0 [ 48.366586] i4: 0000000000000000 i5: 0000000000000000 i6: fffff8000f55eaf1 i7: 000000001000a4e4 [ 48.368098] I7: <register_virtio_device+0x64/0x100 [virtio]> [ 48.369130] Call Trace: [ 48.369740] [000000001000a4e4] register_virtio_device+0x64/0x100 [virtio] [ 48.370915] [0000000010025820] virtio_pci_probe+0xa0/0x160 [virtio_pci] [ 48.371983] [00000000006eaf40] pci_device_probe+0x80/0x100 [ 48.372879] [000000000076928c] driver_probe_device+0x16c/0x480 [ 48.373829] [0000000000769628] __driver_attach+0x88/0xa0 [ 48.374683] [0000000000766fdc] bus_for_each_dev+0x5c/0xa0 [ 48.375554] [0000000000768c1c] driver_attach+0x1c/0x40 [ 48.376408] [0000000000768710] bus_add_driver+0x1f0/0x2a0 [ 48.377281] [000000000076a114] driver_register+0x74/0x120 [ 48.378162] [00000000006e9894] __pci_register_driver+0x34/0x60 [ 48.379186] [000000001002a018] virtio_pci_driver_init+0x18/0x28 [virtio_pci] [ 48.380328] [0000000000426c58] do_one_initcall+0xb8/0x200 [ 48.381226] [0000000000525200] do_init_module+0x50/0x1f0 [ 48.382091] [00000000004c2514] load_module+0x1c54/0x23c0 [ 48.382953] [00000000004c2e74] SyS_finit_module+0x94/0xe0 [ 48.383843] [00000000004060f4] linux_sparc_syscall+0x34/0x44 Fortunately it is possible to force legacy mode with a bit of QEMU command-line fu in order to get the desired behaviour: ./qemu-system-sparc64 -drive file=/home/build/src/qemu/image/sparc64/debian-9.0-sparc64-NETINST-1.iso,if=none,media=cdrom,id=cd,index=0 -device virtio-blk-pci,disable-modern=on,drive=cd -nographic -bios /home/build/src/openbios/openbios.git/openbios/obj-sparc64/openbios-builtin.elf.nostrip -m 256 OpenBIOS for Sparc64 open isn't unique. close isn't unique. Configuration device id QEMU version 1 machine id 0 kernel cmdline CPUs: 1 x SUNW,UltraSPARC-IIi UUID: 00000000-0000-0000-0000-000000000000 Welcome to OpenBIOS v1.1 built on Aug 1 2016 18:06 Type 'help' for detailed information Trying disk:a... No valid state has been set by load or init-program 0 > boot virtio0:d (cut) [ 21.749180] ffe2f1d0: ttyS0 at MMIO 0x1fe020043f8 (irq = 5, base_baud = 115387) is a 16550A [ 21.749779] Console: ttyS0 (SU) [ 21.803436] console [ttyS0] enabled [ 21.824159] mousedev: PS/2 mouse device common for all mice [ 21.842179] rtc-m48t59 rtc-m48t59.0: rtc core: registered m48t59 as rtc0 [ 21.844768] ledtrig-cpu: registered to indicate activity on CPUs [ 21.858976] NET: Registered protocol family 10 [ 21.884821] mip6: Mobile IPv6 [ 21.885565] NET: Registered protocol family 17 [ 21.886398] mpls_gso: MPLS GSO support [ 21.892226] registered taskstats version 1 [ 21.894600] zswap: loaded using pool lzo/zbud [ 21.905840] rtc-m48t59 rtc-m48t59.0: setting system clock to 2016-08-01 20:36:46 UTC (1470083806) [ 23.025258] random: systemd-udevd urandom read with 0 bits of entropy available [ 26.240261] ne2k-pci.c:v1.03 9/22/2003 D. Becker/P. Gortmaker [ 26.364453] ne2k-pci 0000:00:04.0 eth0: RealTek RTL-8029 found at 0x1fe02008000, IRQ 6, 52:54:00:12:34:56. [ 26.386623] virtio-pci 0000:00:06.0: virtio_pci: leaving for legacy driver [ 29.109407] ne2k-pci 0000:00:04.0 enp0s4: renamed from eth0 [ 29.118803] vda: vda1 vda2 vda3 vda4 vda5 vda6 vda7 vda8 Starting system log daemon: syslogd, klogd. Given that transitional/legacy support is complicated enough, I'm quite tempted to rework this to a proper 1.0 driver before commit so the default behaviour "just works" for users. ATB, Mark.

4 years, 1 month

[PATCH 0/2] Add legacy virtio-blk driver to OpenBIOS

by Mark Cave-Ayland

This patchset implements a legacy virtio-blk driver for OpenBIOS based upon Alex Graf's s390 driver included with QEMU and enables it for SPARC64 and PPC builds. Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk> Mark Cave-Ayland (2): drivers: add legacy virtio-blk driver config: enable virtio-blk driver for default PPC and SPARC64 builds config/examples/ppc_config.xml | 1 + config/examples/sparc64_config.xml | 1 + drivers/build.xml | 1 + drivers/pci.c | 18 ++ drivers/pci_database.c | 2 +- drivers/pci_database.h | 1 + drivers/virtio.c | 413 ++++++++++++++++++++++++++++++++++++ drivers/virtio.h | 331 +++++++++++++++++++++++++++++ include/drivers/drivers.h | 4 + 9 files changed, 771 insertions(+), 1 deletion(-) create mode 100644 drivers/virtio.c create mode 100644 drivers/virtio.h -- 1.7.10.4

4 years, 1 month