On Mon, 2026-07-27 at 11:18 +0530, Amit Machhiwal wrote:
> > self.require_accelerator("tcg")
> > self.set_machine('powernv')
> > + self.require_netdev('user')
>
> One minor nit: require_netdev('user') is placed after set_machine().
> Though it's not necessary, I would prefer to group all require_*
> guards
> together at the top before any setup, so it's immediately clear what
> preconditions the test has. Either way:
>
> Reviewed-by: Amit Machhiwal <[email protected]>
>
> ~Amit
Sure Amit, I'll send v3 with the suggested change, and Keep your review
tag.
Thanks
~Shivang.
