2022-08-08 17:33 (UTC-0700), Kadam, Pallavi: [...] > > Hi Pallavi, > > > > In the first place, it was wrong to always test `prev_bar & PCI_TYPE_64BIT` > > because only the first BAR slot of a 64-bit BAR contains flags. > > The current code has a state to track (curr_bar, prev_bar), > > and the fix is complicating it even more without solving the root cause. > > I suggest a simpler fix (not tested!) > > that eliminates both the incorrectness and the state to maintain: > > Thank you. This change works for us. > > Please let me know if you would like to submit this change as a new patch or > if I should include it as a v2 of this same patch.
Please send v2. You can include my Signed-off-by. You might also update the commit message and capitalize "BAR" in the title.