qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 0b0d33: TCG: Final globals clean-up

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 0b0d33: TCG: Final globals clean-up
Date: Sat, 16 Feb 2013 06:00:13 -0800 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0b0d3320db74cde233ee7855ad32a9c121d20eb4
      
https://github.com/qemu/qemu/commit/0b0d3320db74cde233ee7855ad32a9c121d20eb4
  Author: Evgeny Voevodin <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M tcg/tcg.c
    M tcg/tcg.h
    M translate-all.c

  Log Message:
  -----------
  TCG: Final globals clean-up

Signed-off-by: Evgeny Voevodin <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 5e5f07e08f7ed8c8eb9b02d9f9c3c79cf95d65ee
      
https://github.com/qemu/qemu/commit/5e5f07e08f7ed8c8eb9b02d9f9c3c79cf95d65ee
  Author: Evgeny Voevodin <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M cpu-exec.c
    M include/exec/exec-all.h
    M linux-user/main.c
    M tcg/tcg.h
    M translate-all.c

  Log Message:
  -----------
  TCG: Move translation block variables to new context inside tcg_ctx: tb_ctx

It's worth to clean-up translation blocks variables and move them
into one context as was suggested by Swirl.
Also if we use this context directly inside tcg_ctx, then it
speeds up code generation a bit.

Signed-off-by: Evgeny Voevodin <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 9a7e54242910d26d280589e1f5c7ec8814d02a6b
      
https://github.com/qemu/qemu/commit/9a7e54242910d26d280589e1f5c7ec8814d02a6b
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M bsd-user/main.c
    M cpus.c
    M include/qemu/log.h
    M include/sysemu/cpus.h
    M linux-user/main.c
    M monitor.c
    M qemu-log.c
    M vl.c

  Log Message:
  -----------
  qemu-log: Unify {cpu_set,set_cpu}_log_filename as qemu_set_log_filename

The qemu_log() functionality is no longer specific to TCG CPU debug logs.
Rename cpu_set_log_filename() to qemu_set_log_filename() and drop the
pointless wrapper set_cpu_log_filename().

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 59a6fa6e67d2335d867c66c59d992847e5b62879
      
https://github.com/qemu/qemu/commit/59a6fa6e67d2335d867c66c59d992847e5b62879
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M bsd-user/main.c
    M cpus.c
    M include/qemu/log.h
    M linux-user/main.c
    M qemu-log.c

  Log Message:
  -----------
  qemu-log: Abstract out "print usage message about valid log categories"

Abstract out the "print a human readable list of all the
valid log categories" functionality which is currently duplicated
in three separate places. (We leave the monitor.c help_cmd()
implementation as-is since it wants to send the message to
the monitor and add its own information.)

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 4fde1eba0f98779d4fdb64818071f72bb1672438
      
https://github.com/qemu/qemu/commit/4fde1eba0f98779d4fdb64818071f72bb1672438
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M bsd-user/main.c
    M cpus.c
    M include/qemu/log.h
    M linux-user/main.c
    M monitor.c
    M qemu-log.c

  Log Message:
  -----------
  qemu-log: Rename cpu_str_to_log_mask to qemu_str_to_log_mask

Rename cpu_str_to_log_mask() to qemu_str_to_log_mask(), since
the qemu_log functionality is no longer restricted to TCG CPU
debug logging.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 24537a01910f110fe3e343c13df13e48f7968a9e
      
https://github.com/qemu/qemu/commit/24537a01910f110fe3e343c13df13e48f7968a9e
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M bsd-user/main.c
    M cpus.c
    M hw/ppc.c
    M include/qemu/log.h
    M linux-user/main.c
    M monitor.c
    M qemu-log.c
    M target-i386/translate.c
    M tcg/tci/tcg-target.c

  Log Message:
  -----------
  qemu-log: Rename the public-facing cpu_set_log function to qemu_set_log

Rename the public-facing function cpu_set_log to qemu_set_log. This
requires us to rename the internal-only qemu_set_log() to
do_qemu_set_log().

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: b946bffab5e0d359accfcc78faead20fd69f26e8
      
https://github.com/qemu/qemu/commit/b946bffab5e0d359accfcc78faead20fd69f26e8
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M cpus.c
    M include/sysemu/cpus.h
    M vl.c

  Log Message:
  -----------
  cpus.c: Drop unnecessary set_cpu_log()

The set_cpu_log() function in cpus.c is a fairly simple wrapper
which is only called from one location. Just inline the code
into vl.c, since there is no need to indirect it via cpus.c
and the handling of the error case is more appropriate to vl.c.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 38dad9e574746981dfbac52b3bbbd6d894f31d26
      
