rpokala (Ravi Pokala)
User

Projects

User Details

User Since
Nov 20 2015, 9:23 AM (138 w, 3 d)

Recent Activity

Wed, Jul 11

rpokala added a comment to D16211: run(4): Add a new USB device ID..

... the numbers just happened to coincide here.

Wed, Jul 11, 1:53 AM
rpokala added a comment to D16211: run(4): Add a new USB device ID..

The summary says PAU06 is the combination of two already-supported chips: RT5372 radio and RT5392 MAC. But nothing in the diff mentions the latter chip.

Wed, Jul 11, 1:27 AM

Fri, Jun 29

rpokala accepted D16064: Add conv=direct.
Fri, Jun 29, 4:21 PM

Mon, Jun 25

rpokala added inline comments to D15999: Create PCI_MATCH and pci_match_device.
Mon, Jun 25, 7:55 PM

Fri, Jun 22

rpokala added a comment to D15927: Update ixl(4) man page.
In D15927#338098, @erj wrote:
In D15927#337633, @pi wrote:

Please expand LOM to 'onboard interfaces (LAN on Motherboard, LOM)', if possible. I'm a 30-years user of ethernet, but I've never heard that abbreviation.

I don't think it's that rare of an abbreviation, but I can add the expansion.

Fri, Jun 22, 5:53 PM

Jun 9 2018

rpokala accepted D14818: lualoader: Add cli.lua(8) to the tree.
Jun 9 2018, 6:44 PM

Jun 8 2018

rpokala added a comment to D14819: lualoader: Add config.lua(8) to the tree.

Just a note: Anywhere I wrote {kernel} is because I couldn't come up with a better markup. I have no idea what the ideal syntax is for "this is a path with this variable substituted in" rather than "this is a path with this variable name literally in it"

Jun 8 2018, 3:52 AM
rpokala added a comment to D14476: libsa(3): Correct statement about FS Write-support, name change.

Might want to poke @imp and/or @jhb while all three of you are in the same city. :-)

Jun 8 2018, 3:51 AM
rpokala requested changes to D14819: lualoader: Add config.lua(8) to the tree.
Jun 8 2018, 3:49 AM
rpokala requested changes to D14818: lualoader: Add cli.lua(8) to the tree.
Jun 8 2018, 3:49 AM
rpokala requested changes to D14815: lualoader: Add hook.lua(8) to tree.
Jun 8 2018, 3:48 AM
rpokala closed D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
Jun 8 2018, 3:27 AM
rpokala added 1 commit(s) for D14790: jedec_dimm: Use correct string length when populating sc->slotid_str: rS331345: jedec_dimm: Use correct string length when populating sc->slotid_str.
Jun 8 2018, 3:27 AM
rpokala added an edge to rS331345: jedec_dimm: Use correct string length when populating sc->slotid_str: D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
Jun 8 2018, 3:27 AM
rpokala abandoned D5672: MAC of VLAN interface does not change.
Jun 8 2018, 3:22 AM
rpokala commandeered D5672: MAC of VLAN interface does not change.
Jun 8 2018, 3:22 AM

Jun 7 2018

rpokala added a comment to D15689: Including rc_conf_directories to be loaded with rc(8).

