qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v3 0/2] tcg: allocate TB structs preceding translate
@ 2017-06-06 23:12 Emilio G. Cota
  2017-06-06 23:12 ` [Qemu-devel] [PATCH v3 1/2] util: add cacheinfo Emilio G. Cota
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Emilio G. Cota @ 2017-06-06 23:12 UTC (permalink / raw)
  To: qemu-devel
  Cc: Richard Henderson, alex.bennee, Peter Maydell, Paolo Bonzini,
	Pranith Kumar, Geert Martin Ijewski

v2: https://lists.gnu.org/archive/html/qemu-devel/2017-06/msg01147.html

Changes from v2:

- Dropped the configure-time check. Instead, do this at init-time
  as suggested by Richard.

- Added Pranith's R-b tag to the TCG patch.

- Added cache info query for Windows thanks to Geerd's help.

- Added cache info query for MacOS [ #ifdef __APPLE__ ] via sysctl. (Note:
  I only tested the code on a standalone program, since I cannot install
  QEMU's dependences on the MacOS machine I got access to)

- Use int instead of unsigned int for the line sizes. This allows
  us to use ROUND_UP without having to cast to int (ROUND_UP negates
  the alignment parameter)

Question: is it worth adding a paranoid check to make sure the linesizes
we obtain are a pow2? If there's a bug somewhere in the reporting,
we'll get in trouble due to ROUND_UP, which only works with pow2
parameters. A safer alternative is to use QEMU_ALIGN_UP, but it's slower.

This patchset applies cleanly on top of the current master (65dfad62a).

Thanks,

		Emilio

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-06-07 14:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-06 23:12 [Qemu-devel] [PATCH v3 0/2] tcg: allocate TB structs preceding translate Emilio G. Cota
2017-06-06 23:12 ` [Qemu-devel] [PATCH v3 1/2] util: add cacheinfo Emilio G. Cota
2017-06-07  0:17   ` [Qemu-devel] [PATCH v3 1/2 -- fixed] " Emilio G. Cota
2017-06-07 14:05     ` Pranith Kumar
2017-06-06 23:12 ` [Qemu-devel] [PATCH v3 2/2] tcg: allocate TB structs before the corresponding translated code Emilio G. Cota
2017-06-06 23:38 ` [Qemu-devel] [PATCH v3 0/2] tcg: allocate TB structs preceding translate no-reply

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 NNTP newsgroup(s).