From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:60759) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T8ZqH-00070l-RL for qemu-devel@nongnu.org; Mon, 03 Sep 2012 12:49:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T8ZqG-0001I4-T4 for qemu-devel@nongnu.org; Mon, 03 Sep 2012 12:49:41 -0400 Received: from david.siemens.de ([192.35.17.14]:23966) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T8ZqG-0001Hv-IS for qemu-devel@nongnu.org; Mon, 03 Sep 2012 12:49:40 -0400 Message-ID: <5044DD67.6070003@siemens.com> Date: Mon, 03 Sep 2012 18:40:07 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <1346640974-30974-1-git-send-email-mmogilvi_qemu@miniinfo.net> <1346640974-30974-6-git-send-email-mmogilvi_qemu@miniinfo.net> <50446D11.5050904@suse.de> <5044C10D.7050600@redhat.com> <87fw6z5d0e.fsf@elfo.mitica> <5044D243.3050506@redhat.com> <5044D2A7.7000609@siemens.com> <5044D36E.3060505@redhat.com> <5044D494.3070304@siemens.com> <5044D78D.1060803@redhat.com> <5044D96C.3000406@redhat.com> <5044DB34.7030305@redhat.com> <5044DBEC.4020601@redhat.com> In-Reply-To: <5044DBEC.4020601@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 5/5] i8259: fix dynamically masking slave IRQs with IMR register List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: "qemu-devel@nongnu.org" , "quintela@redhat.com" , Avi Kivity , =?UTF-8?B?QW5kcmVhcyBGw6Q=?= =?UTF-8?B?cmJlcg==?= , Matthew Ogilvie On 2012-09-03 18:33, Paolo Bonzini wrote: > Il 03/09/2012 18:30, Avi Kivity ha scritto: >>>>>> The values above are what every user of the PIC cascaded on our targets >>>>>>>>>> must program to use them. So We will find them in the state once any >>>>>>>>>> relevant guest code was able to run (e.g. the BIOS). >>>>>>>>>> >>>>>> Suppose the bios has not run yet? >>>> >>>> Then you transmit the subsection. >> And the migration fails. Needlessly, since icw3 == 0 doesn't affect >> guest operation. > > But the point of subsections is to succeed migration in the common case, > assuming there is more than one case that doesn't affect guest operation. The point is that the common case is icw3 = 4 (for the master), not 0. And if we don't save that case, we must save the rest. If we don't save this as well, we lose state information. This can't work. Jan -- Siemens AG, Corporate Technology, CT RTC ITP SDP-DE Corporate Competence Center Embedded Linux