Re: [Qemu-devel] [PATCH] memory: Rename readable flag to romd_mode

[Paolo Bonzini <pbonzini@redhat.com>]

Il 06/05/2013 18:12, Jan Kiszka ha scritto:
> "Readable" is a very unfortunate name for this flag because even a
> rom_device region will always be readable from the guest POV. What
> differs is the mapping, just like the comments had to explain already.
> Also, readable could currently be understood as being a generic region
> flag, but it only applies to rom_device regions.
> 
> So name the flag and the function to modify it after the original term
> "ROMD" which could also be interpreted as "ROM direct", i.e. ROM mode
> with direct access. In any case, the scope if the flag is clearer now.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
> 
> Depends on "memory: Replace open-coded memory_region_is_romd", but I
> don't think this patch is "trivial" as well, though it is just renaming
> things. Just in case, I'm CCing the trivial list as well.
> 
> Paolo, or would you open a memory branch so that related stuff can flow
> through one queue?

I think this is more trivial than the other actually.  :)  (For code
changes, I think trivial roughly means "if it compiles fine, it won't
break anything").

For now I applied both patches to my iommu branch.  I'll test it a bit
more tomorrow and then post the IOMMU patches for review.  We should
merge them and the subpage reorg soon in 1.6, since it is (together with
ref/unref) the main serialization point for all the work on BQL-less
dispatch.

BTW, a "trivial" patch would be one that actually defines the term "BQL"
somewhere in the code. :)

Paolo