qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 2/5] register: Add Register API


From: Gerd Hoffmann
Subject: Re: [Qemu-devel] [PATCH v2 2/5] register: Add Register API
Date: Mon, 15 Apr 2013 09:11:54 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5

  Hi,

> - Add a "lite" flag that is (automatically) computed once that takes a
> fast path through the read/write handlers. A reg is lite if it
> requires no read-modify-write (no ro, nw, ge, ui, wtc). Only the
> actual write and post_write handlers are executed.

Sounds good to me.

> - Remove the byte loops. Just switch on register size and cast to
> uint_XXt. Remove support for non power-of-two size registers and
> non-host endianess (needed for potential PCI conversion) but should
> make the code faster.

This too.

Also: you might want to pass both old and new value to the post-write
hook so the handler can easily figure what has actually changed.

cheers,
  Gerd



reply via email to

[Prev in Thread] Current Thread [Next in Thread]