Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 1 Feb 2018 08:35:32 -0700
From:      Warner Losh <imp@bsdimp.com>
To:        Alexander Motin <mav@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org,  svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   Re: svn commit: r328666 - stable/11/sys/dev/nvme
Message-ID:  <CANCZdfopZCKc-tuZVbjfrm_a2EedVXxYhbc1dRZbimcCDVFsRA@mail.gmail.com>
In-Reply-To: <201802011532.w11FWmIa088204@repo.freebsd.org>
References:  <201802011532.w11FWmIa088204@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
I'd been specifically not MFC'ing the NVME stuff because I wasn't sure that
its ABI had jelled yet....

Be sure to merge everything because there's 0 support for anything but the
tip of -current for the NVME CAM stuff.

Warner

On Thu, Feb 1, 2018 at 8:32 AM, Alexander Motin <mav@freebsd.org> wrote:

> Author: mav
> Date: Thu Feb  1 15:32:48 2018
> New Revision: 328666
> URL: https://svnweb.freebsd.org/changeset/base/328666
>
> Log:
>   MFC r303466: Kill a few stray debug printfs.
>
> Modified:
>   stable/11/sys/dev/nvme/nvme_sim.c
> Directory Properties:
>   stable/11/   (props changed)
>
> Modified: stable/11/sys/dev/nvme/nvme_sim.c
> ============================================================
> ==================
> --- stable/11/sys/dev/nvme/nvme_sim.c   Thu Feb  1 15:32:14 2018
> (r328665)
> +++ stable/11/sys/dev/nvme/nvme_sim.c   Thu Feb  1 15:32:48 2018
> (r328666)
> @@ -133,8 +133,6 @@ nvme_sim_action(struct cam_sim *sim, union ccb *ccb)
>         ns = sim2ns(sim);
>         ctrlr = sim2ctrlr(sim);
>
> -       printf("Sim action: ctrlr %p ns %p\n", ctrlr, ns);
> -
>         mtx_assert(&ctrlr->lock, MA_OWNED);
>
>         switch (ccb->ccb_h.func_code) {
> @@ -310,8 +308,6 @@ nvme_sim_new_ns(struct nvme_namespace *ns, void *sc_ar
>
>         sc->s_ns = ns;
>
> -       printf("Our SIM's softc %p ctrlr %p ns %p\n", sc, ctrlr, ns);
> -
>         /*
>          * XXX this is creating one bus per ns, but it should be one
>          * XXX target per controller, and one LUN per namespace.
> @@ -349,7 +345,6 @@ nvme_sim_new_ns(struct nvme_namespace *ns, void *sc_ar
>         sc->s_path->device->nvme_cdata = nvme_ctrlr_get_data(ns->ctrlr);
>
>  /* Scan bus */
> -       printf("Initiate rescan of the bus\n");
>         nvme_sim_rescan_target(ctrlr, sc->s_path);
>
>         mtx_unlock(&ctrlr->lock);
>
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfopZCKc-tuZVbjfrm_a2EedVXxYhbc1dRZbimcCDVFsRA>