Page MenuHomeFreeBSD
Feed Advanced Search

Nov 12 2021

avg added a comment to D32926: iflib_stop: drain rx tasks to prevent any data races.

Do transmit task queues need to be drained as well?

Nov 12 2021, 9:59 AM

Nov 10 2021

avg updated the summary of D32926: iflib_stop: drain rx tasks to prevent any data races.
Nov 10 2021, 2:06 PM
avg added reviewers for D32926: iflib_stop: drain rx tasks to prevent any data races: markj, pkelsey, allanjude, gallatin.
Nov 10 2021, 2:02 PM
avg requested review of D32926: iflib_stop: drain rx tasks to prevent any data races.
Nov 10 2021, 1:58 PM

Nov 8 2021

avg committed rGd3aca6fe9bf9: fuser: restore functionality by fixing fsid type (authored by avg).
fuser: restore functionality by fixing fsid type
Nov 8 2021, 6:01 AM
avg committed rGdf838c540632: fuser: restore functionality by fixing fsid type (authored by avg).
fuser: restore functionality by fixing fsid type
Nov 8 2021, 5:56 AM

Nov 6 2021

avg committed rGcc0b35259aed: rk805: add system poweroff support (authored by avg).
rk805: add system poweroff support
Nov 6 2021, 6:01 PM
avg closed D30786: rk805: add system poweroff support.
Nov 6 2021, 6:01 PM
avg committed rG2d1de2d55320: rk3328_codec: set output gain to the value found in linux (authored by avg).
rk3328_codec: set output gain to the value found in linux
Nov 6 2021, 5:53 PM
avg committed rG69d202f22400: rk3328_codec: bump delays in set_power and set_mute (authored by avg).
rk3328_codec: bump delays in set_power and set_mute
Nov 6 2021, 5:53 PM
avg committed rG507fdedd83cd: rk3328_codec: add delays between register writes in the initial setup (authored by avg).
rk3328_codec: add delays between register writes in the initial setup
Nov 6 2021, 5:53 PM
avg committed rG5d1a2edeec2e: rk_i2s: modify GRF_SOC_CON8 only for rk3399 (authored by avg).
rk_i2s: modify GRF_SOC_CON8 only for rk3399
Nov 6 2021, 5:53 PM
avg committed rG8c9bdaf227b1: rk3328_codec: fix a problem with the enable value (authored by avg).
rk3328_codec: fix a problem with the enable value
Nov 6 2021, 5:53 PM
avg committed rGaa917184a9c0: add rk3328_codec and rk_i2s from gonzoua/socaudio (authored by gonzo).
add rk3328_codec and rk_i2s from gonzoua/socaudio
Nov 6 2021, 5:50 PM
avg committed rG24f567bb0e50: files.arm64: fix sort order for aw_i2s.c (authored by avg).
files.arm64: fix sort order for aw_i2s.c
Nov 6 2021, 5:46 PM
avg committed rG43b031a371eb: htu21: don't needlessly bother hardware when measurements are not needed (authored by avg).
htu21: don't needlessly bother hardware when measurements are not needed
Nov 6 2021, 5:42 PM
avg committed rGc01a46d4acab: camcontrol: dump received data for MMC command even if it is unknown (authored by avg).
camcontrol: dump received data for MMC command even if it is unknown
Nov 6 2021, 5:30 PM
avg committed rGa75159eabc90: htu21: allow configuration via hints on FDT-based systems (authored by avg).
htu21: allow configuration via hints on FDT-based systems
Nov 6 2021, 5:25 PM
avg committed rG6354154ef520: pcf8574: driver for 8-pin quasi-bidirectional GPIO over I2C (authored by avg).
pcf8574: driver for 8-pin quasi-bidirectional GPIO over I2C
Nov 6 2021, 5:25 PM
avg committed rG832503063e32: gpioled: allow the driver to be disabled via fdt (authored by avg).
gpioled: allow the driver to be disabled via fdt
Nov 6 2021, 5:21 PM
avg committed rGff6fe2983550: driver for MAX44009 I2C illuminance sensor (authored by avg).
driver for MAX44009 I2C illuminance sensor
Nov 6 2021, 5:20 PM
avg committed rGa60b30469741: pcf8591: driver for adc/dac with i2c interface (authored by avg).
pcf8591: driver for adc/dac with i2c interface
Nov 6 2021, 5:15 PM

