Re: [Xen-devel] [PATCH] vmevent: reduce include dependencies

2020-03-09 Thread Tamas K Lengyel
On Mon, Mar 9, 2020 at 5:51 AM Jan Beulich wrote: > > There's no need for virtually everything to include public/vm_event.h. > Move its inclusion out of sched.h. This requires using the non-typedef > name in p2m_mem_paging_resume()'s prototype; by not changing the > function definition at the same

Re: [Xen-devel] [PATCH] vmevent: reduce include dependencies

2020-03-09 Thread Isaila Alexandru
On 09.03.2020 13:51, Jan Beulich wrote: There's no need for virtually everything to include public/vm_event.h. Move its inclusion out of sched.h. This requires using the non-typedef name in p2m_mem_paging_resume()'s prototype; by not changing the function definition at the same time it'll remai

Re: [Xen-devel] [PATCH] vmevent: reduce include dependencies

2020-03-09 Thread Ross Lagerwall
On 3/9/20 11:51 AM, Jan Beulich wrote: > There's no need for virtually everything to include public/vm_event.h. > Move its inclusion out of sched.h. This requires using the non-typedef > name in p2m_mem_paging_resume()'s prototype; by not changing the > function definition at the same time it'll re

[Xen-devel] [PATCH] vmevent: reduce include dependencies

2020-03-09 Thread Jan Beulich
There's no need for virtually everything to include public/vm_event.h. Move its inclusion out of sched.h. This requires using the non-typedef name in p2m_mem_paging_resume()'s prototype; by not changing the function definition at the same time it'll remain certain that the build would fail if the t