In D29182#658119, @tobik wrote:Thanks for the review. I'll make the suggested manual changes later.
This bothers me a whole lot. Did you really have to add a swipe on Linux or glibc? I wonder where exactly I have given the impression that my justification to want to import get_current_dir_name is "because Linux".
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Mar 27 2021
Mar 27 2021
Mar 23 2021
Mar 23 2021
Mar 22 2021
Mar 22 2021
This appears to conform to the description of the Linux version:
Mar 15 2021
Mar 15 2021
Most of that info is already available through kenv(1) and/or kenv(2):
[threepio:~] rpokala% kenv | grep smbios hint.smbios.0.mem="0xf05b0" smbios.bios.reldate="11/22/2019" smbios.bios.vendor="American Megatrends Inc." smbios.bios.version="2.1" smbios.chassis.maker="Supermicro" smbios.chassis.serial="0123456789" smbios.chassis.tag="To Be Filled By O.E.M." smbios.chassis.type="Main Server Chassis" smbios.chassis.version="0123456789" smbios.memory.enabled="33554432" smbios.planar.location="To be filled by O.E.M." smbios.planar.maker="Supermicro" smbios.planar.product="X10SDV-TLN4F" smbios.planar.serial="ZM15AS030025" smbios.planar.tag="To be filled by O.E.M." smbios.planar.version="1.02" smbios.socket.enabled="1" smbios.socket.populated="1" smbios.system.family="To be filled by O.E.M." smbios.system.maker="Supermicro" smbios.system.product="Super Server" smbios.system.serial="0123456789" smbios.system.sku="To be filled by O.E.M." smbios.system.uuid="00000000-0000-0000-0000-0cc47a7c0a92" smbios.system.version="0123456789" smbios.version="2.7"
This looks grossly correct to me, but it's been a long time since I've thought about this stuff. IIRC @ambrisko did some stuff with SMBIOS a ways back; Doug?
This function will be used for exposing DMI info as sysctls in the smbios module (in an upcoming review).
Mar 12 2021
Mar 12 2021
Mar 11 2021
Mar 11 2021
LGTM. Remember to bump .Dd when you commit.
Mar 10 2021
Mar 10 2021
LGTM, for whatever little that's worth.
Mar 9 2021
Mar 9 2021
Can drivers are created in a callback in xpt_done_td.
Mar 2 2021
Mar 2 2021
Document the new depop command.
Implement and document the new "depop" command.
Feb 28 2021
Feb 28 2021
Feb 27 2021
Feb 27 2021
Feb 25 2021
Feb 25 2021
In D28922#647557, @markj wrote:@rpokala could you please see if this patch fixes the hang you see when reading vm.pmap.kernel_maps while an nvdimm is mapped?
Feb 21 2021
Feb 21 2021
In D28818#645488, @khng300_gmail.com wrote:The old virtio_pci.c code handling legacy Virtio is ported over to
virtio_pci_legacy.c in FreeBSD 13 and -CURRENT
rpokala committed rG49318a0aac64: nvdimm(4): Export NVDIMM health flags via sysctl (authored by rpokala).
nvdimm(4): Export NVDIMM health flags via sysctl
rpokala committed rGea5a304c734f: nvdimm(4): Export NVDIMM health flags via sysctl (authored by rpokala).
nvdimm(4): Export NVDIMM health flags via sysctl
Feb 20 2021
Feb 20 2021
D26915 is marked as superseded by this one, but that one was for virtio_pci.c, and this one is for virtio_pci_legacy.c...?
Feb 19 2021
Feb 19 2021
Ignore generated LINT files
Ignore generated LINT files
Feb 18 2021
Feb 18 2021
rpokala committed rGbdde49b7c723: nvdimm(4): Export NVDIMM health flags via sysctl (authored by rpokala).
nvdimm(4): Export NVDIMM health flags via sysctl
LGTM, FWIW.
In D28738#644234, @allanjude wrote:In D28738#643660, @rpokala wrote:If the intention is to support smbios(4) on non-x86, shouldn't it be added to a non-x86 KERNCONF file?
That is done here: https://reviews.freebsd.org/D28744
Address review comments from @cem
Feb 17 2021
Feb 17 2021
Address review comments from @cem
If the intention is to support smbios(4) on non-x86, shouldn't it be added to a non-x86 KERNCONF file?
In D28700#642544, @mav wrote:In my driver in TrueNAS I am printing this in dmesg if anything other than ACPI_NFIT_MEM_HEALTH_ENABLED is set. Everything read from NFIT applies more to boot state, rather than run-time.
Address review comments from @mav
rpokala added inline comments to D28720: Simplify and update the hardware notes for x86 architectures..
Feb 16 2021
Feb 16 2021
Feb 11 2021
Feb 11 2021
nvdimm: Fix error path mis-free
It probably goes without saying, but please confirm that contrib/mtree/mtree.8 is what's getting installed as mtree.8, not usr.sbin/fmtree/mtree.8.
Feb 8 2021
Feb 8 2021
Feb 4 2021
Feb 4 2021
Jan 28 2021
Jan 28 2021
Jan 26 2021
Jan 26 2021
In D27194#634238, @khng300_gmail.com wrote:In D27194#634011, @rpokala wrote:Don't forget a manpage.
Seems like a manpage for vnode_pager_setsize(9) is also missing. Should I do it all together or separate the manpage into another differential?
Don't forget a manpage.
Jan 21 2021
Jan 21 2021
LGTM
Jan 20 2021
Jan 20 2021
Jan 19 2021
Jan 19 2021
Remember to bump .Dd at the top, to reflect the commit date. Other than that, looks good from a manpage standpoint.
Jan 14 2021
Jan 14 2021
Jan 11 2021
Jan 11 2021
Jan 4 2021
Jan 4 2021
In D27517#624368, @chuck wrote:In D27517#623929, @rpokala wrote:That having been said - this is because the failing_lba field isn't aligned, right? But according to NVMe-1.4, it is aligned: failing_lba is bytes [23:16] of the data structure, which is clearly both 4-byte and 8-byte aligned. So, what's the problem with just having it be a uint64_t?
In the first element of the results[] array, yes, failing_lba is nicely aligned. But the packed array element size (28 bytes) is not divisible by 8 which will force every other failing_lba field to only be 4 byte aligned.
Jan 3 2021
Jan 3 2021
In D27517#623771, @chuck wrote:Any feedback on the change of failing_lba to uint8_t [8]? Does anything else stick out?
Jan 1 2021
Jan 1 2021
Dec 10 2020
Dec 10 2020
Looks good! Thanks for making the changes. :-)
Dec 9 2020
Dec 9 2020
Dec 3 2020
Dec 3 2020
I don't have the hardware to test, but the change looks fine.
- The zero-initialization is obviously valid
- It's clear in context that XGBE_SET_ADV() (which sets the flag) is what was really intended, instead of testing the flag with XGBE_ADV() and throwing away the result.
Dec 1 2020
Dec 1 2020
In D27354#612664, @cy wrote:Correct. A constant rate limit allows the attackers to infer which source port the request is coming from reducing its randomness from 32 bits to the original 16 bits. (Same mistake WPA made with WPS.)
Nov 30 2020
Nov 30 2020
Forgive my (carefully cultivated) ignorance of the network stack, but I'd like to understand this:
Nov 29 2020
Nov 29 2020
Code change looks good to me; please update any manual pages.
In D27373#612130, @tsoome wrote:I guess it is better just to keep them all separate.
Nov 25 2020
Nov 25 2020
To reduce confusion, we maybe should pick better name to replace "efi_fb". The VT internal state is filled from this structure.
Implement vt_vbefb to support VBE framebuffer with VT
Nov 23 2020
Nov 23 2020
LGTM, but I'm not authoritative when it comes to CAM.
Nov 19 2020
Nov 19 2020
Nov 18 2020
Nov 18 2020
Oct 30 2020
Oct 30 2020
Oct 29 2020
Oct 29 2020
Oct 28 2020
Oct 28 2020
In D26994#602203, @imp wrote:All this was cut and pasted from sys/disk.h, almost vebatim
Oct 25 2020
Oct 25 2020
Oct 20 2020
Oct 20 2020
In D26254#599501, @melifaro wrote:In D26254#599487, @hselasky wrote:Hi,
It is not possible to re-use lagg<N> for infiniband, because we set the type of the network device when it is created. lagg<N> are created like ethernet and bond<N> are created like infiniband.
Let's try to work it backwards?
I'd start with an assumption that from user POV we should be able to use the same interface for the same type of job, similarly to bond in Linux.
I see multiple options here:
- make lagg auto-change type based on the first interface attached. That would involve quite a lot of mechanics with notifications when changing between eth/ib.
- require lagg creation to specify laggtype in the creation params, assuming ethernet by default. ifconfig(8) needs to be updated (changes similar to vxlan_create() in sbin/ifconfig/ifvxlan.c).
(1) benefit that it won't require any userland changes when merging, compared to (2), however it comes at the cost of greater kernel complexity.
What is your feeling on requiring users to rebuild ifconfig(8)? Is it an absolute no-go?
In D26254#599454, @melifaro wrote:The only real question/concern I have is the name. I'm a bit afraid that we're saying that "bond" is an IB port-channel and "lagg" is an ethernet port-channel. It's a bit confusing, given one cannot distinguish which is which by name and the fact that Linux folks are used to "bond" as the ethernet port-channel. If we really want to have the different names (to keep some applications happy?), would it be possible to have a name other than "bond"?
Oct 16 2020
Oct 16 2020
cdefs: require gcc 4.2
Oct 13 2020
Oct 13 2020
to ensure that any commands that completed after we last poll are handled in a timely manner.
Allow IP over IB to work with multiple FIBs.
Oct 12 2020
Oct 12 2020
In D26733#595994, @hselasky wrote:This piece of code is about to change with D26254 . I've added the patch there instead.
Oct 10 2020
Oct 10 2020
rpokala retitled D26733: Allow IPoIB to work with multiple FIBs from Allow IPoIB to work with mutiple FIBs to Allow IPoIB to work with multiple FIBs.
Oct 8 2020
Oct 8 2020
Anmol is no longer with Panasas. 🙁 We remain interested in this feature, but we no longer have anyone with the cycles to participate in the review.
rpokala removed a reviewer for D26254: Add support for IPoIB lagg devices in FreeBSD: anmolk_panasas.com.
Sep 25 2020
Sep 25 2020
Sep 23 2020
Sep 23 2020
Sep 21 2020
Sep 21 2020
FWIW, in consultation w/ @kevans, I did s/beadm/bectl/g on beinstall,sh to get myself out of a problem, and it worked without issue.
Sep 4 2020
Sep 4 2020
rpokala edited reviewers for D26254: Add support for IPoIB lagg devices in FreeBSD, added: anmolk_panasas.com; removed: rpokala, khirbat_gmail.com.
Removing myself as a reviewer, and adding (the correct) Anmol from Panasas instead.
Sep 3 2020
Sep 3 2020
I am nowhere near competent to review any of this, but I did skim it. One thing that jumped out at me is that you're using "bond", not "lagg", only for Infiniband. That seems like a gratuitous difference; why?
Sep 1 2020
Sep 1 2020
rpokala added inline comments to D26241: update exports.5 to include information on the TLS export options.
Aug 13 2020
Aug 13 2020
I don't know enough about ACPI to really comment on the code change. The manpage change looks fine, but please remember to bump .Dd when you actually commit.
Aug 12 2020
Aug 12 2020
rpokala added inline comments to D26012: Add apple-zfs and solaris-reserved to make `gpart show` more convenient.
Aug 7 2020
Aug 7 2020