From owner-svn-src-all@freebsd.org Sat Jul 13 13:08:22 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id B8F0915E8EFA; Sat, 13 Jul 2019 13:08:22 +0000 (UTC) (envelope-from gljennjohn@gmail.com) Received: from mail-ed1-x536.google.com (mail-ed1-x536.google.com [IPv6:2a00:1450:4864:20::536]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 3013885DD5; Sat, 13 Jul 2019 13:08:22 +0000 (UTC) (envelope-from gljennjohn@gmail.com) Received: by mail-ed1-x536.google.com with SMTP id v15so11503543eds.9; Sat, 13 Jul 2019 06:08:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:in-reply-to:references:reply-to :mime-version:content-transfer-encoding; bh=7GyKwOp/lmGL6X2F1ftFXVgc11O4S5xTHnsCGV4biDA=; b=WXVl2RosCb2Vfup91Jad4jMRZXgoS33XfoW4cG+gIfZYPT3CMt1s/XEP06VVmdKX43 IWHyopiN++Xh7HfEpWODugstvV8nXjepHRU5c0/1PWz9H+0I6qft1y7LQ6WdmiF1KjK4 6vEcrAOUE10o6R/iWpuhffx+t3BHyEMGWE6x0EkT0G2olrfmqXCyAncRjLXbrET4njq6 9rw/yDNEfDPVapdMmzglk71ZoSZhf022DKS5nFne8LDDXBdKHdEcaFsugIgFtI0hA6/K 0ZlPHB5pkhIJ9quEJnKgP/mAhzJi6mq4oPT3P7YnRRlNq9Da9Jh5J8dqAO6tmh5U7ogm wDHg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:reply-to:mime-version:content-transfer-encoding; bh=7GyKwOp/lmGL6X2F1ftFXVgc11O4S5xTHnsCGV4biDA=; b=UiX342pzXC51n3zT/c9Z7mCfjIhDjqR/z/67CY/P1pr+pRjLf5nVRJogrBWsZ1Wzr4 xGKOFc7VFnjeBKKPQILEZNfLrlOIlmtTRd09vleJGA3f1QdznFgqogu7HEIB1dCVQKs7 qgQt5mGdAP3pSEm8C6xsvgxspi+bGt6DEHk2X2INTQZFJLsprJdLQq28SooMHH75QQ5T QNWVKjZ4Ioiouvl6BuDXuGcmKsw64bgGvtRFehChZ+awGwIHV/g+mY8wvNlV9L6xJu4X mvBFyo4uphGmAvoX55y5LXGubOU6nB4eo4kgCd/+BDACOKma1cyY9B8xMmAqyAXZQvUc i61A== X-Gm-Message-State: APjAAAW13p75CRg2X+D41ADL09hn+fuNmo7zgXiRoADlxJ4FSXpPdOkU RxOTK//zmVzevAv9U0Mv0IITJD+W X-Google-Smtp-Source: APXvYqxDPS6AKO9tuCACPiPj08XbYAmW2ogfTwrid/lMmp4GIUtZYehzsg84nw6ntZFP/VGGo/JRHg== X-Received: by 2002:aa7:d30d:: with SMTP id p13mr14496044edq.292.1563023300968; Sat, 13 Jul 2019 06:08:20 -0700 (PDT) Received: from ernst.home (p5B3BE885.dip0.t-ipconnect.de. [91.59.232.133]) by smtp.gmail.com with ESMTPSA id w27sm3628672edw.63.2019.07.13.06.08.19 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Sat, 13 Jul 2019 06:08:20 -0700 (PDT) Date: Sat, 13 Jul 2019 15:08:18 +0200 From: Gary Jennejohn To: Chuck Tuffli Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r349969 - in head: sys/net usr.sbin/bhyve Message-ID: <20190713150818.11ba99f8@ernst.home> In-Reply-To: <201907131248.x6DCmTgG050521@repo.freebsd.org> References: <201907131248.x6DCmTgG050521@repo.freebsd.org> Reply-To: gljennjohn@gmail.com X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; amd64-portbld-freebsd12.0) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 3013885DD5 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.96 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.96)[-0.964,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 13 Jul 2019 13:08:23 -0000 On Sat, 13 Jul 2019 12:48:29 +0000 (UTC) Chuck Tuffli wrote: > Author: chuck > Date: Sat Jul 13 12:48:28 2019 > New Revision: 349969 > URL: https://svnweb.freebsd.org/changeset/base/349969 > > Log: > bhyve: Create EUI64 for NVMe namespaces > > Accept an IEEE Extended Unique Identifier (EUI-64) from the command > line for each NVMe namespace. If one isn't provided, it will create one > based on the CRC16 of: > - the FreeBSD IEEE OUI > - PCI bus, device/slot, function values > - Namespace ID > > Reviewed by: imp, araujo, jhb, rgrimes > Approved by: imp (mentor), jhb (maintainer) > MFC after: 2 weeks > Differential Revision: https://reviews.freebsd.org/D19905 > > Modified: > head/sys/net/ieee_oui.h > head/usr.sbin/bhyve/pci_nvme.c > > Modified: head/sys/net/ieee_oui.h > ============================================================================== > --- head/sys/net/ieee_oui.h Sat Jul 13 12:45:08 2019 (r349968) > +++ head/sys/net/ieee_oui.h Sat Jul 13 12:48:28 2019 (r349969) > @@ -77,4 +77,9 @@ > */ > #define OUI_FREEBSD_GENERATED_MASK 0x10ffff > #define OUI_FREEBSD_GENERATED_LOW OUI_FREEBSD(0x100000) > -#define OUI_FREEBSD_GENERATED_HIGH OUI_FREEBSD(OU_FREEBSD_GENERATED_MASK) > +#define OUI_FREEBSD_GENERATED_HIGH OUI_FREEBSD(OUI_FREEBSD_GENERATED_MASK) > + > +/* Allocate 16 bits for emulated NVMe devices */ > +#define OUI_FREEBSD_NVME_MASK 0x20ffff > +#define OUI_FREEBSD_NVME_LOW OUI_FREEBSD(0x200000) > +#define OUI_FRREBSD_NVME_HIGH OUI_FREEBSD(OUI_FREEBSD_NVME_MASK) TYPO (FRRE instead of FREE) - this works because the macro isn't used anywhere. > > Modified: head/usr.sbin/bhyve/pci_nvme.c > ============================================================================== > --- head/usr.sbin/bhyve/pci_nvme.c Sat Jul 13 12:45:08 2019 (r349968) > +++ head/usr.sbin/bhyve/pci_nvme.c Sat Jul 13 12:48:28 2019 (r349969) > @@ -4,6 +4,9 @@ > * Copyright (c) 2017 Shunsuke Mie > * Copyright (c) 2018 Leon Dang > * > + * Function crc16 Copyright (c) 2017, Fedor Uporov > + * Obtained from function ext2_crc16() in sys/fs/ext2fs/ext2_csum.c > + * > * Redistribution and use in source and binary forms, with or without > * modification, are permitted provided that the following conditions > * are met: > @@ -30,7 +33,7 @@ > * bhyve PCIe-NVMe device emulation. > * > * options: > - * -s ,nvme,devpath,maxq=#,qsz=#,ioslots=#,sectsz=#,ser=A-Z > + * -s ,nvme,devpath,maxq=#,qsz=#,ioslots=#,sectsz=#,ser=A-Z,eui64=# > * > * accepted devpath: > * /dev/blockdev > @@ -42,6 +45,7 @@ > * ioslots = max number of concurrent io requests > * sectsz = sector size (defaults to blockif sector size) > * ser = serial number (20-chars max) > + * eui64 = IEEE Extended Unique Identifier (8 byte value) > * > */ > > @@ -54,6 +58,7 @@ > __FBSDID("$FreeBSD$"); > > #include > +#include > > #include > #include > @@ -164,6 +169,7 @@ struct pci_nvme_blockstore { > uint64_t size; > uint32_t sectsz; > uint32_t sectsz_bits; > + uint64_t eui64; > }; > > struct pci_nvme_ioreq { > @@ -352,23 +358,87 @@ pci_nvme_init_ctrldata(struct pci_nvme_softc *sc) > cd->power_state[0].mp = 10; > } > > -static void > -pci_nvme_init_nsdata(struct pci_nvme_softc *sc) > +/* > + * Calculate the CRC-16 of the given buffer > + * See copyright attribution at top of file > + */ > +static uint16_t > +crc16(uint16_t crc, const void *buffer, unsigned int len) > { > - struct nvme_namespace_data *nd; > + const unsigned char *cp = buffer; > + /* CRC table for the CRC-16. The poly is 0x8005 (x16 + x15 + x2 + 1). */ > + static uint16_t const crc16_table[256] = { > + 0x0000, 0xC0C1, 0xC181, 0x0140, 0xC301, 0x03C0, 0x0280, 0xC241, > + 0xC601, 0x06C0, 0x0780, 0xC741, 0x0500, 0xC5C1, 0xC481, 0x0440, > + 0xCC01, 0x0CC0, 0x0D80, 0xCD41, 0x0F00, 0xCFC1, 0xCE81, 0x0E40, > + 0x0A00, 0xCAC1, 0xCB81, 0x0B40, 0xC901, 0x09C0, 0x0880, 0xC841, > + 0xD801, 0x18C0, 0x1980, 0xD941, 0x1B00, 0xDBC1, 0xDA81, 0x1A40, > + 0x1E00, 0xDEC1, 0xDF81, 0x1F40, 0xDD01, 0x1DC0, 0x1C80, 0xDC41, > + 0x1400, 0xD4C1, 0xD581, 0x1540, 0xD701, 0x17C0, 0x1680, 0xD641, > + 0xD201, 0x12C0, 0x1380, 0xD341, 0x1100, 0xD1C1, 0xD081, 0x1040, > + 0xF001, 0x30C0, 0x3180, 0xF141, 0x3300, 0xF3C1, 0xF281, 0x3240, > + 0x3600, 0xF6C1, 0xF781, 0x3740, 0xF501, 0x35C0, 0x3480, 0xF441, > + 0x3C00, 0xFCC1, 0xFD81, 0x3D40, 0xFF01, 0x3FC0, 0x3E80, 0xFE41, > + 0xFA01, 0x3AC0, 0x3B80, 0xFB41, 0x3900, 0xF9C1, 0xF881, 0x3840, > + 0x2800, 0xE8C1, 0xE981, 0x2940, 0xEB01, 0x2BC0, 0x2A80, 0xEA41, > + 0xEE01, 0x2EC0, 0x2F80, 0xEF41, 0x2D00, 0xEDC1, 0xEC81, 0x2C40, > + 0xE401, 0x24C0, 0x2580, 0xE541, 0x2700, 0xE7C1, 0xE681, 0x2640, > + 0x2200, 0xE2C1, 0xE381, 0x2340, 0xE101, 0x21C0, 0x2080, 0xE041, > + 0xA001, 0x60C0, 0x6180, 0xA141, 0x6300, 0xA3C1, 0xA281, 0x6240, > + 0x6600, 0xA6C1, 0xA781, 0x6740, 0xA501, 0x65C0, 0x6480, 0xA441, > + 0x6C00, 0xACC1, 0xAD81, 0x6D40, 0xAF01, 0x6FC0, 0x6E80, 0xAE41, > + 0xAA01, 0x6AC0, 0x6B80, 0xAB41, 0x6900, 0xA9C1, 0xA881, 0x6840, > + 0x7800, 0xB8C1, 0xB981, 0x7940, 0xBB01, 0x7BC0, 0x7A80, 0xBA41, > + 0xBE01, 0x7EC0, 0x7F80, 0xBF41, 0x7D00, 0xBDC1, 0xBC81, 0x7C40, > + 0xB401, 0x74C0, 0x7580, 0xB541, 0x7700, 0xB7C1, 0xB681, 0x7640, > + 0x7200, 0xB2C1, 0xB381, 0x7340, 0xB101, 0x71C0, 0x7080, 0xB041, > + 0x5000, 0x90C1, 0x9181, 0x5140, 0x9301, 0x53C0, 0x5280, 0x9241, > + 0x9601, 0x56C0, 0x5780, 0x9741, 0x5500, 0x95C1, 0x9481, 0x5440, > + 0x9C01, 0x5CC0, 0x5D80, 0x9D41, 0x5F00, 0x9FC1, 0x9E81, 0x5E40, > + 0x5A00, 0x9AC1, 0x9B81, 0x5B40, 0x9901, 0x59C0, 0x5880, 0x9841, > + 0x8801, 0x48C0, 0x4980, 0x8941, 0x4B00, 0x8BC1, 0x8A81, 0x4A40, > + 0x4E00, 0x8EC1, 0x8F81, 0x4F40, 0x8D01, 0x4DC0, 0x4C80, 0x8C41, > + 0x4400, 0x84C1, 0x8581, 0x4540, 0x8701, 0x47C0, 0x4680, 0x8641, > + 0x8201, 0x42C0, 0x4380, 0x8341, 0x4100, 0x81C1, 0x8081, 0x4040 > + }; > > - nd = &sc->nsdata; > + while (len--) > + crc = (((crc >> 8) & 0xffU) ^ > + crc16_table[(crc ^ *cp++) & 0xffU]) & 0x0000ffffU; > + return crc; > +} > > +static void > +pci_nvme_init_nsdata(struct pci_nvme_softc *sc, > + struct nvme_namespace_data *nd, uint32_t nsid, > + uint64_t eui64) > +{ > + > nd->nsze = sc->nvstore.size / sc->nvstore.sectsz; > nd->ncap = nd->nsze; > nd->nuse = nd->nsze; > > /* Get LBA and backstore information from backing store */ > nd->nlbaf = 0; /* NLBAF is a 0's based value (i.e. 1 LBA Format) */ > + nd->flbas = 0; > + > + /* Create an EUI-64 if user did not provide one */ > + if (eui64 == 0) { > + char *data = NULL; > + > + asprintf(&data, "%s%u%u%u", vmname, sc->nsc_pi->pi_bus, > + sc->nsc_pi->pi_slot, sc->nsc_pi->pi_func); > + > + if (data != NULL) { > + eui64 = OUI_FREEBSD_NVME_LOW | crc16(0, data, strlen(data)); > + free(data); > + } > + eui64 = (eui64 << 16) | (nsid & 0xffff); > + } > + be64enc(nd->eui64, eui64); > + > /* LBA data-sz = 2^lbads */ > nd->lbaf[0] = sc->nvstore.sectsz_bits << NVME_NS_DATA_LBAF_LBADS_SHIFT; > - > - nd->flbas = 0; > } > > static void > @@ -1817,6 +1887,8 @@ pci_nvme_parse_opts(struct pci_nvme_softc *sc, char *o > free(uopt); > return (-1); > } > + } else if (!strcmp("eui64", xopts)) { > + sc->nvstore.eui64 = htobe64(strtoull(config, NULL, 0)); > } else if (optidx == 0) { > snprintf(bident, sizeof(bident), "%d:%d", > sc->nsc_pi->pi_slot, sc->nsc_pi->pi_func); > @@ -1937,7 +2009,7 @@ pci_nvme_init(struct vmctx *ctx, struct pci_devinst *p > > pci_nvme_reset(sc); > pci_nvme_init_ctrldata(sc); > - pci_nvme_init_nsdata(sc); > + pci_nvme_init_nsdata(sc, &sc->nsdata, 1, sc->nvstore.eui64); > pci_nvme_init_logpages(sc); > > pci_lintr_request(pi); > _______________________________________________ > svn-src-all@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-all > To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org" -- Gary Jennejohn