[PATCH] net/ncsi: Add NCSI OEM command for FB Tiogapass
vijaykhemka at fb.com
Thu Sep 27 03:07:43 AEST 2018
> Hi Vijay,
> Thanks for the patch; before I get too into a review though I'd like to
> loop in Justin (cc'd) who I know is also working on an OEM command patch.
> The changes here are very specific (eg. a command specific config option
> "CONFIG_NCSI_OEM_CMD_GET_MAC"), which is ok on a small scale but if we
> start to add an increasing amount of commands could get out of hand.
> As I understand Justin's version adds a generic handler, using the NCSI
> Netlink interface to pass OEM commands and responses to and from
> userspace, which does the actual packet handling.
> It would be good to compare these two approaches first before committing
> to any one path
My oem command handler is generic and can be used for any oem commands and oem response handler can be made more generic. We can certainly write a wrapper to support netlink oem command to receive form user space. There are Mellanox specific functions which sends Mellanox specific request. These needed as a part of initial configuration. We can remove Kconfig option with more generic approach.
More information about the openbmc