I will look into this. Thanks, -Steve

>-----Original Message-----
>From: Love, Robert W
>Sent: Wednesday, May 06, 2009 4:09 PM
>To: Ma, Steve
>Cc: [email protected]
>Subject: Re: [Open-FCoE] [PATCH] fcoe-utils: fcoeadm -i displays HBA header
>for every port.
>
>On Tue, 2009-04-28 at 13:54 -0700, Steve Ma wrote:
>> The "fcoeadm -i" should only display HBA header per HBA,
>> not per-port.
>> ---
>>
>>  fcoeadm_display.c |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>This patch does not apply to the current tree. I tried to fix this one
>since it's a simple change, but the line being removed isn't in the code
>base.  So, please update the patch, retest and re-post. Also, please add
>your Signed-off line.
>
>> diff --git a/fcoeadm_display.c b/fcoeadm_display.c
>> index 050563f..795113a 100644
>> --- a/fcoeadm_display.c
>> +++ b/fcoeadm_display.c
>> @@ -1223,7 +1223,8 @@ display_adapter_info(struct opt_info *opt_info)
>>                      if (opt_info->ifname &&
>>                          !strstr(port_attrs.PortSymbolicName, opt_info-
>>ifname))
>>                              continue;
>> -                    show_hba_info(hba_index, &hba_attrs, 0);
>> +                    if (rc)
>> +                            show_hba_info(hba_index, &hba_attrs, 0);
>>                      show_port_info(hba_index, lp_index, &hba_attrs,
>>                                      &port_attrs);
>>              }
>>
>> _______________________________________________
>> devel mailing list
>> [email protected]
>> http://www.open-fcoe.org/mailman/listinfo/devel

_______________________________________________
devel mailing list
[email protected]
http://www.open-fcoe.org/mailman/listinfo/devel

Reply via email to