On 4/12/21 5:50 AM, Jan Beulich wrote: > On 09.04.2021 23:45, Boris Ostrovsky wrote: >> On 4/7/21 10:37 AM, Jan Beulich wrote: >>> There's no point in comparing SBDF - we can simply compare the struct >>> pci_dev pointers. If they weren't the same for a given device, we'd have >>> bigger problems from having stored a stale pointer. >>> >>> Signed-off-by: Jan Beulich <jbeul...@suse.com> >> >> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> > Thanks. As the 1st patch of this series still looks to have an unclear > disposition (unless not getting back a reply on my responses to your > comments means silent agreement),
TBH I have been struggling with understanding both the original problem (just like you have) and the solution you are proposing (apart from making dev_list[] more compact). > I can't predict yet when I'd be able > to submit v2. Hence I'd like to point out that this patch is > independent of the former two, and hence would need to wait further if > you wanted to apply it. After all this one (unlike the other two) is > merely cleanup, and hence would rather want to go in during a merge > window. Given that next Sunday may be when 5.12 is released I think everything but stoppers will have to wait for the merge window. -boris
