[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions |
Date: |
Tue, 15 Oct 2013 14:36:58 +0100 |
Ping^2, seven weeks after initial post and code review.
Frankly, our mechanism for committing patches which aren't
obviously localised to a specific subsystem sucks :-(
It seems like pretty much every time it takes multiple
pings and a month or two before anything gets into the tree.
(at least this one doesn't require a rebase...)
thanks
-- PMM
On 9 September 2013 11:38, Peter Maydell <address@hidden> wrote:
> Ping!
>
> thanks
> -- PMM
>
>
> On 25 August 2013 15:59, Peter Maydell <address@hidden> wrote:
>> The bswap.h header includes a set of "legacy unaligned functions"
>> that (since commit c732a52d3 at the beginning of this year) are
>> just wrappers for underlying {ld,st}<type> functions. The legacy
>> functions aren't used in many places, so just replace all their
>> uses with uses of the new-style {ld,st} functions; this lets us
>> remove the legacy wrappers altogether.
>>
>> Since we know the {ld,st}* routines are definitely functions,
>> we can in the process remove some casts which were left over
>> from when the legacy unaligned functions were previously macros.
>>
>> The patchset is divided up by function being removed, rather
>> than by which device/subsystem is being fixed; I think this way
>> round is easier to review since you only have to keep one
>> substitution in your head when reading a patch.
>>
>> Peter Maydell (9):
>> bswap.h: Remove cpu_to_le16wu()
>> bswap.h: Remove cpu_to_le32wu()
>> bswap.h: Remove le16_to_cpupu()
>> bswap.h: Remove le32_to_cpupu()
>> bswap.h: Remove be32_to_cpupu()
>> bswap.h: Remove cpu_to_be16wu()
>> bswap.h: Remove cpu_to_be32wu()
>> bswap.h: Remove cpu_to_be64wu()
>> bswap.h: Remove cpu_to_32wu()
>>
>> block/qcow2-cluster.c | 2 +-
>> hw/acpi/core.c | 3 +--
>> hw/block/cdrom.c | 10 +++++-----
>> hw/display/vga_template.h | 14 ++++++++------
>> hw/ide/atapi.c | 16 +++++++--------
>> hw/net/e1000.c | 22 +++++++++------------
>> hw/net/ne2000.c | 4 ++--
>> hw/pci/pcie_aer.c | 4 ++--
>> include/hw/pci/pci.h | 8 ++++----
>> include/qemu/bswap.h | 47
>> ---------------------------------------------
>> 10 files changed, 40 insertions(+), 90 deletions(-)
>>
>> --
>> 1.7.9.5
- Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions,
Peter Maydell <=