> -----Original Message----- > From: David Miller [mailto:da...@davemloft.net] > Sent: Saturday, November 14, 2015 12:40 AM > To: f.faine...@gmail.com > Cc: Premkumar Jonnala; netdev@vger.kernel.org; sfel...@gmail.com; > j...@resnulli.us; niko...@cumulusnetworks.com; ido...@mellanox.com; > go...@cumulusnetworks.com > Subject: Re: [PATCH] bonding: Offloading bonds to hardware > > From: Florian Fainelli <f.faine...@gmail.com> > Date: Fri, 13 Nov 2015 10:38:48 -0800 > > > This is a good explanation of why you want the changes, and how this is > > implemented in a system utilizing that, but this is not documenting why > > you are making these changes to the bonding code, nor how they are > > supposed to be used by an implementor driver, since there is no such > > user posted (yet?). > > I am basically not even going to look at proposals for new device ops > that don't also show an actual user of the new interfaces. > > That's not how we do development, and not providing a real example > user of a new set of interfaces makes it impossible to properly review > such changes.
Thank you for the feedback David. I am currently handling/implementing these notifications in a custom (closed source) driver. I will update the diff with an actual demo of the handler, and resend the diffs. Thanks Prem -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
