Thanks for the review. Comments below.
The 09/04/2020 08:02, Jakub Kicinski wrote:
On Fri, 4 Sep 2020 09:15:24 +0000 Henrik Bjoernlund wrote:
quoted
+ rcu_read_lock();
+ b_port = rcu_dereference(mep->b_port);
+ if (!b_port)
+ return NULL;
+ skb = dev_alloc_skb(CFM_CCM_MAX_FRAME_LENGTH);
+ if (!skb)
+ return NULL;
net/bridge/br_cfm.c:171:23: warning: context imbalance in 'ccm_frame_build' - different lock contexts for basic block
I will assure that rcu_read_unlock() is called before any return.
--
/Henrik