public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: Tom Tromey <tromey@redhat.com>
Cc: Doug Evans <dje@google.com>, gdb-patches@sourceware.org
Subject: Re: [patch] Fix DW_OP_call2 and DW_OP_call4 for max-cache-age 0
Date: Fri, 03 Sep 2010 16:14:00 -0000	[thread overview]
Message-ID: <20100903153955.GA3236@host1.dyn.jankratochvil.net> (raw)
In-Reply-To: <m3hbi66drd.fsf@fleche.redhat.com>

On Fri, 03 Sep 2010 17:35:50 +0200, Tom Tromey wrote:
> I don't get the rationale for putting it in prepare_execute_command.
> If we are flushing the cache based on memory use, then we only need to
> consider flushing it just before we read a CU.

There is currently no way of "locking" CUs.  Some processing may arbitrarily
access more and more CUs, and even the previous ones.  Processing may need
generally unlimited number of CUs, therefore it can reach the limit and flush
still referenced CU.

Therefore I find prepare_execute_command as the only safe place to flush any
CU.

(I may miss there exist some more strict rules than I am aware of.)


Thanks,
Jan

  reply	other threads:[~2010-09-03 15:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-23 18:50 Jan Kratochvil
2010-08-23 19:30 ` Doug Evans
2010-09-02 17:13   ` Jan Kratochvil
2010-09-02 19:33     ` Doug Evans
2011-07-13 15:21       ` [patch] Fix DW_OP_call2 and DW_OP_call4 for max-cache-age 0 #2 Jan Kratochvil
2011-07-19 20:55         ` Jan Kratochvil
2010-09-03 15:42     ` [patch] Fix DW_OP_call2 and DW_OP_call4 for max-cache-age 0 Tom Tromey
2010-09-03 16:14       ` Jan Kratochvil [this message]
2010-09-03 18:06         ` Doug Evans
2010-09-06 11:29           ` Jan Kratochvil
2010-09-06 22:29             ` Doug Evans
2010-09-08 12:26             ` Tom Tromey

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=20100903153955.GA3236@host1.dyn.jankratochvil.net \
    --to=jan.kratochvil@redhat.com \
    --cc=dje@google.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@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 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 read-only IMAP folder(s) and NNTP newsgroup(s).