"Michael S. Tsirkin" <m...@redhat.com> wrote: > On Thu, Aug 09, 2018 at 08:03:03PM +0100, Dr. David Alan Gilbert wrote: >> * Juan Quintela (quint...@redhat.com) wrote: >> > They are all already included in virtio_pci.h. > > All I see in virtio_pci.h is: > > #include "standard-headers/linux/types.h" > > Weird. > > BTW what's the point of this patch? Generally it's best not to depend > on headers including each other, it makes refactoring harder.
If you see the following patches, we remove blk, net, serial, scsi, balloon and input bits from that file, so I was removing includes patch by patch. And at the end, I found that we only need that ones. "virtio-pci.h" does too many things here, I could have split it also, because the mayority of the bits are only used now inside their own virtio-foo-pci.c. But then, there are things that share bits, virtio-bus-pci is used for lots of stuff, virtio-input-pci bits are used in virtio-input-host-pci.c, etc, So I decided to only do the direct split. And about including directly all the files that you use, and including only the files that are extrictly needed, the normal argument is that the less includes, the faster compiler times. Later, Juan.
