Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
On 11/24/2009 02:16 PM, Glauber Costa wrote: That commit is already merged. Applying it ontop of 5d968c942adbc48188dbbcf7d85e69c2e0d7cd8a, which is your merge commit that includes my patch, make qemu-kvm go from non-working to working again. So the problem must be something else. It is my misport of Jan's irq bitmap refactoring (init/reset was not ported); pretty easy to fix. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
>> FYI: Combining that patch with a later fix from Juan fix the issue for >> qemu-kvm. >> (504c2948) >> Also, the patch seems to apply fine ontop of next. >> >> I can send a combined version if you want, or you can combine it yourself >> >> > > That commit is already merged. > Applying it ontop of 5d968c942adbc48188dbbcf7d85e69c2e0d7cd8a, which is your merge commit that includes my patch, make qemu-kvm go from non-working to working again. So the problem must be something else. -- Glauber Costa. "Free as in Freedom" http://glommer.net "The less confident you are, the more serious you have to act." -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
On 11/23/2009 11:07 PM, Glauber Costa wrote: On Mon, Nov 23, 2009 at 5:00 PM, Glauber Costa wrote: On Sun, Nov 22, 2009 at 04:42:12PM +0200, Avi Kivity wrote: A qemu-kvm which merges this commit breaks badly (see qemu-kvm.git next branch). In the commit log for this commit, you write I tested it with qemu (with and without io-thread) and qemu-kvm, and it seems to be doing okay - although qemu-kvm uses a slightly different patch. Can you share the slightly different patch (against 'next') please? Sorry, I don't follow. You said you tested it and it works, so what exactly do we need from me here? FYI: Combining that patch with a later fix from Juan fix the issue for qemu-kvm. (504c2948) Also, the patch seems to apply fine ontop of next. I can send a combined version if you want, or you can combine it yourself That commit is already merged. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
On 11/23/2009 09:00 PM, Glauber Costa wrote: On Sun, Nov 22, 2009 at 04:42:12PM +0200, Avi Kivity wrote: A qemu-kvm which merges this commit breaks badly (see qemu-kvm.git next branch). In the commit log for this commit, you write "you" = Glauber Costa I tested it with qemu (with and without io-thread) and qemu-kvm, and it seems to be doing okay - although qemu-kvm uses a slightly different patch. Can you share the slightly different patch (against 'next') please? Sorry, I don't follow. You said you tested it and it works, so what exactly do we need from me here? No. _You_ tested it and it worked for you. qemu-kvm next fails badly (doesn't even run the bios). I need help from you in finding out why and fixing it. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
On Mon, Nov 23, 2009 at 5:00 PM, Glauber Costa wrote: > On Sun, Nov 22, 2009 at 04:42:12PM +0200, Avi Kivity wrote: >> A qemu-kvm which merges this commit breaks badly (see qemu-kvm.git next >> branch). In the commit log for this commit, you write >> >> I tested it with qemu (with and without io-thread) and qemu-kvm, and it >> seems to be doing okay - although qemu-kvm uses a slightly different >> patch. >> >> Can you share the slightly different patch (against 'next') please? > Sorry, I don't follow. You said you tested it and it works, so what exactly > do we need from me here? FYI: Combining that patch with a later fix from Juan fix the issue for qemu-kvm. (504c2948) Also, the patch seems to apply fine ontop of next. I can send a combined version if you want, or you can combine it yourself -- Glauber Costa. "Free as in Freedom" http://glommer.net "The less confident you are, the more serious you have to act." -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
On Sun, Nov 22, 2009 at 04:42:12PM +0200, Avi Kivity wrote: > A qemu-kvm which merges this commit breaks badly (see qemu-kvm.git next > branch). In the commit log for this commit, you write > > I tested it with qemu (with and without io-thread) and qemu-kvm, and it > seems to be doing okay - although qemu-kvm uses a slightly different > patch. > > Can you share the slightly different patch (against 'next') please? Sorry, I don't follow. You said you tested it and it works, so what exactly do we need from me here? -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Breakage due to commit c1699988 ("v3: don't call reset functions on cpu initialization")
A qemu-kvm which merges this commit breaks badly (see qemu-kvm.git next branch). In the commit log for this commit, you write I tested it with qemu (with and without io-thread) and qemu-kvm, and it seems to be doing okay - although qemu-kvm uses a slightly different patch. Can you share the slightly different patch (against 'next') please? -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html