On Tue, Nov 5, 2024 at 7:25 PM Paolo Abeni <pabeni@xxxxxxxxxx> wrote: > > On 10/30/24 02:41, Menglong Dong wrote: > > @@ -1820,7 +1822,8 @@ static int __mkroute_input(struct sk_buff *skb, const struct fib_result *res, > > */ > > if (out_dev == in_dev && > > IN_DEV_PROXY_ARP_PVLAN(in_dev) == 0) { > > - err = -EINVAL; > > + /* what do we name this situation? */ > > + reason = SKB_DROP_REASON_ARP_PVLAN_DISABLE; > > I don't have a better suggestion :( > > Please drop the comment and re-iterate the question in the commit > message after a '---' separator, so we can merge the patch unmodified if > nobody suggests a better one. Okay, I'll move this comment into the commit log, which I usually place the change logs in. Thanks! Menglong Dong > > Thanks, > > Paolo >