staging/rdma/hfi1: Unexpected link up pkey values are not an error
Only warn when link up pkeys are not what we expect. Also, allow for the pkey to already be initialized. Reviewed-by: Arthur Kepner <arthur.kepner@intel.com> Signed-off-by: Dean Luick <dean.luick@intel.com> Signed-off-by: Jubin John <jubin.john@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
		
							parent
							
								
									e8597eb014
								
							
						
					
					
						commit
						8764522e52
					
				| @ -6716,10 +6716,10 @@ static void add_full_mgmt_pkey(struct hfi1_pportdata *ppd) | |||||||
| { | { | ||||||
| 	struct hfi1_devdata *dd = ppd->dd; | 	struct hfi1_devdata *dd = ppd->dd; | ||||||
| 
 | 
 | ||||||
| 	/* Sanity check - ppd->pkeys[2] should be 0 */ | 	/* Sanity check - ppd->pkeys[2] should be 0, or already initalized */ | ||||||
| 	if (ppd->pkeys[2] != 0) | 	if (!((ppd->pkeys[2] == 0) || (ppd->pkeys[2] == FULL_MGMT_P_KEY))) | ||||||
| 		dd_dev_err(dd, "%s pkey[2] already set to 0x%x, resetting it to 0x%x\n", | 		dd_dev_warn(dd, "%s pkey[2] already set to 0x%x, resetting it to 0x%x\n", | ||||||
| 			   __func__, ppd->pkeys[2], FULL_MGMT_P_KEY); | 			    __func__, ppd->pkeys[2], FULL_MGMT_P_KEY); | ||||||
| 	ppd->pkeys[2] = FULL_MGMT_P_KEY; | 	ppd->pkeys[2] = FULL_MGMT_P_KEY; | ||||||
| 	(void)hfi1_set_ib_cfg(ppd, HFI1_IB_CFG_PKEYS, 0); | 	(void)hfi1_set_ib_cfg(ppd, HFI1_IB_CFG_PKEYS, 0); | ||||||
| } | } | ||||||
|  | |||||||
		Loading…
	
		Reference in New Issue
	
	Block a user