Re: [PATCH V7 02/29] migration: fix populate_vfio_info

2022-03-03 Thread Steven Sistare
On 3/3/2022 11:21 AM, Peter Maydell wrote: > On Thu, 3 Mar 2022 at 15:55, Steven Sistare wrote: >> >> On 2/24/2022 1:42 PM, Peter Maydell wrote: >>> ...it also seems to be making a no-change-of-behaviour rewrite >>> of the rest of the file. Is there a reason I'm missing for doing >>> that ? > >>

Re: [PATCH V7 02/29] migration: fix populate_vfio_info

2022-03-03 Thread Peter Maydell
On Thu, 3 Mar 2022 at 15:55, Steven Sistare wrote: > > On 2/24/2022 1:42 PM, Peter Maydell wrote: > > ...it also seems to be making a no-change-of-behaviour rewrite > > of the rest of the file. Is there a reason I'm missing for doing > > that ? > I'll change the commit message to explain: > >

Re: [PATCH V7 02/29] migration: fix populate_vfio_info

2022-03-03 Thread Steven Sistare
On 2/24/2022 1:42 PM, Peter Maydell wrote: > On Wed, 22 Dec 2021 at 19:45, Steve Sistare wrote: >> >> Include CONFIG_DEVICES so that populate_vfio_info is instantiated for >> CONFIG_VFIO. > > The commit message says "include CONFIG_DEVICES"... > >> Signed-off-by: Steve Sistare >> --- >> migrat

Re: [PATCH V7 02/29] migration: fix populate_vfio_info

2022-02-24 Thread Peter Maydell
On Wed, 22 Dec 2021 at 19:45, Steve Sistare wrote: > > Include CONFIG_DEVICES so that populate_vfio_info is instantiated for > CONFIG_VFIO. The commit message says "include CONFIG_DEVICES"... > Signed-off-by: Steve Sistare > --- > migration/target.c | 10 +++--- > 1 file changed, 7 inserti

[PATCH V7 02/29] migration: fix populate_vfio_info

2021-12-22 Thread Steve Sistare
Include CONFIG_DEVICES so that populate_vfio_info is instantiated for CONFIG_VFIO. Signed-off-by: Steve Sistare --- migration/target.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/migration/target.c b/migration/target.c index 907ebf0..4390bf0 100644 --- a/migrat