On Wed, Jul 24, 2013 at 10:10:06AM -0400, Ales Ledvinka wrote:
> > From: "Zdenek Styblik" <zdenek.styb...@gmail.com>
> > However, changes to ipmi_lanp.{c,h} aren't as discussed 5 months
> > ago, because they replace one, resp. existing, OEM functionality
> > with another.
> 
> Were you able to locate the OEM code that uses the lan options
> oem codes defined in ipmi_lanp.h ?
> Since I failed to locate any code associated with that. Either
> it's some dropped functionality of obsoleted hardware.
> Or feature that did not made it into the repository. But that's
> speculation. I don't feel like checking whole commit history.


That was my understanding also: the cxoem patch removes some existing
lanp codes, but nothing is using those lanp codes, so no harm done.

The 4 removed lanp codes are:
        IPMI_LANP_DHCP_SERVER_IP=192,
        IPMI_LANP_DHCP_SERVER_MAC=193,
        IPMI_LANP_DHCP_ENABLE=194,
        IPMI_LANP_CHAN_ACCESS_MODE=201,

Currently the only file that mentions those codes are the definitions
themselves in ipmi_lanp.h:

$ find . -type f | xargs egrep -l \
    'IPMI_LANP_(DHCP_SERVER_IP|DHCP_SERVER_MAC|DHCP_ENABLE|CHAN_ACCESS_MODE)'
./include/ipmitool/ipmi_lanp.h


Jeff

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to