* [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot [not found] ` <470CD22C.5050003@qumranet.com> @ 2007-10-10 13:47 ` Aurelien Jarno 2007-11-08 6:41 ` Avi Kivity 0 siblings, 1 reply; 6+ messages in thread From: Aurelien Jarno @ 2007-10-10 13:47 UTC (permalink / raw) To: Avi Kivity; +Cc: kvm-devel, qemu-devel Avi Kivity a écrit : > Aurelien Jarno wrote: >> Avi Kivity a écrit : >> >>> Aurelien Jarno wrote: >>> >>>>>> I also confirm that using -no-acpi fixes the problem. However, I have >>>>>> seen strange data corruption, even on Intel. >>>>>> >>>>>> Basically, booting a recently installed FreeBSD leads to the following >>>>>> message from the bootloader: "No kernel found". And the next time, I get >>>>>> from the *BIOS*: "Boot from Hard Disk failed: not a bootable disk". >>>>>> >>>>>> Looking at the disk image, the partition table (and maybe more?) has >>>>>> disappeared. This is with a raw image disk. >>>>>> [snip] >> After a few more tests, I have been able to reproduce it with the >> current CVS version of QEMU. I conclude this is a QEMU or a BIOS >> problem, and not a KVM one. >> >> Sorry for the noise. >> > > Well, it still needs to be fixed. I think there's a git import of qemu > available somewhere, that can be used for bisecting. > Well the IDE code hasn't changed a lot recently, so I checked the CVS history and easily (first test) found the commit that causes the problem: Last AIO patch, by Vladimir N. Oleynik. http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 -- .''`. Aurelien Jarno | GPG: 1024D/F1BCDB73 : :' : Debian developer | Electrical Engineer `. `' aurel32@debian.org | aurelien@aurel32.net `- people.debian.org/~aurel32 | www.aurel32.net ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot 2007-10-10 13:47 ` [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot Aurelien Jarno @ 2007-11-08 6:41 ` Avi Kivity 2007-11-08 9:25 ` Fabrice Bellard 2007-11-08 9:44 ` Aurelien Jarno 0 siblings, 2 replies; 6+ messages in thread From: Avi Kivity @ 2007-11-08 6:41 UTC (permalink / raw) To: Aurelien Jarno; +Cc: kvm-devel, qemu-devel Aurelien Jarno wrote: > Well the IDE code hasn't changed a lot recently, so I checked the CVS > history and easily (first test) found the commit that causes the problem: > > Last AIO patch, by Vladimir N. Oleynik. > > http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 > > Have we learned something about this issue? Should I revert the patch from kvm? Should qemu? -- Do not meddle in the internals of kernels, for they are subtle and quick to panic. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot 2007-11-08 6:41 ` Avi Kivity @ 2007-11-08 9:25 ` Fabrice Bellard 2007-11-08 10:00 ` Avi Kivity 2007-11-08 9:44 ` Aurelien Jarno 1 sibling, 1 reply; 6+ messages in thread From: Fabrice Bellard @ 2007-11-08 9:25 UTC (permalink / raw) To: qemu-devel Avi Kivity wrote: > Aurelien Jarno wrote: >> Well the IDE code hasn't changed a lot recently, so I checked the CVS >> history and easily (first test) found the commit that causes the problem: >> >> Last AIO patch, by Vladimir N. Oleynik. >> >> http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 >> >> > > Have we learned something about this issue? Should I revert the patch > from kvm? Should qemu? Sure. The conversion from sync to async mode done in this patch is incomplete to say the least. Fabrice. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot 2007-11-08 9:25 ` Fabrice Bellard @ 2007-11-08 10:00 ` Avi Kivity 2007-11-08 10:22 ` Fabrice Bellard 0 siblings, 1 reply; 6+ messages in thread From: Avi Kivity @ 2007-11-08 10:00 UTC (permalink / raw) To: qemu-devel Fabrice Bellard wrote: > Avi Kivity wrote: >> Aurelien Jarno wrote: >>> Well the IDE code hasn't changed a lot recently, so I checked the CVS >>> history and easily (first test) found the commit that causes the >>> problem: >>> >>> Last AIO patch, by Vladimir N. Oleynik. >>> >>> http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 >>> >>> >>> >> >> Have we learned something about this issue? Should I revert the patch >> from kvm? Should qemu? > > Sure. The conversion from sync to async mode done in this patch is > incomplete to say the least. > Is reverting the patch safe, or will it cause new problems? (I'm not sure what "Sure" above refers to) -- error compiling committee.c: too many arguments to function ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot 2007-11-08 10:00 ` Avi Kivity @ 2007-11-08 10:22 ` Fabrice Bellard 0 siblings, 0 replies; 6+ messages in thread From: Fabrice Bellard @ 2007-11-08 10:22 UTC (permalink / raw) To: qemu-devel Avi Kivity wrote: > Fabrice Bellard wrote: >> Avi Kivity wrote: >>> Aurelien Jarno wrote: >>>> Well the IDE code hasn't changed a lot recently, so I checked the CVS >>>> history and easily (first test) found the commit that causes the >>>> problem: >>>> >>>> Last AIO patch, by Vladimir N. Oleynik. >>>> >>>> http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 >>>> >>>> >>>> >>> >>> Have we learned something about this issue? Should I revert the patch >>> from kvm? Should qemu? >> >> Sure. The conversion from sync to async mode done in this patch is >> incomplete to say the least. >> > > Is reverting the patch safe, or will it cause new problems? The patch must be reverted because it implements incorrectly the AIO write case (e.g. nsector is modified before the IO actually takes place). The previous code was at least correct if win2k_install_hack was not enabled and if one considered than the PIO I/Os were instantaneous. Regards, Fabrice. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot 2007-11-08 6:41 ` Avi Kivity 2007-11-08 9:25 ` Fabrice Bellard @ 2007-11-08 9:44 ` Aurelien Jarno 1 sibling, 0 replies; 6+ messages in thread From: Aurelien Jarno @ 2007-11-08 9:44 UTC (permalink / raw) To: Avi Kivity; +Cc: kvm-devel, qemu-devel Avi Kivity a écrit : > Aurelien Jarno wrote: >> Well the IDE code hasn't changed a lot recently, so I checked the CVS >> history and easily (first test) found the commit that causes the problem: >> >> Last AIO patch, by Vladimir N. Oleynik. >> >> http://cvs.savannah.nongnu.org/viewvc/qemu/hw/ide.c?root=qemu&r1=1.64&r2=1.65 >> >> > > Have we learned something about this issue? Should I revert the patch > from kvm? Should qemu? > >From my side, I haven't found time to debug the problem. Instead I am reverting this patch locally each time I install qemu or kvm. -- .''`. Aurelien Jarno | GPG: 1024D/F1BCDB73 : :' : Debian developer | Electrical Engineer `. `' aurel32@debian.org | aurelien@aurel32.net `- people.debian.org/~aurel32 | www.aurel32.net ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-11-08 10:23 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <47022E7E.3030509@aurel32.net> [not found] ` <470234DF.6030802@qumranet.com> [not found] ` <470237DC.5060008@aurel32.net> [not found] ` <47027161.50004@qumranet.com> [not found] ` <47027586.9020508@aurel32.net> [not found] ` <47027A01.5080903@aurel32.net> [not found] ` <47027F25.80001@aurel32.net> [not found] ` <470CB4DE.20707@qumranet.com> [not found] ` <470CD1F1.20305@aurel32.net> [not found] ` <470CD22C.5050003@qumranet.com> 2007-10-10 13:47 ` [Qemu-devel] Re: [kvm-devel] FreeBSD image hangs during boot Aurelien Jarno 2007-11-08 6:41 ` Avi Kivity 2007-11-08 9:25 ` Fabrice Bellard 2007-11-08 10:00 ` Avi Kivity 2007-11-08 10:22 ` Fabrice Bellard 2007-11-08 9:44 ` Aurelien Jarno
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).