Re: setting write cache fails in recent git

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

 



On Sun, 19 Oct 2008 09:49:32 +0200
Doron Shoham <dorons@xxxxxxxxxxxx> wrote:

> >>
> >> Signed-off-by: root <root@xxxxxxxxxxxxxxxxxxxxxxxxx>
> > 
> > Are you sure about this Signed-off-by?
> 
> Of course, my mistake.
> > 
> > 
> >> ---
> >>  usr/tgtadm.c |    4 ++--
> >>  1 files changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/usr/tgtadm.c b/usr/tgtadm.c
> >> index 06f18e2..c9a3d74 100644
> >> --- a/usr/tgtadm.c
> >> +++ b/usr/tgtadm.c
> >> @@ -140,7 +140,7 @@ Linux SCSI Target Framework Administration Utility.\n\
> >>                          enable the target to accept the specific initiators.\n\
> >>    --lld [driver] --mode target --op unbind --tid=[id] --initiator-address=[src]\n\
> >>                          disable the specific permitted initiators.\n\
> >> -  --lld [driver] --mode logicalunit --op new --tid=[id] --lun=[lun] --backing-store=[path]\n\
> >> +  --lld [driver] --mode logicalunit --op new --tid=[id] --lun=[lun] --backing-store=[path] --bstype=[type]\n\
> > 
> > Well, bstype is optional. I think that we need to make it clear here.
> > 
> 
> enable bstype option when creating a new logicalunit
> 
> Signed-off-by: Doron Shoham <dorons@xxxxxxxxxxxx>
> ---
>  usr/tgtadm.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/usr/tgtadm.c b/usr/tgtadm.c
> index 06f18e2..6e1c61b 100644
> --- a/usr/tgtadm.c
> +++ b/usr/tgtadm.c
> @@ -140,11 +140,12 @@ Linux SCSI Target Framework Administration Utility.\n\
>                          enable the target to accept the specific initiators.\n\
>    --lld [driver] --mode target --op unbind --tid=[id] --initiator-address=[src]\n\
>                          disable the specific permitted initiators.\n\
> -  --lld [driver] --mode logicalunit --op new --tid=[id] --lun=[lun] --backing-store=[path]\n\
> +  --lld [driver] --mode logicalunit --op new --tid=[id] --lun=[lun] --backing-store=[path] --bstype=[type]\n\
>                          add a new logical unit with [lun] to the specific\n\
>                          target with [id]. The logical unit is offered\n\
>                          to the initiators. [path] must be block device files\n\
>                          (including LVM and RAID devices) or regular files.\n\
> +                        bstype is optional (default is 'rdwr').\n\

default is 'rdwr' is not correct. Check out default_bst in struct
tgt_driver. But we need a new mechanism to select an appropriate
bstype and check it, as someone pointed out.

Anyway, I just removed this description and applied the patch.

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

[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux