Thread (16 messages) flat view 16 messages, 2 authors, 2021-01-11

Re: [PATCH 2/7] ibmvnic: update reset function prototypes

From: Sukadev Bhattiprolu <hidden>
Date: 2021-01-11 03:13:10

Jakub Kicinski [kuba@kernel.org] wrote:
On Thu,  7 Jan 2021 23:12:31 -0800 Sukadev Bhattiprolu wrote:
quoted
The reset functions need just the 'reset reason' parameter and not
the ibmvnic_rwi list element. Update the functions so we can simplify
the handling of the ->rwi_list in a follow-on patch.

Fixes: 2770a7984db5 ("ibmvnic: Introduce hard reset recovery")
No empty lines after Fixes tags, please. They should also not be
wrapped.
Ah ok, will fix.
quoted
Signed-off-by: Sukadev Bhattiprolu <redacted>
Are these patches for net or net-next? It looks like they are fixing
races, but at the same time they are rather large. Can you please
produce minimal fixes, e.g. patch 3 should just fix the existing leaks
rather than refactor the code to not do allocations. 130+ LoC is a lot
for a fix.
This is a set of bug fixes, but yes a bit large. Should I submit to
net-next instead?

Thanks,

Sukadev
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help