I think this looks good now. Now you can point someone who actually knows `rc' at it. :-)

Jun 7 2018, 4:08 PM
rpokala added inline comments to D15689: Including rc_conf_directories to be loaded with rc(8).
Jun 7 2018, 2:44 PM
rpokala added inline comments to D15689: Including rc_conf_directories to be loaded with rc(8).
Jun 7 2018, 2:32 PM
rpokala added inline comments to D15689: Including rc_conf_directories to be loaded with rc(8).
Jun 7 2018, 2:22 PM

May 30 2018

rpokala committed rS334364: Remove the mlx5 driver from LINT kernel config for 32-bit PPC.
Remove the mlx5 driver from LINT kernel config for 32-bit PPC
May 30 2018, 2:26 AM
rpokala closed D15611: Remove mlx5(4) from LINT kernel config for 32-bit PPC.
May 30 2018, 2:26 AM

May 29 2018

rpokala retitled D15611: Remove mlx5(4) from LINT kernel config for 32-bit PPC from mlx5(4) requires 64-bit atomics, which are not supported on 32-bit PPC exclude that driver from the LINT kernel config. to Remove mlx5(4) from LINT kernel config for 32-bit PPC.
May 29 2018, 10:15 PM
rpokala created D15611: Remove mlx5(4) from LINT kernel config for 32-bit PPC.
May 29 2018, 9:54 PM

May 17 2018

rpokala added a comment to D15465: Enable kernel dump options on most architectures..
In D15465#326493, @cem wrote:

It looks like the GENERIC configs that you're not updating are for ARM, RISC-V, and PPC64. Why are they being excluded?

PPC64 is not excluded. That's powerpc/conf/GENERIC64.

May 17 2018, 11:57 PM
rpokala added a comment to D15465: Enable kernel dump options on most architectures..

It looks like the GENERIC configs that you're not updating are for ARM, RISC-V, and PPC64. Why are they being excluded?

May 17 2018, 11:31 PM

Apr 10 2018

rpokala updated subscribers of D15029: new utility usr.sbin/spi.

Oh, heh, I didn't even realize that this wasn't @ian, since he and I talked about this yesterday. :-)

Apr 10 2018, 11:59 PM
rpokala added a comment to D15029: new utility usr.sbin/spi.

I started but didn't finish reading through the manpage. My comments so far are in-line.

Apr 10 2018, 6:42 PM

Mar 29 2018

rpokala added a comment to D14897: Fallback to copy if chown/chgrp would land in different filesystems.

Out of curiosity, why specifically for chown / chgrp?

Mar 29 2018, 5:30 PM
rpokala committed rS331734: MFC 331345:.
MFC 331345:
Mar 29 2018, 3:01 PM
rpokala committed rS331733: MFC 331345:.
MFC 331345:
Mar 29 2018, 3:01 PM

Mar 24 2018

rpokala added a comment to D14815: lualoader: Add hook.lua(8) to tree.

It would be good if you explained what a hook was, somewhere before the Exported functions section.

Mar 24 2018, 12:35 AM

Mar 22 2018

rpokala committed rS331345: jedec_dimm: Use correct string length when populating sc->slotid_str.
jedec_dimm: Use correct string length when populating sc->slotid_str
Mar 22 2018, 6:31 AM
rpokala added a comment to D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
In D14790#311049, @cem wrote:

Feel free to spend a bunch of time documenting libkern if you want. It's definitely valuable and shouldn't be too hard. Mostly just tedious. Maybe a thin document that just defines all of the libc-alike aliases and gets linked in a million places would be enough to start.

Mar 22 2018, 6:14 AM
rpokala added a comment to D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
# Much longer than sizeof(sc->slotid_str)
% kenv hint.jedec_dimm.0.slotid
DIMM0LONGERSTRING
Mar 22 2018, 5:58 AM
rpokala updated the diff for D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.

As @cem pointed out, it's even easier to use strdup().

Mar 22 2018, 5:51 AM
rpokala added inline comments to D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
Mar 22 2018, 2:52 AM
rpokala updated the diff for D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.

@imp correctly observed that since the buffer is allocated to fit, then
fully initialized, there's no need to use M_ZERO.

Mar 22 2018, 1:19 AM
rpokala created D14790: jedec_dimm: Use correct string length when populating sc->slotid_str.
Mar 22 2018, 1:12 AM

Mar 9 2018

rpokala accepted D14569: Permit sysctl(8) to set an array of numeric values for a single node..
Mar 9 2018, 10:47 PM
rpokala committed rS330680: MFC r330304: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus.
MFC r330304: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus
Mar 9 2018, 2:55 AM
rpokala committed rS330679: MFC r330304: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus.
MFC r330304: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus
Mar 9 2018, 2:55 AM

Mar 7 2018

rpokala accepted D14604: Add CTASSERTs for where CTLFLAG_MPSAFE is redundant..

Looks fine to my relatively-untrained eye.

Mar 7 2018, 4:42 PM

Mar 3 2018

rpokala committed rS330304: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver
Mar 3 2018, 1:54 AM
rpokala closed D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Mar 3 2018, 1:54 AM
rpokala added a comment to D14569: Permit sysctl(8) to set an array of numeric values for a single node..

Modulo adding a comment, LGTM.

Mar 3 2018, 12:06 AM

Mar 2 2018

rpokala accepted D14479: Add core.lua(8), but do not add to distribution.
Mar 2 2018, 5:30 AM
rpokala accepted D14480: Add menu.lua(8), but do not add to distribution.
Mar 2 2018, 5:06 AM
rpokala added inline comments to D14479: Add core.lua(8), but do not add to distribution.
Mar 2 2018, 5:01 AM

Mar 1 2018

rpokala added a comment to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

@avg / @jhb : I think I've addressed everything you were concerned about -- modulo some personal style, which is still conformant with style(9).

Mar 1 2018, 8:34 PM
rpokala added inline comments to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Mar 1 2018, 2:00 AM
rpokala updated the diff for D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

Address a few more things @jhb mentioned.

Mar 1 2018, 2:00 AM
rpokala added inline comments to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Mar 1 2018, 1:07 AM
rpokala added inline comments to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Mar 1 2018, 1:00 AM
rpokala updated the diff for D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

Call only bus_generic_attach(), but not bus_generic_probe(), from the attach
methods. This behaves properly for both pre-loading via loader.conf and loading
via kldload.

Mar 1 2018, 12:59 AM

Feb 28 2018

rpokala committed rS330109: MFC r329843:.
MFC r329843:
Feb 28 2018, 10:00 AM
rpokala committed rS330107: MFC r329843:.
MFC r329843:
Feb 28 2018, 10:00 AM
rpokala added a comment to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

At this point, it looks like the only outstanding issue is either adding device_probe_and_attach(), or confirming that it's not necessary. I'll try to do that tomorrow.

Feb 28 2018, 8:22 AM
rpokala added inline comments to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Feb 28 2018, 2:37 AM
rpokala updated the diff for D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

Address most of @avg's review comments.

Feb 28 2018, 2:31 AM
rpokala committed rS330093: MFC r329682:.
MFC r329682:
Feb 28 2018, 12:30 AM
rpokala committed rS330092: MFC r329682:.
MFC r329682:
Feb 28 2018, 12:30 AM
rpokala added inline comments to D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Feb 28 2018, 12:15 AM

Feb 27 2018

rpokala updated subscribers of D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

Thanks for finding time for this! I'll address many of your comments and post an updated diff later today.

Feb 27 2018, 5:38 PM

Feb 23 2018

rpokala committed rS329871: Somehow r329870 disn't include the mergeinfo...?.
Somehow r329870 disn't include the mergeinfo...?
Feb 23 2018, 4:51 PM
rpokala committed rS329870: MFC r323508:.
MFC r323508:
Feb 23 2018, 4:47 PM
rpokala committed rS329869: MFC r323508:.
MFC r323508:
Feb 23 2018, 4:46 PM
rpokala added inline comments to D14479: Add core.lua(8), but do not add to distribution.
Feb 23 2018, 5:06 AM
rpokala added inline comments to D14480: Add menu.lua(8), but do not add to distribution.
Feb 23 2018, 4:51 AM
rpokala added inline comments to D14480: Add menu.lua(8), but do not add to distribution.
Feb 23 2018, 3:55 AM

Feb 22 2018

rpokala committed rS329843: jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs.
jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs
Feb 22 2018, 11:19 PM
rpokala closed D14392: jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs.
Feb 22 2018, 11:19 PM
rpokala accepted D14476: libsa(3): Correct statement about FS Write-support, name change.

LGTM.

Feb 22 2018, 10:57 PM
rpokala committed rS329834: MFC r329295:.
MFC r329295:
Feb 22 2018, 7:40 PM
rpokala committed rS329833: MFC r329295:.
MFC r329295:
Feb 22 2018, 7:39 PM

Feb 21 2018

rpokala updated the diff for D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

Remove some stray comments from the manpage.

Feb 21 2018, 2:29 AM
rpokala updated the diff for D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.

style(9) changes: Correct formatting of function prototypes and declarations, and re-order variables.

Feb 21 2018, 1:43 AM
rpokala committed rS329682: mountd: Return proper errno values in a few error paths.
mountd: Return proper errno values in a few error paths
Feb 21 2018, 12:19 AM
rpokala closed D14438: Return proper errno values in a few error paths..
Feb 21 2018, 12:19 AM

Feb 20 2018

rpokala updated subscribers of D14392: jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs.
Feb 20 2018, 11:28 PM
rpokala updated the diff for D14392: jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs.

Address some of @cem's review comments.

Feb 20 2018, 11:27 PM
rpokala updated subscribers of D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Feb 20 2018, 3:40 AM
rpokala created D14447: imcsmb(4): Intel integrated Memory Controller (iMC) SMBus controller driver.
Feb 20 2018, 3:39 AM

Feb 19 2018

rpokala created D14438: Return proper errno values in a few error paths..
Feb 19 2018, 5:09 PM

Feb 16 2018

rpokala added inline comments to D14402: PID Controlled page daemon.
Feb 16 2018, 9:17 PM
rpokala added a comment to D14381: mxge(4) should pass unhandled ioctls to ether_ioctl().

Well thats embarrassing. Thanks for the fix.

Feb 16 2018, 2:29 AM

Feb 15 2018

rpokala created D14392: jedec_dimm(4): report asset info and temperatures for DDR3 and DDR4 DIMMs.
Feb 15 2018, 11:05 PM
rpokala closed D14381: mxge(4) should pass unhandled ioctls to ether_ioctl().
Feb 15 2018, 3:22 AM
rpokala committed rS329295: mxge(4) should pass unhandled ioctls to ether_ioctl().
mxge(4) should pass unhandled ioctls to ether_ioctl()
Feb 15 2018, 3:22 AM
rpokala created D14381: mxge(4) should pass unhandled ioctls to ether_ioctl().
Feb 15 2018, 3:01 AM

Feb 8 2018

rpokala closed D14055: jedec_ts(4) is broken on stable/10.
Feb 8 2018, 9:24 AM
rpokala committed rS329015: jedec_ts(4) uses a sysctl format specifier of "IK4", to indicate that it.
jedec_ts(4) uses a sysctl format specifier of "IK4", to indicate that it
Feb 8 2018, 9:24 AM

Feb 6 2018

rpokala accepted D14231: Avoid find -s, use find | sort instead..
Feb 6 2018, 10:12 PM

Jan 31 2018

rpokala updated the diff for D14055: jedec_ts(4) is broken on stable/10.

Leading 8-space -> hard-tab. Whoops!

Jan 31 2018, 4:00 PM
rpokala updated the diff for D14055: jedec_ts(4) is broken on stable/10.

Use the obvious one-liner, which somehow escaped me at the time of the
original diff. Thanks, avg@!

Jan 31 2018, 3:16 PM
rpokala added inline comments to D14055: jedec_ts(4) is broken on stable/10.
Jan 31 2018, 3:13 PM

Jan 26 2018

rpokala archived P141 Is this license BSD-compatible?.
Jan 26 2018, 9:35 PM
rpokala created D14055: jedec_ts(4) is broken on stable/10.
Jan 26 2018, 12:39 AM

Jan 25 2018

rpokala added inline comments to D14022: NVMe emulation for bhyve (improved).
Jan 25 2018, 11:51 PM
rpokala accepted D14051: Remove K&R stuff: never use it in new code..
Jan 25 2018, 3:56 PM