Nov 2 2021

avg accepted D32803: scsi_cd: Improve TOC access validation.

LGTM

Nov 2 2021, 3:23 PM

Nov 1 2021

avg committed rGe18fbe6f19ed: fuser: restore functionality by fixing fsid type (authored by avg).
fuser: restore functionality by fixing fsid type
Nov 1 2021, 6:55 AM

Oct 6 2021

avg added a comment to D32324: bsd.port.mk: Do not pass FLAVOR in dependency make environment.

I think I also had to do the following as well.
If a recursive target is used in a flavored port without that change then the port's default flavor would get used instead of a user specified flavor.
That happens because FLAVOR variable gets unset and the original port's directory is specified without the flavor suffix.

Oct 6 2021, 9:28 AM

Oct 5 2021

avg accepted D32324: bsd.port.mk: Do not pass FLAVOR in dependency make environment.
Oct 5 2021, 4:04 PM
avg added a comment to D32324: bsd.port.mk: Do not pass FLAVOR in dependency make environment.

Good catch! Thank you!
I also noticed this problem just recently and was about to propose a slightly different patch:

diff --git a/Mk/bsd.port.mk b/Mk/bsd.port.mk
index 0fe16fa5426d..fea489a8c42f 100644
--- a/Mk/bsd.port.mk
+++ b/Mk/bsd.port.mk
@@ -4095,7 +4095,7 @@ _FLAVOR_RECURSIVE_SH= \
                /*) ;; \
                *) dir=${PORTSDIR}/$$dir ;; \
                esac; \
-               (cd $$dir; ${SETENV} $${flavor:+FLAVOR=$${flavor}} ${MAKE} $${recursive_cmd}); \
+               (cd $$dir; unset FLAVOR; ${SETENV} $${flavor:+FLAVOR=$${flavor}} ${MAKE} $${recursive_cmd}); \
        done
Oct 5 2021, 4:03 PM

Sep 22 2021

avg accepted D30387: Make the code consistent with the comments.

Please commit this.
If not too much trouble please also update dwmmc_get_tran_settings (used with MMCCAM) to use the same formula.
Thank you very much!

Sep 22 2021, 7:30 AM

Jul 21 2021

avg added a comment to D31215: led: call led state functions in a context with no locks held.

@wulf , I considered that well. But if we use a dedicate taskqueue then it means having a thread anyway. And I didnd't want to put potentially slow LED operations on one of the global taskqueues.

Jul 21 2021, 7:38 AM

Jul 19 2021

avg added a comment to D31215: led: call led state functions in a context with no locks held.

@adrian , in my opinion two things guarantee correct operation:

  • the current element can never be removed (thanks to in_use)
  • an iteration is always done with the lock held

So, whatever perturbations might happen while the lock is dropped, they should settle down before the next iteration.

Jul 19 2021, 7:23 PM
avg added a comment to D31215: led: call led state functions in a context with no locks held.

@adrian , I thought that I took care of that via the in_use flag.
Could you please look again?

Jul 19 2021, 5:30 PM
avg added a comment to D31215: led: call led state functions in a context with no locks held.

Yeah, me too... but doing it directly from a callout puts a lot of restrictions.
I considered creating and destroying the thread dynamically as "blinkers" are added and removed.
I can implement that if you think it would be better than having the thread permanently.

Jul 19 2021, 8:15 AM
avg added a comment to D31215: led: call led state functions in a context with no locks held.

Tested this with D30787 where LEDs are controlled via I²C and had no issues with turning the LEDs on / off and making then blink in various combinations and sequences.
No crashes, no LORs, no WITNESS warnings.

Jul 19 2021, 8:10 AM
avg added reviewers for D31215: led: call led state functions in a context with no locks held: phk, gonzo, loos, adrian, mav, ganbold.
Jul 19 2021, 8:04 AM
avg requested review of D31215: led: call led state functions in a context with no locks held.
Jul 19 2021, 8:01 AM
avg added reviewers for D31214: gpioloed: no need for locking in this driver: gonzo, adrian, loos.
Jul 19 2021, 8:00 AM
avg requested review of D31214: gpioloed: no need for locking in this driver.
Jul 19 2021, 7:58 AM
avg committed rG038279d2a86f: rtwn: make sure to not write in upper bits of txdseq (authored by avg).
rtwn: make sure to not write in upper bits of txdseq
Jul 19 2021, 6:50 AM
avg committed rGc5fa78d35d71: rtwn: make sure to not write in upper bits of txdseq (authored by avg).
rtwn: make sure to not write in upper bits of txdseq
Jul 19 2021, 6:46 AM

Jul 12 2021

avg committed rG66c183f43f0c: mmc_cam_sim_default_action: do not touch the ccb after dispatching it (authored by avg).
mmc_cam_sim_default_action: do not touch the ccb after dispatching it
Jul 12 2021, 6:32 PM

Jun 21 2021

avg committed rG7544c1d20d8b: rtwn: make sure to not write in upper bits of txdseq (authored by avg).
rtwn: make sure to not write in upper bits of txdseq
Jun 21 2021, 8:07 AM
avg closed D30814: rtwn: make sure to not write in upper bits of txdseq.
Jun 21 2021, 8:07 AM
avg added a comment to D30787: rk805: add support for controlling two output pins via gpio interface.

I cannot think of any way.
rk_i2c needs to sleep to wait for an interrupt.
led(8) uses a non-sleepable mutex.

Jun 21 2021, 8:02 AM

Jun 18 2021

avg added reviewers for D30814: rtwn: make sure to not write in upper bits of txdseq: adrian, avos, bz, kevlo.
Jun 18 2021, 3:02 PM
avg requested review of D30814: rtwn: make sure to not write in upper bits of txdseq.
Jun 18 2021, 3:00 PM

Jun 17 2021

avg updated the diff for D30786: rk805: add system poweroff support.

address reviews for rk805 poweroff support

Jun 17 2021, 7:05 AM

Jun 16 2021

avg added inline comments to D30786: rk805: add system poweroff support.
Jun 16 2021, 2:49 PM
avg added reviewers for D30787: rk805: add support for controlling two output pins via gpio interface: peterj, gonzo. avg added 1 blocking reviewer(s) for D30787: rk805: add support for controlling two output pins via gpio interface: manu.
Jun 16 2021, 10:55 AM
avg requested review of D30787: rk805: add support for controlling two output pins via gpio interface.
Jun 16 2021, 10:53 AM
avg added reviewers for D30786: rk805: add system poweroff support: peterj, bz. avg added 1 blocking reviewer(s) for D30786: rk805: add system poweroff support: manu.
Jun 16 2021, 10:52 AM
avg committed rG0dc7e55dcb0e: rc.d: liberate powerd from ACPI dependency (authored by avg).
rc.d: liberate powerd from ACPI dependency
Jun 16 2021, 10:50 AM
avg requested review of D30786: rk805: add system poweroff support.
Jun 16 2021, 10:49 AM
avg committed rGe7ca75cb75be: rc.d: liberate powerd from ACPI dependency (authored by avg).
rc.d: liberate powerd from ACPI dependency
Jun 16 2021, 10:47 AM
avg committed rG118ff4e9b3f1: rk3328_cru: fix a typo in the SCLK_I2S2 gate definition (authored by avg).
rk3328_cru: fix a typo in the SCLK_I2S2 gate definition
Jun 16 2021, 10:43 AM

Jun 9 2021

avg committed rGffc5dc788f05: rk3328_cru: fix a typo in the SCLK_I2S2 gate definition (authored by avg).
rk3328_cru: fix a typo in the SCLK_I2S2 gate definition
Jun 9 2021, 9:43 AM
avg committed rG20eb6bd8c598: rc.d: liberate powerd from ACPI dependency (authored by avg).
rc.d: liberate powerd from ACPI dependency
Jun 9 2021, 9:42 AM

May 31 2021

avg requested review of D30579: bsd.port.mk: fix setting of FLAVOR in _FLAVOR_RECURSIVE_SH.
May 31 2021, 4:22 PM

May 21 2021

avg committed rG9120716d91d4: storvsc: fix auto-sense reporting (authored by avg).
storvsc: fix auto-sense reporting
May 21 2021, 1:53 PM
avg committed rG4b9d54e2aeca: mpsutil: extend show adapter information, add NCQ control (authored by Daniel Austin <freebsd-ports@dan.me.uk>).
mpsutil: extend show adapter information, add NCQ control
May 21 2021, 1:45 PM
avg committed rG4a7dcb174ce8: storvsc: fix auto-sense reporting (authored by avg).
storvsc: fix auto-sense reporting
May 21 2021, 10:33 AM
avg committed rG2def2de60a6d: PCI hot-plug: use dedicated taskqueue for device attach / detach (authored by avg).
PCI hot-plug: use dedicated taskqueue for device attach / detach
May 21 2021, 10:32 AM
avg committed rG395bc3598b47: mpsutil: extend show adapter information, add NCQ control (authored by Daniel Austin <freebsd-ports@dan.me.uk>).
mpsutil: extend show adapter information, add NCQ control
May 21 2021, 10:30 AM
avg committed rGd7d4592ada54: PCI hot-plug: use dedicated taskqueue for device attach / detach (authored by avg).
PCI hot-plug: use dedicated taskqueue for device attach / detach
May 21 2021, 10:29 AM

May 7 2021

avg committed rGe2ea6942ab31: mpsutil: extend show adapter information, add NCQ control (authored by Daniel Austin <freebsd-ports@dan.me.uk>).
mpsutil: extend show adapter information, add NCQ control
May 7 2021, 12:24 PM
avg committed rG8afecefd57c5: storvsc: fix auto-sense reporting (authored by avg).
storvsc: fix auto-sense reporting
May 7 2021, 7:20 AM
avg closed D30124: storvsc: fix auto-sense reporting.
May 7 2021, 7:20 AM

May 6 2021

avg closed D30144: PCI hot-plug: use dedicated taskqueue for device attach / detach.
May 6 2021, 6:53 PM
avg committed rG12588ce02dd8: PCI hot-plug: use dedicated taskqueue for device attach / detach (authored by avg).
PCI hot-plug: use dedicated taskqueue for device attach / detach
May 6 2021, 6:53 PM
avg updated the summary of D30144: PCI hot-plug: use dedicated taskqueue for device attach / detach.
May 6 2021, 5:58 AM
avg added reviewers for D30144: PCI hot-plug: use dedicated taskqueue for device attach / detach: imp, jhb, chuck, mav, scottl.
May 6 2021, 5:57 AM
avg requested review of D30144: PCI hot-plug: use dedicated taskqueue for device attach / detach.
May 6 2021, 5:55 AM

May 5 2021

avg added reviewers for D30124: storvsc: fix auto-sense reporting: whu, decui_microsoft.com, imp.
May 5 2021, 2:16 PM
avg requested review of D30124: storvsc: fix auto-sense reporting.
May 5 2021, 2:14 PM
avg added a comment to D26311: gpioled_acpi: driver for GPIO controlled LEDs declared using ACPI _DSD.

The only reason is that I thought that I could do some things better but never got around to actually working on them.
Specifically, I don't like having so many walk+callback patterns. Especially for _DSD.
I hoped that I would be able to represent it as an nvlist or something similar.

May 5 2021, 7:36 AM
avg added a comment to D26311: gpioled_acpi: driver for GPIO controlled LEDs declared using ACPI _DSD.

@mm , do you have amdgpio driver? Has it attached?

May 5 2021, 6:54 AM

May 3 2021

avg committed rG0a28a6a608d0: gpioc_detach: fix freeing of wrong pointers (authored by avg).
gpioc_detach: fix freeing of wrong pointers
May 3 2021, 8:16 PM
avg committed rGdec9f377531d: gpioc_detach: fix freeing of wrong pointers (authored by avg).
gpioc_detach: fix freeing of wrong pointers
May 3 2021, 8:12 PM

Apr 15 2021

avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.

@kibab , you still may want to fix this pseudo-loop: https://github.com/kibab/freebsd/blob/mmccam-current/sys/cam/mmc/mmc_da.c#L1638
Maybe the last 'else' was supposed to be 'else if' ?

Apr 15 2021, 8:44 AM
avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.

I think that I needed that patch because earlier I had a lot of bits from https://github.com/kibab/freebsd/tree/mmccam-current in my tree and now I don't.
Specifically, @kibab's mmc_set_timing supports a lot more timings and so on.

Apr 15 2021, 8:40 AM
avg abandoned D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.
Apr 15 2021, 8:20 AM
avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.

And in fact, it seems that I do not need that change anymore.

Apr 15 2021, 8:20 AM
avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.
In D29758#667600, @manu wrote:

For the 1.2V report it's probably the same problem as the one fixed by 1a96c1434f4b4487126f13ede29e67ad44a8776a

Apr 15 2021, 8:06 AM

Apr 14 2021

avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.

Yes, most likely.

Apr 14 2021, 6:07 PM
avg added a comment to D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.

Is this the information you asked for?

# camcontrol devlist -v       
scbus0 on aw_mmc_sim0 bus 0:
scbus1 on aw_mmc_sim1 bus 0:
<SDIO card>                        at scbus1 target 0 lun 0 (sdiob0,pass1)
scbus2 on aw_mmc_sim2 bus 0:
<MMCHC 8GME4R 0.1 SN 4DAD8E5D MFG 09/200>  at scbus2 target 0 lun 0 (sdda0,pass0)
scbus-1 on xpt0 bus 0:
<>                                 at scbus-1 target -1 lun ffffffff (xpt0)
Apr 14 2021, 3:14 PM
avg requested review of D29758: mmc_da: restore HS200 support that was present in MMCCAM branch.
Apr 14 2021, 9:56 AM

Mar 31 2021

avg accepted D29513: lock_delay(9): improve interaction with restrict_starvation.

LGTM. Thank you!

Mar 31 2021, 2:39 PM

Mar 23 2021

avg committed rG395e612b7bb1: pchtherm: fix a wrong bit and a wrong register use (authored by Michal Vanco <michal.vanco@gmail.com>).
pchtherm: fix a wrong bit and a wrong register use
Mar 23 2021, 11:03 AM
avg committed rG3c6b59567f61: gpioc_detach: fix freeing of wrong pointers (authored by avg).
gpioc_detach: fix freeing of wrong pointers
Mar 23 2021, 10:50 AM

Mar 5 2021

avg committed rG5084dde5f087: pchtherm: fix a wrong bit and a wrong register use (authored by Michal Vanco <michal.vanco@gmail.com>).
pchtherm: fix a wrong bit and a wrong register use
Mar 5 2021, 9:04 AM
avg added a comment to D24077: New driver pchtherm(4), PCH installed thermal sensor..

fix man page.
Add files.x86 entry to static configuration.

Mar 5 2021, 8:53 AM

Feb 18 2021

avg accepted D28769: kobj: avoid gcc -Wcast-function-type.

LGTM. Thank you.

Feb 18 2021, 9:13 AM

Jan 20 2021

avg accepted D28196: hconf(4): Do not fetch report before writing new usage values back..

LGTM. Thank you!
I did not realize that hardware can be this quirky.

Jan 20 2021, 12:41 PM

Jan 19 2021

avg committed rG2c98edd6d6c7: htu21: driver for HTU21D I2C temperature and humidity sensor (authored by avg).
htu21: driver for HTU21D I2C temperature and humidity sensor
Jan 19 2021, 1:12 PM

Dec 18 2020

avg accepted D27653: acpi_wmi: Allow attachment to ACPI node if EC is not found. .
Dec 18 2020, 9:32 AM

Dec 13 2020

avg committed R9:b0d9f04266c0: add myself to authors.ent (first commit) (authored by avg).
add myself to authors.ent (first commit)
Dec 13 2020, 6:06 PM
avg committed R9:b7acae7d968f: further add myself to documentation (authored by avg).
further add myself to documentation
Dec 13 2020, 6:06 PM
avg committed R9:bbeeb6d23b57: further add myself to www pages (authored by avg).
further add myself to www pages
Dec 13 2020, 5:42 PM

Dec 11 2020

avg accepted D27570: superio,wbwd: Add Nuvoton NCT6796.

LGTM.

Dec 11 2020, 1:41 PM

Dec 10 2020

avg added a comment to D27535: Handle return from KDB_WHY_TRAP on arm and mips.

It's been a while since I touched that kind of code, but I think that this change is good.

Dec 10 2020, 3:24 PM

Dec 3 2020

avg committed rS368300: dtrace: honor LC_NUMERIC for %'d and alike, and LC_TIME for %T.
dtrace: honor LC_NUMERIC for %'d and alike, and LC_TIME for %T
Dec 3 2020, 11:59 AM