Author: trasz
Date: Tue Sep 15 18:01:59 2015
New Revision: 287827
URL: https://svnweb.freebsd.org/changeset/base/287827

Log:
  Add a way to specify stripesize and stripeoffset to gnop(8). This makes
  it possible to "simulate" 4K media, to eg test alignment handling.
  
  Reviewed by:  mav@
  MFC after:    1 month
  Sponsored by: The FreeBSD Foundation
  Differential Revision:        https://reviews.freebsd.org/D3664

Modified:
  head/sbin/geom/class/nop/geom_nop.c
  head/sbin/geom/class/nop/gnop.8
  head/sys/geom/nop/g_nop.c
  head/sys/geom/nop/g_nop.h

Modified: head/sbin/geom/class/nop/geom_nop.c
==============================================================================
--- head/sbin/geom/class/nop/geom_nop.c Tue Sep 15 17:16:31 2015        
(r287826)
+++ head/sbin/geom/class/nop/geom_nop.c Tue Sep 15 18:01:59 2015        
(r287827)
@@ -43,14 +43,16 @@ struct g_command class_commands[] = {
            {
                { 'e', "error", "-1", G_TYPE_NUMBER },
                { 'o', "offset", "0", G_TYPE_NUMBER },
+               { 'p', "stripesize", "0", G_TYPE_NUMBER },
+               { 'P', "stripeoffset", "0", G_TYPE_NUMBER },
                { 'r', "rfailprob", "-1", G_TYPE_NUMBER },
                { 's', "size", "0", G_TYPE_NUMBER },
                { 'S', "secsize", "0", G_TYPE_NUMBER },
                { 'w', "wfailprob", "-1", G_TYPE_NUMBER },
                G_OPT_SENTINEL
            },
-           "[-v] [-e error] [-o offset] [-r rfailprob] [-s size] "
-           "[-S secsize] [-w wfailprob] dev ..."
+           "[-v] [-e error] [-o offset] [-p stripesize] [-P stripeoffset] "
+           "[-r rfailprob] [-s size] [-S secsize] [-w wfailprob] dev ..."
        },
        { "configure", G_FLAG_VERBOSE, NULL,
            {

Modified: head/sbin/geom/class/nop/gnop.8
==============================================================================
--- head/sbin/geom/class/nop/gnop.8     Tue Sep 15 17:16:31 2015        
(r287826)
+++ head/sbin/geom/class/nop/gnop.8     Tue Sep 15 18:01:59 2015        
(r287827)
@@ -24,7 +24,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd April 14, 2013
+.Dd September 15, 2015
 .Dt GNOP 8
 .Os
 .Sh NAME
@@ -36,6 +36,8 @@
 .Op Fl v
 .Op Fl e Ar error
 .Op Fl o Ar offset
+.Op Fl p Ar stripesize
+.Op Fl P Ar stripeoffset
 .Op Fl r Ar rfailprob
 .Op Fl s Ar size
 .Op Fl S Ar secsize
@@ -115,6 +117,10 @@ Specifies the error number to return on 
 Force the removal of the specified provider.
 .It Fl o Ar offset
 Where to begin on the original provider.
+.It Fl p Ar stripesize
+Value of the stripesize property of the transparent provider.
+.It Fl P Ar stripeoffset
+Value of the stripeoffset property of the transparent provider.
 .It Fl r Ar rfailprob
 Specifies read failure probability in percent.
 .It Fl s Ar size

Modified: head/sys/geom/nop/g_nop.c
==============================================================================
--- head/sys/geom/nop/g_nop.c   Tue Sep 15 17:16:31 2015        (r287826)
+++ head/sys/geom/nop/g_nop.c   Tue Sep 15 18:01:59 2015        (r287827)
@@ -162,7 +162,7 @@ g_nop_access(struct g_provider *pp, int 
 static int
 g_nop_create(struct gctl_req *req, struct g_class *mp, struct g_provider *pp,
     int ioerror, u_int rfailprob, u_int wfailprob, off_t offset, off_t size,
-    u_int secsize)
+    u_int secsize, u_int stripesize, u_int stripeoffset)
 {
        struct g_nop_softc *sc;
        struct g_geom *gp;
@@ -208,6 +208,18 @@ g_nop_create(struct gctl_req *req, struc
                return (EINVAL);
        }
        size -= size % secsize;
+       if ((stripesize % pp->sectorsize) != 0) {
+               gctl_error(req, "Invalid stripesize for provider %s.", 
pp->name);
+               return (EINVAL);
+       }
+       if ((stripeoffset % pp->sectorsize) != 0) {
+               gctl_error(req, "Invalid stripeoffset for provider %s.", 
pp->name);
+               return (EINVAL);
+       }
+       if (stripesize != 0 && stripeoffset >= stripesize) {
+               gctl_error(req, "stripeoffset is too big.");
+               return (EINVAL);
+       }
        snprintf(name, sizeof(name), "%s%s", pp->name, G_NOP_SUFFIX);
        LIST_FOREACH(gp, &mp->geom, geom) {
                if (strcmp(gp->name, name) == 0) {
@@ -219,6 +231,8 @@ g_nop_create(struct gctl_req *req, struc
        sc = g_malloc(sizeof(*sc), M_WAITOK | M_ZERO);
        sc->sc_offset = offset;
        sc->sc_explicitsize = explicitsize;
+       sc->sc_stripesize = stripesize;
+       sc->sc_stripeoffset = stripeoffset;
        sc->sc_error = ioerror;
        sc->sc_rfailprob = rfailprob;
        sc->sc_wfailprob = wfailprob;
@@ -238,6 +252,8 @@ g_nop_create(struct gctl_req *req, struc
        newpp->flags |= G_PF_DIRECT_SEND | G_PF_DIRECT_RECEIVE;
        newpp->mediasize = size;
        newpp->sectorsize = secsize;
+       newpp->stripesize = stripesize;
+       newpp->stripeoffset = stripeoffset;
 
        cp = g_new_consumer(gp);
        cp->flags |= G_CF_DIRECT_SEND | G_CF_DIRECT_RECEIVE;
@@ -304,7 +320,8 @@ static void
 g_nop_ctl_create(struct gctl_req *req, struct g_class *mp)
 {
        struct g_provider *pp;
-       intmax_t *error, *rfailprob, *wfailprob, *offset, *secsize, *size;
+       intmax_t *error, *rfailprob, *wfailprob, *offset, *secsize, *size,
+           *stripesize, *stripeoffset;
        const char *name;
        char param[16];
        int i, *nargs;
@@ -370,6 +387,24 @@ g_nop_ctl_create(struct gctl_req *req, s
                gctl_error(req, "Invalid '%s' argument", "secsize");
                return;
        }
+       stripesize = gctl_get_paraml(req, "stripesize", sizeof(*stripesize));
+       if (stripesize == NULL) {
+               gctl_error(req, "No '%s' argument", "stripesize");
+               return;
+       }
+       if (*stripesize < 0) {
+               gctl_error(req, "Invalid '%s' argument", "stripesize");
+               return;
+       }
+       stripeoffset = gctl_get_paraml(req, "stripeoffset", 
sizeof(*stripeoffset));
+       if (stripeoffset == NULL) {
+               gctl_error(req, "No '%s' argument", "stripeoffset");
+               return;
+       }
+       if (*stripeoffset < 0) {
+               gctl_error(req, "Invalid '%s' argument", "stripeoffset");
+               return;
+       }
 
        for (i = 0; i < *nargs; i++) {
                snprintf(param, sizeof(param), "arg%d", i);
@@ -390,7 +425,8 @@ g_nop_ctl_create(struct gctl_req *req, s
                    *error == -1 ? EIO : (int)*error,
                    *rfailprob == -1 ? 0 : (u_int)*rfailprob,
                    *wfailprob == -1 ? 0 : (u_int)*wfailprob,
-                   (off_t)*offset, (off_t)*size, (u_int)*secsize) != 0) {
+                   (off_t)*offset, (off_t)*size, (u_int)*secsize,
+                   (u_int)*stripesize, (u_int)*stripeoffset) != 0) {
                        return;
                }
        }

Modified: head/sys/geom/nop/g_nop.h
==============================================================================
--- head/sys/geom/nop/g_nop.h   Tue Sep 15 17:16:31 2015        (r287826)
+++ head/sys/geom/nop/g_nop.h   Tue Sep 15 18:01:59 2015        (r287827)
@@ -59,6 +59,8 @@ struct g_nop_softc {
        int             sc_error;
        off_t           sc_offset;
        off_t           sc_explicitsize;
+       off_t           sc_stripesize;
+       off_t           sc_stripeoffset;
        u_int           sc_rfailprob;
        u_int           sc_wfailprob;
        uintmax_t       sc_reads;
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"

Reply via email to