Re: [PATCH 4/4] libceph: a few small changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/02/2012 01:40 PM, Sage Weil wrote:
+	default:
>  +		WARN(1, "unexpected socket state (%d)", sk->sk_state);
>  +		break;
>    	}
This is a red herring... the switch isn't meant to be exhaustive, only to
catch the interesting (shutdown) states.  I think #2099 can be closed as
well.

That's pretty funny.  Didn't I open that bug?

I will drop the WARN() but will add a comment to indicate it's
explicitly not meant to cover all expected cases.

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


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux