Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-08-19 Thread Peter Maydell
On 13 August 2015 at 18:37, Peter Maydell wrote: > On 22 July 2015 at 17:43, Peter Maydell wrote: >> This series makes a start at cleaning up some of our headers >> to avoid the common problem of header files including qemu-common.h >> (which then in turn can lead to awkward circular includes). >

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-08-13 Thread Peter Maydell
On 22 July 2015 at 17:43, Peter Maydell wrote: > This series makes a start at cleaning up some of our headers > to avoid the common problem of header files including qemu-common.h > (which then in turn can lead to awkward circular includes). The general consensus here seems to be that patches 1-9

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-07-31 Thread Daniel P. Berrange
On Fri, Jul 31, 2015 at 01:57:29PM +0100, Peter Maydell wrote: > On 31 July 2015 at 13:54, Daniel P. Berrange wrote: > > I broadly like the direction of this change. I think it is > > probably worth being strict about requiring #include of > > osdep.h as the first header in every .c file. And vali

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-07-31 Thread Peter Maydell
On 31 July 2015 at 13:54, Daniel P. Berrange wrote: > I broadly like the direction of this change. I think it is > probably worth being strict about requiring #include of > osdep.h as the first header in every .c file. And validating > such a rule programmatically at make check time or equiv. Yea

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-07-31 Thread Daniel P. Berrange
On Wed, Jul 22, 2015 at 05:43:59PM +0100, Peter Maydell wrote: > This series makes a start at cleaning up some of our headers > to avoid the common problem of header files including qemu-common.h > (which then in turn can lead to awkward circular includes). > > One common cause of this is that we

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-07-23 Thread Markus Armbruster
Peter Maydell writes: > This series makes a start at cleaning up some of our headers > to avoid the common problem of header files including qemu-common.h > (which then in turn can lead to awkward circular includes). Yes, please! > One common cause of this is that we don't have any header > whi

Re: [Qemu-devel] [PATCH 00/12] qemu-common.h/osdep.h refactoring

2015-07-22 Thread Paolo Bonzini
On 22/07/2015 18:43, Peter Maydell wrote: > The real question here I guess is whether people like the > direction I'm trying to go with this. If so, we can further > reduce the number of qemu-common.h includes without too much > difficulty with further patches. I like it, and it nicely complemen