Re: [PATCH] bnx2i: Make boot_nic entry visible in the sysfs session objects

2014-09-08 Thread Maurizio Lombardi
Hi Cristoph, Jens,

On 05/19/2014 01:32 PM, vikas.chaudh...@qlogic.com wrote:
> From: Tej Parkash 
> 
> Signed-off-by: Tej Parkash 
> Signed-off-by: Vikas Chaudhary 
> ---
>  drivers/scsi/bnx2i/bnx2i_iscsi.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c 
> b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> index 166543f..cabc8c1 100644
> --- a/drivers/scsi/bnx2i/bnx2i_iscsi.c
> +++ b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> @@ -2234,6 +2234,9 @@ static umode_t bnx2i_attr_is_visible(int param_type, 
> int param)
>   case ISCSI_PARAM_TGT_RESET_TMO:
>   case ISCSI_PARAM_IFACE_NAME:
>   case ISCSI_PARAM_INITIATOR_NAME:
> + case ISCSI_PARAM_BOOT_ROOT:
> + case ISCSI_PARAM_BOOT_NIC:
> + case ISCSI_PARAM_BOOT_TARGET:
>   return S_IRUGO;
>   default:
>   return 0;
> 

Can you merge this patch? It has been ACKed already.

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


Re: [PATCH] bnx2i: Make boot_nic entry visible in the sysfs session objects

2014-05-22 Thread Mike Christie
On 05/19/2014 06:32 AM, vikas.chaudh...@qlogic.com wrote:
> From: Tej Parkash 
> 
> Signed-off-by: Tej Parkash 
> Signed-off-by: Vikas Chaudhary 
> ---
>  drivers/scsi/bnx2i/bnx2i_iscsi.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c 
> b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> index 166543f..cabc8c1 100644
> --- a/drivers/scsi/bnx2i/bnx2i_iscsi.c
> +++ b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> @@ -2234,6 +2234,9 @@ static umode_t bnx2i_attr_is_visible(int param_type, 
> int param)
>   case ISCSI_PARAM_TGT_RESET_TMO:
>   case ISCSI_PARAM_IFACE_NAME:
>   case ISCSI_PARAM_INITIATOR_NAME:
> + case ISCSI_PARAM_BOOT_ROOT:
> + case ISCSI_PARAM_BOOT_NIC:
> + case ISCSI_PARAM_BOOT_TARGET:
>   return S_IRUGO;
>   default:
>   return 0;

I think we also wanted this for the chelsio driver, but I need to set it
up to test. So for now, looks ok to me. If chelsio needs it, I can send
a patch for that when done testing.


Reviewed-by: Mike Christie 

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


Re: [PATCH] bnx2i: Make boot_nic entry visible in the sysfs session objects

2014-05-19 Thread Eddie Wai
FYI, this was part of the boot_nic sysfs patch which would allow iscsid
to sync_session with info from the iBFT for iSCSI BFS via offload.

This looks good to me.

Acked-by: Eddie Wai 

On Mon, 2014-05-19 at 07:32 -0400, vikas.chaudh...@qlogic.com wrote:
> From: Tej Parkash 
> 
> Signed-off-by: Tej Parkash 
> Signed-off-by: Vikas Chaudhary 
> ---
>  drivers/scsi/bnx2i/bnx2i_iscsi.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c 
> b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> index 166543f..cabc8c1 100644
> --- a/drivers/scsi/bnx2i/bnx2i_iscsi.c
> +++ b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> @@ -2234,6 +2234,9 @@ static umode_t bnx2i_attr_is_visible(int param_type, 
> int param)
>   case ISCSI_PARAM_TGT_RESET_TMO:
>   case ISCSI_PARAM_IFACE_NAME:
>   case ISCSI_PARAM_INITIATOR_NAME:
> + case ISCSI_PARAM_BOOT_ROOT:
> + case ISCSI_PARAM_BOOT_NIC:
> + case ISCSI_PARAM_BOOT_TARGET:
>   return S_IRUGO;
>   default:
>   return 0;


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


[PATCH] bnx2i: Make boot_nic entry visible in the sysfs session objects

2014-05-19 Thread vikas.chaudhary
From: Tej Parkash 

Signed-off-by: Tej Parkash 
Signed-off-by: Vikas Chaudhary 
---
 drivers/scsi/bnx2i/bnx2i_iscsi.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c b/drivers/scsi/bnx2i/bnx2i_iscsi.c
index 166543f..cabc8c1 100644
--- a/drivers/scsi/bnx2i/bnx2i_iscsi.c
+++ b/drivers/scsi/bnx2i/bnx2i_iscsi.c
@@ -2234,6 +2234,9 @@ static umode_t bnx2i_attr_is_visible(int param_type, int 
param)
case ISCSI_PARAM_TGT_RESET_TMO:
case ISCSI_PARAM_IFACE_NAME:
case ISCSI_PARAM_INITIATOR_NAME:
+   case ISCSI_PARAM_BOOT_ROOT:
+   case ISCSI_PARAM_BOOT_NIC:
+   case ISCSI_PARAM_BOOT_TARGET:
return S_IRUGO;
default:
return 0;
-- 
1.8.2.GIT

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