From: Igor Mitsyanko <i.mitsyanko@samsung.com>
To: Avi Kivity <avi@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"Andreas Färber" <afaerber@suse.de>,
"Anthony Liguori" <anthony@codemonkey.ws>,
quintela@redhat.com,
"Developers qemu-devel" <qemu-devel@nongnu.org>,
"KVM devel mailing list" <kvm@vger.kernel.org>,
"Dmitry Solodkiy" <d.solodkiy@samsung.com>
Subject: Re: [Qemu-devel] KVM call agenda for tuesday 31
Date: Mon, 05 Mar 2012 18:37:23 +0400 [thread overview]
Message-ID: <4F54CFA3.6080400@samsung.com> (raw)
In-Reply-To: <4F54CA04.4070804@redhat.com>
On 03/05/2012 06:13 PM, Avi Kivity wrote:
> On 03/05/2012 03:38 PM, Igor Mitsyanko wrote:
>>> Short summary:
>>> * switch wp groups to bitfield rather than int array
>>> * convert sd.c to use memory_region_init_ram() to allocate the wp
>>> groups
>>> (being careful to use memory_region_set_dirty() when we touch them)
>>> * we don't need variable-length fields for sd.c any more
>>> * rest of the vmstate conversion is straightforward
>>>
>>
>> OK, it turned out to be not so simple, we can't use memory API in sd.c
>> because TARGET_PHYS_ADDR_BITS value (and, consequently,
>> target_phys_addr_t) is not defined for common objects.
>>
>
> Well, can't you make sd.c target dependent? It's not so nice, but it
> does solve the problem.
>
OK, but it will turn qemu from it's "long term path to suppress *all*
target specific code" :)
--
Mitsyanko Igor
ASWG, Moscow R&D center, Samsung Electronics
email: i.mitsyanko@samsung.com
WARNING: multiple messages have this Message-ID (diff)
From: Igor Mitsyanko <i.mitsyanko@samsung.com>
To: Avi Kivity <avi@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"KVM devel mailing list" <kvm@vger.kernel.org>,
quintela@redhat.com,
"Developers qemu-devel" <qemu-devel@nongnu.org>,
"Dmitry Solodkiy" <d.solodkiy@samsung.com>,
"Anthony Liguori" <anthony@codemonkey.ws>,
"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] KVM call agenda for tuesday 31
Date: Mon, 05 Mar 2012 18:37:23 +0400 [thread overview]
Message-ID: <4F54CFA3.6080400@samsung.com> (raw)
In-Reply-To: <4F54CA04.4070804@redhat.com>
On 03/05/2012 06:13 PM, Avi Kivity wrote:
> On 03/05/2012 03:38 PM, Igor Mitsyanko wrote:
>>> Short summary:
>>> * switch wp groups to bitfield rather than int array
>>> * convert sd.c to use memory_region_init_ram() to allocate the wp
>>> groups
>>> (being careful to use memory_region_set_dirty() when we touch them)
>>> * we don't need variable-length fields for sd.c any more
>>> * rest of the vmstate conversion is straightforward
>>>
>>
>> OK, it turned out to be not so simple, we can't use memory API in sd.c
>> because TARGET_PHYS_ADDR_BITS value (and, consequently,
>> target_phys_addr_t) is not defined for common objects.
>>
>
> Well, can't you make sd.c target dependent? It's not so nice, but it
> does solve the problem.
>
OK, but it will turn qemu from it's "long term path to suppress *all*
target specific code" :)
--
Mitsyanko Igor
ASWG, Moscow R&D center, Samsung Electronics
email: i.mitsyanko@samsung.com
next prev parent reply other threads:[~2012-03-05 14:37 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-30 18:55 KVM call agenda for tuesday 31 Juan Quintela
2012-01-30 18:55 ` [Qemu-devel] " Juan Quintela
2012-01-30 23:41 ` Andreas Färber
2012-01-30 23:41 ` Andreas Färber
2012-01-30 23:53 ` Anthony Liguori
2012-01-30 23:53 ` [Qemu-devel] " Anthony Liguori
2012-01-31 0:37 ` Alexander Graf
2012-01-31 0:37 ` Alexander Graf
2012-01-31 10:49 ` Lluís Vilanova
2012-01-31 13:15 ` Andreas Färber
2012-01-31 13:15 ` Andreas Färber
2012-01-31 14:01 ` Mitsyanko Igor
2012-01-31 14:01 ` Mitsyanko Igor
2012-08-08 16:25 ` Andreas Färber
2012-08-08 16:25 ` Andreas Färber
2012-08-09 7:53 ` Igor Mitsyanko
2012-08-09 7:53 ` Igor Mitsyanko
2012-01-31 14:12 ` Anthony Liguori
2012-01-31 14:12 ` Anthony Liguori
2012-01-31 15:04 ` Michael S. Tsirkin
2012-01-31 15:04 ` [Qemu-devel] " Michael S. Tsirkin
2012-01-31 15:10 ` Michael S. Tsirkin
2012-01-31 15:10 ` [Qemu-devel] " Michael S. Tsirkin
2012-01-31 15:12 ` Paolo Bonzini
2012-01-31 15:12 ` Paolo Bonzini
2012-02-09 22:23 ` Peter Maydell
2012-02-09 22:23 ` [Qemu-devel] " Peter Maydell
2012-02-09 22:37 ` Anthony Liguori
2012-02-09 22:37 ` Anthony Liguori
2012-02-09 22:42 ` Peter Maydell
2012-02-09 22:42 ` [Qemu-devel] " Peter Maydell
2012-02-09 23:17 ` Andreas Färber
2012-02-09 23:17 ` Andreas Färber
2012-02-09 23:21 ` Alexander Graf
2012-02-09 23:21 ` Alexander Graf
2012-02-21 15:33 ` Peter Maydell
2012-02-21 15:33 ` Peter Maydell
2012-02-22 8:06 ` Mitsyanko Igor
2012-02-22 8:06 ` Mitsyanko Igor
2012-03-05 13:38 ` Igor Mitsyanko
2012-03-05 13:38 ` Igor Mitsyanko
2012-03-05 14:13 ` Avi Kivity
2012-03-05 14:13 ` Avi Kivity
2012-03-05 14:37 ` Igor Mitsyanko [this message]
2012-03-05 14:37 ` Igor Mitsyanko
2012-03-05 15:10 ` Avi Kivity
2012-03-05 15:10 ` Avi Kivity
2012-03-05 15:15 ` Anthony Liguori
2012-03-05 15:15 ` Anthony Liguori
2012-03-05 15:17 ` Avi Kivity
2012-03-05 15:17 ` Avi Kivity
2012-03-05 18:53 ` Blue Swirl
2012-03-05 18:53 ` Blue Swirl
2012-03-05 20:58 ` malc
2012-03-05 20:58 ` [Qemu-devel] " malc
2012-03-05 15:20 ` Peter Maydell
2012-03-05 15:20 ` Peter Maydell
2012-03-05 15:21 ` Avi Kivity
2012-03-05 15:21 ` Avi Kivity
2012-03-05 15:27 ` Peter Maydell
2012-03-05 15:27 ` Peter Maydell
2012-03-05 15:43 ` Andreas Färber
2012-03-05 15:43 ` [Qemu-devel] " Andreas Färber
2012-03-05 15:47 ` Avi Kivity
2012-03-05 15:47 ` Avi Kivity
2012-03-05 15:50 ` Peter Maydell
2012-03-05 15:50 ` Peter Maydell
2012-03-05 17:27 ` Avi Kivity
2012-03-05 17:27 ` Avi Kivity
2012-03-12 9:43 ` Igor Mitsyanko
2012-03-12 9:43 ` Igor Mitsyanko
2012-03-14 9:30 ` Igor Mitsyanko
2012-03-14 9:30 ` Igor Mitsyanko
2012-03-14 10:16 ` Avi Kivity
2012-03-14 10:16 ` Avi Kivity
2012-01-31 13:59 ` Anthony Liguori
2012-01-31 13:59 ` Anthony Liguori
2012-01-31 14:09 ` Avi Kivity
2012-01-31 14:09 ` Avi Kivity
2012-01-31 14:17 ` Anthony Liguori
2012-01-31 14:44 ` Avi Kivity
2012-01-31 16:23 ` Andreas Färber
2012-01-31 16:23 ` Andreas Färber
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F54CFA3.6080400@samsung.com \
--to=i.mitsyanko@samsung.com \
--cc=afaerber@suse.de \
--cc=anthony@codemonkey.ws \
--cc=avi@redhat.com \
--cc=d.solodkiy@samsung.com \
--cc=kvm@vger.kernel.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.