[Stgt-devel] [Patch 1/2] Add dynamic lu configuration.

FUJITA Tomonori fujita.tomonori
Fri Jun 15 00:46:22 CEST 2007


From: "Mark Harvey" <markh794 at gmail.com>
Subject: Re: [Stgt-devel] [Patch 1/2] Add dynamic lu configuration.
Date: Fri, 15 Jun 2007 08:32:00 +1000

> On 6/15/07, FUJITA Tomonori <fujita.tomonori at lab.ntt.co.jp> wrote:
> > >
> > > Signed-off-by: Mark Harvey <markh794 at gmail.com>
> >
> > > @@ -286,18 +286,29 @@ int spc_illegal_op(int host_no, struct scsi_cmd *cmd)
> > >  }
> > >
> > >  enum {
> > > -     Opt_scsiid, Opt_scsisn, Opt_err,
> > > +     Opt_scsiid, Opt_scsisn,
> > > +     Opt_vendorid, Opt_productid,
> > > +     Opt_productrev, Opt_sense_format,
> > > +     Opt_removable, Opt_online,
> > > +     Opt_err,
> > >  };
> > >
> > >  static match_table_t tokens = {
> > >       {Opt_scsiid, "scsi_id=%s"},
> > >       {Opt_scsisn, "scsi_sn=%s"},
> > > +     {Opt_vendorid, "vendor_id=%s"},
> > > +     {Opt_productid, "product_id=%s"},
> > > +     {Opt_productrev, "product_rev=%s"},
> > > +     {Opt_sense_format, "sense_format=%s"},
> > > +     {Opt_removable, "removable=%s"},
> > > +     {Opt_online, "online=%s"},
> > >       {Opt_err, NULL},
> > >  };
> >
> > I want name consistency here though I'm not arguing that Opt_scsiid
> > and scsi_id / Opt_scsisn and scsi_sn naming is the best choice.
> >
> > Previously, you used:
> >
> > > +     {Opt_VendorIdent, "VendorIdent=%s"},
> > > +     {Opt_ProductIdent, "ProductIdent=%s"},
> > > +     {Opt_ProductRev, "ProductRev=%s"},
> > > +     {Opt_SenseFormat, "SenseFormat=%s"},
> > > +     {Opt_Removable, "Removable=%s"},
> > > +     {Opt_Online, "Online=%s"},
> >
> > I'm ok with changing Opt_scsiid and scsi_id / Opt_scsisn and scsi_sn
> > naming.
> >
> > Suggestions?
> >
> 
> Sorry, I took the 'consistency' to reflect the Upper/Lower-case. I
> missed the scsisn/scsi_sn all together.

I meant to it and Upper/Lower-case.

You prefer Upper/Lower-case? I meant you want something like SCSI_ID=%s?

This is visible to users, we shouldn't change this often.



More information about the stgt mailing list