https://github.com/qemu/qemu/commit/38dad9e574746981dfbac52b3bbbd6d894f31d26
  Author: Peter Maydell <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/log.h
    M monitor.c
    M qemu-log.c

  Log Message:
  -----------
  qemu-log: Rename CPULogItem, cpu_log_items to QEMULogItem, qemu_log_items

Rename the typedef CPULogItem and the public array cpu_log_items
to names that better reflect the fact that the qemu_log functionality
isn't restricted to TCG CPU debug logs any more.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 0165437302448d210742cc3def362d1de0413621
      
https://github.com/qemu/qemu/commit/0165437302448d210742cc3def362d1de0413621
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/host-utils.h

  Log Message:
  -----------
  host-utils: Add host long specific aliases for clz, ctz, ctpop

We will standardize on these names, rather than the similar routines
currently residing in qemu/bitops.h.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 72d81155d0b88ba757c5a972d3fff83027e7a6e7
      
https://github.com/qemu/qemu/commit/72d81155d0b88ba757c5a972d3fff83027e7a6e7
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/host-utils.h

  Log Message:
  -----------
  host-utils: Fix coding style and add comments

Add function comments to the routines, documenting the corner
cases upon which we are standardizing.  Fix the few instances
of non-standard coding style.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 18331e7c189507513008a2936f5fe79c3d8b8a08
      
https://github.com/qemu/qemu/commit/18331e7c189507513008a2936f5fe79c3d8b8a08
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/hbitmap.h
    M util/hbitmap.c

  Log Message:
  -----------
  hbitmap: Use non-bitops ctzl

Both uses of ctz have already eliminated zero, and thus the difference
in edge conditions between the two routines is irrelevant.

Signed-off-by: Richard Henderson <address@hidden>
Acked-by: Paolo Bonzini <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 265ce4a5ca39fb7f74a803dd61bbd9108ca6cdee
      
https://github.com/qemu/qemu/commit/265ce4a5ca39fb7f74a803dd61bbd9108ca6cdee
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M util/bitops.c

  Log Message:
  -----------
  bitops: Use non-bitops ctzl

The use of ctz has already eliminated zero, and thus the difference
in edge conditions between the two routines is irrelevant.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 5bbf90be97203c472f47da070c0040b464c0460f
      
https://github.com/qemu/qemu/commit/5bbf90be97203c472f47da070c0040b464c0460f
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M memory.c

  Log Message:
  -----------
  memory: Use non-bitops ctzl

A memory size of zero is invalid, and so that edge condition
does not occur.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 0cfa6adc7fd1eba4694515bde6bbfb9ecd892f2f
      
https://github.com/qemu/qemu/commit/0cfa6adc7fd1eba4694515bde6bbfb9ecd892f2f
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/bitops.h

  Log Message:
  -----------
  bitops: Write bitops_flsl in terms of clzl

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 14e534265aad0796690a5aea429f5bd2877ccff0
      
https://github.com/qemu/qemu/commit/14e534265aad0796690a5aea429f5bd2877ccff0
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M target-i386/topology.h

  Log Message:
  -----------
  target-i386: Inline bitops_flsl

Use clz32 directly.  Which makes slightly more sense given
that the input is type "int" and not type "long".

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 4932398fac273b8ebe5688bc4b79407a7f41edbd
      
https://github.com/qemu/qemu/commit/4932398fac273b8ebe5688bc4b79407a7f41edbd
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M util/bitops.c

  Log Message:
  -----------
  bitops: Inline bitops_flsl

This is the only remaining user.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 0f9d8bd386c9b7b17fc68fef36caa81750c39494
      
https://github.com/qemu/qemu/commit/0f9d8bd386c9b7b17fc68fef36caa81750c39494
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M util/bitops.c

  Log Message:
  -----------
  bitops: Replace bitops_ctol with ctzl

The is the only remaining user.

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 453776e5746be23c66df65fadf12e115b7d2dadd
      
https://github.com/qemu/qemu/commit/453776e5746be23c66df65fadf12e115b7d2dadd
  Author: Richard Henderson <address@hidden>
  Date:   2013-02-16 (Sat, 16 Feb 2013)

  Changed paths:
    M include/qemu/bitops.h

  Log Message:
  -----------
  bitops: Remove routines redundant with host-utils

Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


Compare: https://github.com/qemu/qemu/compare/2742f56d134b...453776e5746b
reply via email to
[Prev in Thread] Current Thread [Next in Thread]