On Wed, Jul 24, 2013 at 7:38 PM, Jeffrey Bastian <jbast...@redhat.com> wrote:
> 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
>
We had this conversation 5 months ago and as far as I'm concerned,
nothing has changed.
Z.
------------------------------------------------------------------------------
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
