Re: [PATCH 4/6] bnx2fc: Improve error recovery by handling parity errors

["Bhanu Prakash Gollapudi" <bprakash@xxxxxxxxxxxx>]

On 6/6/2012 8:03 PM, James Bottomley wrote:
> On Mon, 2012-06-04 at 16:15 -0700, Bhanu Prakash Gollapudi wrote:
> > @@ -321,6 +332,9 @@ static void bnx2fc_upload_session(struct fcoe_port *port,
> >
> >   		del_timer_sync(&tgt->upld_timer);
> >
> > +	} else if (test_bit(BNX2FC_FLAG_DISABLE_FAILED,&tgt->flags))
> > +		printk(KERN_ERR PFX "ERROR!! DISABLE req failed, destroy"
> > +				" not sent to FW\n");
> >   	} else
> >   		printk(KERN_ERR PFX "ERROR!! DISABLE req timed out, destroy"
> >   				" not sent to FW\n");
>
> You didn't actually compile this did you?  That else if has to have an
> opening brace to match the closing one on the next else ...

My bad. I fixed this compilation error, but by mistake I applied that 
change to next patch. I'll correct it and submit the last three patches 
shortly, as you've already picked up the first three.

Thanks,
Bhanu

> James
>
> ��칻�&�~�&���+-��ݶ��w��˛���m�b��lrȢ��^n�r���z���h����&���G���h�(�階�ݢj"���m�����z�ޖ���f���h���~�m�


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html