Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-10-16 Thread Michael S. Tsirkin
On Tue, Oct 15, 2013 at 02:36:58PM +0100, Peter Maydell wrote: > 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 >

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-10-15 Thread Michael S. Tsirkin
On Sun, Aug 25, 2013 at 03:59:28PM +0100, Peter Maydell 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} functions. The legacy > functions aren't used in many places, s

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-10-15 Thread Peter Maydell
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 thi

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-09-09 Thread Peter Maydell
On 9 September 2013 12:54, Michael S. Tsirkin wrote: > On Sun, Aug 25, 2013 at 03:59:28PM +0100, Peter Maydell 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} func

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-09-09 Thread Michael S. Tsirkin
On Sun, Aug 25, 2013 at 03:59:28PM +0100, Peter Maydell 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} functions. The legacy > functions aren't used in many places, s

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-09-09 Thread Peter Maydell
Ping! thanks -- PMM On 25 August 2013 15:59, Peter Maydell 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} functions. The legacy > functions aren't used in many pl

Re: [Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-08-26 Thread Richard Henderson
On 08/25/2013 07:59 AM, Peter Maydell wrote: > 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: Remo

[Qemu-devel] [PATCH 0/9] Remove legacy unaligned bswap functions

2013-08-25 Thread Peter Maydell
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} functions. The legacy functions aren't used in many places, so just replace all their uses with uses of the new-style {ld,st} functi