From: Paolo Bonzini <pbonzini@redhat.com>
To: liguang <lig.fnst@cn.fujitsu.com>
Cc: qemu-trivial <qemu-trivial@nongnu.org>,
"Jan Kiszka" <jan.kiszka@siemens.com>,
qemu-devel <qemu-devel@nongnu.org>,
"Andreas Färber" <afaerber@suse.de>,
"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-trivial] [PATCH] cputlb: remove dead function tlb_update_dirty
Date: Tue, 03 Sep 2013 09:22:44 +0200 [thread overview]
Message-ID: <52258E44.9010704@redhat.com> (raw)
In-Reply-To: <1378191917-6964-1-git-send-email-lig.fnst@cn.fujitsu.com>
Il 03/09/2013 09:05, liguang ha scritto:
> Signed-off-by: liguang <lig.fnst@cn.fujitsu.com>
> ---
> cputlb.c | 15 ---------------
> 1 files changed, 0 insertions(+), 15 deletions(-)
>
> diff --git a/cputlb.c b/cputlb.c
> index 977c0ca..08e50e0 100644
> --- a/cputlb.c
> +++ b/cputlb.c
> @@ -169,21 +169,6 @@ static inline ram_addr_t qemu_ram_addr_from_host_nofail(void *ptr)
> return ram_addr;
> }
>
> -static inline void tlb_update_dirty(CPUTLBEntry *tlb_entry)
> -{
> - ram_addr_t ram_addr;
> - void *p;
> -
> - if (tlb_is_dirty_ram(tlb_entry)) {
> - p = (void *)(uintptr_t)((tlb_entry->addr_write & TARGET_PAGE_MASK)
> - + tlb_entry->addend);
> - ram_addr = qemu_ram_addr_from_host_nofail(p);
> - if (!cpu_physical_memory_is_dirty(ram_addr)) {
> - tlb_entry->addr_write |= TLB_NOTDIRTY;
> - }
> - }
> -}
> -
> void cpu_tlb_reset_dirty_all(ram_addr_t start1, ram_addr_t length)
> {
> CPUState *cpu;
>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
and CCing qemu-trivial.
Paolo
WARNING: multiple messages have this Message-ID (diff)
From: Paolo Bonzini <pbonzini@redhat.com>
To: liguang <lig.fnst@cn.fujitsu.com>
Cc: qemu-trivial <qemu-trivial@nongnu.org>,
"Jan Kiszka" <jan.kiszka@siemens.com>,
qemu-devel <qemu-devel@nongnu.org>,
"Andreas Färber" <afaerber@suse.de>,
"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty
Date: Tue, 03 Sep 2013 09:22:44 +0200 [thread overview]
Message-ID: <52258E44.9010704@redhat.com> (raw)
In-Reply-To: <1378191917-6964-1-git-send-email-lig.fnst@cn.fujitsu.com>
Il 03/09/2013 09:05, liguang ha scritto:
> Signed-off-by: liguang <lig.fnst@cn.fujitsu.com>
> ---
> cputlb.c | 15 ---------------
> 1 files changed, 0 insertions(+), 15 deletions(-)
>
> diff --git a/cputlb.c b/cputlb.c
> index 977c0ca..08e50e0 100644
> --- a/cputlb.c
> +++ b/cputlb.c
> @@ -169,21 +169,6 @@ static inline ram_addr_t qemu_ram_addr_from_host_nofail(void *ptr)
> return ram_addr;
> }
>
> -static inline void tlb_update_dirty(CPUTLBEntry *tlb_entry)
> -{
> - ram_addr_t ram_addr;
> - void *p;
> -
> - if (tlb_is_dirty_ram(tlb_entry)) {
> - p = (void *)(uintptr_t)((tlb_entry->addr_write & TARGET_PAGE_MASK)
> - + tlb_entry->addend);
> - ram_addr = qemu_ram_addr_from_host_nofail(p);
> - if (!cpu_physical_memory_is_dirty(ram_addr)) {
> - tlb_entry->addr_write |= TLB_NOTDIRTY;
> - }
> - }
> -}
> -
> void cpu_tlb_reset_dirty_all(ram_addr_t start1, ram_addr_t length)
> {
> CPUState *cpu;
>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
and CCing qemu-trivial.
Paolo
next prev parent reply other threads:[~2013-09-03 7:23 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-03 7:05 [Qemu-devel] [PATCH] cputlb: remove dead function tlb_update_dirty liguang
2013-09-03 7:22 ` Paolo Bonzini [this message]
2013-09-03 7:22 ` Paolo Bonzini
2013-09-03 8:35 ` [Qemu-trivial] " Andreas Färber
2013-09-03 8:35 ` [Qemu-devel] " Andreas Färber
2013-09-03 8:41 ` [Qemu-trivial] " Paolo Bonzini
2013-09-03 8:41 ` [Qemu-devel] " Paolo Bonzini
2013-09-03 8:48 ` [Qemu-trivial] " Peter Maydell
2013-09-03 8:48 ` [Qemu-devel] " Peter Maydell
2013-09-03 11:17 ` [Qemu-trivial] " Michael Tokarev
2013-09-03 11:17 ` [Qemu-devel] " Michael Tokarev
2013-09-03 16:54 ` Andreas Färber
2013-09-03 16:54 ` [Qemu-devel] " Andreas Färber
2013-09-04 1:36 ` Li Guang
2013-09-04 1:36 ` [Qemu-devel] " Li Guang
2013-09-04 10:59 ` Paolo Bonzini
2013-09-04 10:59 ` [Qemu-devel] " Paolo Bonzini
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=52258E44.9010704@redhat.com \
--to=pbonzini@redhat.com \
--cc=afaerber@suse.de \
--cc=jan.kiszka@siemens.com \
--cc=lig.fnst@cn.fujitsu.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@nongnu.org \
--cc=rth@twiddle.net \
/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.