Page MenuHomeFreeBSD

ehem_freebsd_m5p.com (Elliott Mitchell)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 20 2021, 4:27 AM (92 w, 6 d)

Recent Activity

Mon, Nov 28

ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Finally tracked down an actual bug I'd been running into. Since I fear you may wish to take small steps and fix strictly what is buggy, I've split this into a separate commit at the front of my repository. This is clearly a bug with vmstat since this violates the sysctl() interface.

Mon, Nov 28, 8:43 PM

Mon, Nov 14

ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Updating to current tree status. At least one serious bug was fixed.

Mon, Nov 14, 8:35 PM

Sat, Nov 12

ehem_freebsd_m5p.com added a comment to D32504: kern/intr: remove "irq" from kernel event API.

The number of reviewers and subscribers to D32504 keeps increasing, but the review activity is stale. Since the previous approach was refused, there is now a new approach.

Sat, Nov 12, 1:11 AM
ehem_freebsd_m5p.com added a comment to D35417: kern/intr: allow for event allocation outside kern_intr.c.

Any reviews for this? While it will take a while to make use of this, I have a definite use for this. I was actually hoping for progress, not merely silence on this.

Sat, Nov 12, 1:00 AM

Thu, Nov 10

ehem_freebsd_m5p.com updated subscribers of D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.
In D36628#837924, @kib wrote:

You did exactly the opposite to what I suggested: you adjusted userspace to kernel interface, instead of fixing the ABI and keeping old vmstat -i working, by fixing kernel. I do not want to enforce my opinion, but at least I thin we need to get the rationale for going this way instead.

Thu, Nov 10, 10:16 PM

Wed, Nov 9

ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Thank you for figuring that out. Though I just lost rather a lot of sanity points.

Wed, Nov 9, 10:07 PM

Tue, Nov 8

ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

I just noticed a serious issue. Turns out intrnames is being interpreted in two very distinct fashions in different places. The resultant output is certainly incorrect and may have been broken before intrnames was introduced, yet never fixed.

Tue, Nov 8, 4:54 PM

Oct 21 2022

ehem_freebsd_m5p.com added a comment to D30006: xen/intr: adjust xen_intr_handle_upcall() to match interrupt filter.

This needs moving earlier since it needs to alter xen_intr_handle_upcall()'s declaration.

Oct 21 2022, 12:41 AM
ehem_freebsd_m5p.com updated the diff for D30236: xen/intr: move sys/x86/xen/xen_intr.c to sys/xen/xen_intr.c.

Splitting this. Depending upon how things evolve, there could be a need to have the two files live side-by-side for a bit. The non-split version was okayed in e-mail.

Oct 21 2022, 12:24 AM
ehem_freebsd_m5p.com added a comment to D35898: intrng: Extract arm/arm64 IPI->PIC glue code.

I've tried this and it unsurprisingly passed. Still have the objection, but noting it does appear to work.

Oct 21 2022, 12:20 AM
ehem_freebsd_m5p.com added a comment to D30006: xen/intr: adjust xen_intr_handle_upcall() to match interrupt filter.

@cperciva could I persuade you to try testing D30006? I'm cautiously optimistic it does in fact work for x86, but the problem is I'm setup for testing ARM64, not x86.

Oct 21 2022, 12:17 AM
ehem_freebsd_m5p.com added a reviewer for D30006: xen/intr: adjust xen_intr_handle_upcall() to match interrupt filter: cperciva.
Oct 21 2022, 12:13 AM
ehem_freebsd_m5p.com removed reviewers for D30006: xen/intr: adjust xen_intr_handle_upcall() to match interrupt filter: julien_xen.org, mhorne.
Oct 21 2022, 12:12 AM
ehem_freebsd_m5p.com removed a reviewer for D30909: xen/intr: introduce xen_arch_intr.c: mhorne.
Oct 21 2022, 12:12 AM
ehem_freebsd_m5p.com updated the diff for D30006: xen/intr: adjust xen_intr_handle_upcall() to match interrupt filter.

Updating to ensure this is consistent with the state of my tree. This has been moved earlier, before D30909.

Oct 21 2022, 12:07 AM

Oct 20 2022

ehem_freebsd_m5p.com updated the diff for D37063: sys/conf: add options to test for x86.

Updating per @kevans91 on IRC. I would prefer to have !x86, but !i386 !amd64 is good enough.

Oct 20 2022, 4:41 AM
ehem_freebsd_m5p.com requested review of D37063: sys/conf: add options to test for x86.
Oct 20 2022, 3:25 AM

Oct 15 2022

ehem_freebsd_m5p.com added a comment to D36901: intr/x86: remove intr_bind() from x86.

Nope. This is purely a one-off which happened to show up in front of me when I was looking at something else. I kind of wonder whether intr_bind() should be fixed and turn into an inline static to start encouraging its use. Right now that single use in nexus was the only use and intr_event_bind() is used all over.

Oct 15 2022, 12:14 AM

Oct 14 2022

ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Seems I've run into a benefit of this I hadn't been aiming for. This causes all event counters to be reported. As a result this will report events which aren't hooked up to the platform interrupt subsystem.

Oct 14 2022, 1:33 AM
ehem_freebsd_m5p.com added a comment to D36901: intr/x86: remove intr_bind() from x86.

@mjg aren't you discussing a distinct potential commit here? I don't believe that is how this is supposed to work.

Oct 14 2022, 1:26 AM

Oct 12 2022

ehem_freebsd_m5p.com added a comment to D36901: intr/x86: remove intr_bind() from x86.

Having a cpu_t seems a bit overkill. Nearly everywhere uses u_int which works fine. Just in this instance I spotted something rather out of date. On the other hand if I was asked to review a diff which created a cpu_t, I would accept it if it looked reasonable.

Oct 12 2022, 9:46 PM
ehem_freebsd_m5p.com added inline comments to D32343: physmem: add ram0 pseudo-driver.
Oct 12 2022, 5:21 PM
ehem_freebsd_m5p.com retitled D32866: xen/intr: merge CPU assignment during port binding together from xen/inter: merge CPU assignment during port binding together to xen/intr: merge CPU assignment during port binding together.
Oct 12 2022, 4:09 PM
ehem_freebsd_m5p.com updated the diff for D31355: xen/intr: hand off closing of ports sooner.

Updating to present tree status. Like the others in this sequence, suddenly a distinct approach has a massive impact.

Oct 12 2022, 4:08 PM
ehem_freebsd_m5p.com updated the summary of D32866: xen/intr: merge CPU assignment during port binding together.
Oct 12 2022, 4:06 PM
ehem_freebsd_m5p.com updated the diff for D32866: xen/intr: merge CPU assignment during port binding together.

Updating Phabricator due to discussions elsewhere. I suspect this will get further adjustment.

Oct 12 2022, 4:05 PM
ehem_freebsd_m5p.com retitled D31188: xen/intr: move isrc allocation out of xen_intr_bind_isrc() from xen/intr: switch to passing initial parameters as temporary isrc to xen/intr: move isrc allocation out of xen_intr_bind_isrc().
Oct 12 2022, 4:03 PM
ehem_freebsd_m5p.com updated the diff for D31188: xen/intr: move isrc allocation out of xen_intr_bind_isrc().

Updating. New approach to this situation which may be more likely to be accepted.

Oct 12 2022, 4:02 PM
ehem_freebsd_m5p.com updated the diff for D32343: physmem: add ram0 pseudo-driver.

Two fixups. First, D34691 makes the ram device need to be surrounded by #ifdef _KERNEL. Second, due to @jhb's work removing device classes from DRIVER_MODULE(), this needs the same treatment.

Oct 12 2022, 3:58 PM
ehem_freebsd_m5p.com commandeered D32343: physmem: add ram0 pseudo-driver.

Stealing D32343 from @mhorne since this feature is rather important for the project I'm working on.

Oct 12 2022, 3:53 PM
ehem_freebsd_m5p.com abandoned D33624: xen/intr: add channel closing in case of early setup failure.

Due to coming up with an alternative approach, this appears certain to disappear.

Oct 12 2022, 3:51 PM

Oct 7 2022

ehem_freebsd_m5p.com added a comment to D36901: intr/x86: remove intr_bind() from x86.

Checking turned up a single use of intr_bind(), everywhere else intr_event_bind() was used. At this point the u_char is hazardous, so time to nuke the almost unused function.

Oct 7 2022, 8:43 PM
ehem_freebsd_m5p.com updated the summary of D36901: intr/x86: remove intr_bind() from x86.
Oct 7 2022, 8:40 PM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

And I thought I had responded, but hadn't. Sorry.

Oct 7 2022, 7:36 PM
ehem_freebsd_m5p.com requested review of D36901: intr/x86: remove intr_bind() from x86.
Oct 7 2022, 4:11 AM

Oct 3 2022

ehem_freebsd_m5p.com added a reviewer for D35607: intrng: purge INTR_SOLO: skra.
Oct 3 2022, 10:04 PM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Certainly my primary goal could be accomplished that way, I'm presently very dubious about that being better though. Notably the only two benefits of having intrcnt and intrnames I observe are: it makes the sysctl() interface extremely simple; and it is architecture-independent. What you're suggesting would add roughly the same amount of complexity to sysctl() and placing the counters in intr_event improves architecture-independence.

Oct 3 2022, 9:59 PM
ehem_freebsd_m5p.com updated subscribers of D35607: intrng: purge INTR_SOLO.
In D35607#836545, @mmel wrote:

So if you have time and want to help, please finish it. But unfortunately I can't help you much, due to external conditions I'm only online a few hours a week, usually without the energy to do anything for FreeBSD. And I can't expect it to improve by the end of the year.

That is your word choice. My interpretation: "I can't be bothered to finish this feature, but it has huge potential, so please take care of finishing this feature for me! Oh, I'm going to block your patch since it would interfere with the plans I don't have time to implement myself"
No. That is not how this works. Blocking with that sort of justification is an abuse of this system.

Oct 3 2022, 8:39 PM

Oct 1 2022

ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

This likely needs discussion elsewhere, but I believe the time has come for this. intrnames simply duplicates ie_fullname (notice how all the intrcnt_setname() functions simply copy). Then in turn forces isrc_release_counters() to be broken.

Oct 1 2022, 10:06 PM
ehem_freebsd_m5p.com updated the diff for D35607: intrng: purge INTR_SOLO.

Switching to completely removing one comment.

Oct 1 2022, 9:39 PM
ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Small tweak. Finally managed to break this into a reasonable series. Now need that okay for how this changes things.

Oct 1 2022, 9:29 PM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Note this is out there at https://gitlab.com/ehem/freebsd-src.git the branch "D36628" (https://gitlab.com/ehem/freebsd-src/-/commits/D36628/).

Oct 1 2022, 9:20 PM
ehem_freebsd_m5p.com updated the diff for D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Bringing in the remaining bits.

Oct 1 2022, 9:13 PM
ehem_freebsd_m5p.com reopened D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

I guess this is being done this way.

Oct 1 2022, 9:13 PM

Sep 30 2022

ehem_freebsd_m5p.com added a comment to D35898: intrng: Extract arm/arm64 IPI->PIC glue code.

I'm extremely suspicious how ipi_hardclock()/ipi_preempt() modify td_intr_frame and td_nesting_level on ARM, but not ARM64. Is this actually needed on ARM?

Sep 30 2022, 11:20 PM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Hmm, not quite how I thought this was going to work. I'm wondering what action, if any, I'm supposed to take with this now.

Sep 30 2022, 11:07 PM
ehem_freebsd_m5p.com requested changes to D35898: intrng: Extract arm/arm64 IPI->PIC glue code.

Even though intr_ipi_dispatch() duplicates how the existing implementations act, I must object to having such in INTRNG. Likely intr_ipi_dispatch() should handle the trapframe similar to how intr_event_handle() handles them. Everything else seems fairly minor and could be handled afterwords, but I find intr_ipi_dispatch() unacceptable.

Sep 30 2022, 4:30 AM

Sep 29 2022

ehem_freebsd_m5p.com added a comment to D36609: intr: add interrupt FOREACH macro.

This doesn't do much more than provide some support for D36610. I'm unsure of the approach, but needed something to maintain compatibility with D36610. I won't be surprised by abandoning this if a better approach is suggested.

Sep 29 2022, 1:54 AM
ehem_freebsd_m5p.com added a comment to D35559: intr: rename "machine/intr_machdep.h" headers to "machine/intr.h".

I was under the impression D35559 has the right reviewers, but no action has been observed. This is meant as a precursor to the updated D32504. This can improve several places, but does need someone to okay (and commit to the main tree). Then there are the child commits.

Sep 29 2022, 1:51 AM
ehem_freebsd_m5p.com retitled D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface from vmstat: always print interrupts with non-zero counts to vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.
Sep 29 2022, 1:43 AM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Appears none of the callers of intr_event_create() nor intr_event_destroy() hold locks, therefore sleeping is acceptable. As such switching to a shared-exclusive lock takes care of the concern with the sysctl. Only remaining issue is the watchdog_fire() function and needing an okay for the rather substantial architecture change.

Sep 29 2022, 1:40 AM
ehem_freebsd_m5p.com added a reviewer for D35607: intrng: purge INTR_SOLO: jhb.
Sep 29 2022, 12:58 AM

Sep 28 2022

ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Potential solution to the sysctl issue. Previous round has been tested and confirmed to work (once D36628 is in). This one hasn't reached testing yet. I'm unsure I've got the right reviewers, but fixing intr_isrc_deregister() seems pretty high value.

Sep 28 2022, 6:14 PM

Sep 25 2022

ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Hopefully fusing these together doesn't make your job much harder...

Sep 25 2022, 10:52 PM
ehem_freebsd_m5p.com updated the diff for D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Update with fixes for newly found issues. vmstat wasn't matching the kernel.

Sep 25 2022, 10:39 PM

Sep 23 2022

ehem_freebsd_m5p.com retitled D36610: intr: remove intrcnt/intrnames and move counters to intr_event from intr: remove intrcnt/intrnams and move counters to intr_event to intr: remove intrcnt/intrnames and move counters to intr_event.
Sep 23 2022, 2:11 AM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

The situation with the potential need to sleep during sysctl_old_user() still needs addressing and I need a suggestion.

Sep 23 2022, 2:11 AM
ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Updating to match current tree status. I discovered I hadn't fully done the job for PowerPC, so fix that problem. Things have shown during testing...

Sep 23 2022, 2:01 AM
ehem_freebsd_m5p.com updated the diff for D36609: intr: add interrupt FOREACH macro.

Naturally finding bugs during full build...

Sep 23 2022, 1:56 AM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

I'm not trying to avoid libxo, but I'm not trying to engage it either; I suspect it is similar to Python's string formatting. Don't think about libxo there though, think like the typical C programmer. intrnames[0] == '\0' => empty string, thus blank followed by numbers.

Sep 23 2022, 12:35 AM

Sep 21 2022

ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

There is a fair bit of history in vmstat -i. Some portions really needed updates long ago. I think I've fixed the additional major problems. Big one was it was using an odd algorithm to identify interrupt name length which didn't really work too well.

Sep 21 2022, 2:58 AM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Ugh, type up most of how I wanted to respond and then Phabricator drops it on the floor when one thing needs doing...

Sep 21 2022, 1:23 AM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Based on what I'm looking at, it appears the sysctl interface needs a flag (perhaps CTLFLAG_NOSLEEP?) to indicate the ->oldfunc()/->newfunc() must not sleep. If that is passed those functions would likely return EAGAIN to indicate they need a sleep to continue. In which case the handler needs to either abort, or release locks and indicate the sysctl functions can do their sleep-requiring work.

Sep 21 2022, 1:21 AM
ehem_freebsd_m5p.com retitled D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface from vmstat: make -a print *all* interrupts unconditionally to vmstat: always print interrupts with non-zero counts.
Sep 21 2022, 1:02 AM
ehem_freebsd_m5p.com updated the diff for D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Updating since I've concluded what I was suggesting really is rather better.

Sep 21 2022, 12:49 AM
ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Update to reflect current status. Unfortunately appears vmstat has added assumptions on what the sysctl output will look like and this breaks them. Now to fix those...

Sep 21 2022, 12:47 AM

Sep 20 2022

ehem_freebsd_m5p.com added a comment to D35607: intrng: purge INTR_SOLO.

This appears to have been an idea for a really small system where needing multiple handlers for a single interrupt was rare. Given how I'm unable to find any evidence of this ever being deployed and their presence serves to cause confusion, I suggest they be completely nuked.

Sep 20 2022, 4:44 AM
ehem_freebsd_m5p.com edited reviewers for D35607: intrng: purge INTR_SOLO, added: andrew; removed: markj.
Sep 20 2022, 4:40 AM
ehem_freebsd_m5p.com added a comment to D36609: intr: add interrupt FOREACH macro.

While at it, appears Herald rule H67 has been broken by the configuration change for arc (defaults to rG, not rS).

Sep 20 2022, 4:38 AM
ehem_freebsd_m5p.com updated subscribers of D36609: intr: add interrupt FOREACH macro.
Sep 20 2022, 4:37 AM
ehem_freebsd_m5p.com added a comment to D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.

Trying for random reviewers since I don't have a good idea of who to ask for review/commit for this.

Sep 20 2022, 4:32 AM
ehem_freebsd_m5p.com added reviewers for D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface: markj, manu, kib, andrew.
Sep 20 2022, 4:23 AM
ehem_freebsd_m5p.com requested review of D36628: vmstat: cleanup, better handle interrupts with zero count, fix sysctl interface.
Sep 20 2022, 3:08 AM

Sep 18 2022

ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Looks like the problem is troublesome. At the same time though it looks like the present sysctl interface has issues.

Sep 18 2022, 9:00 PM

Sep 17 2022

ehem_freebsd_m5p.com updated the diff for D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Addressing one minor issue. In other news this implementation has been confirmed to partially work, but it definitely has at least one bug.

Sep 17 2022, 5:59 AM
ehem_freebsd_m5p.com abandoned D32876: xen/intr: move handler removal to release from unbind.

Finally figured out what had been confusing here. This portion is troublesome and I now think another approach to this is rather better.

Sep 17 2022, 2:33 AM
ehem_freebsd_m5p.com abandoned D30634: kern/intr: implement isrc_release_counters().

Obsoleted by several other bits.

Sep 17 2022, 2:31 AM
ehem_freebsd_m5p.com abandoned D30605: xen/intr: remove xen_intr_alloc_and_bind_ipi() from !x86.

Obsoleted by other work.

Sep 17 2022, 2:30 AM
ehem_freebsd_m5p.com abandoned D30600: xen/intr: disable xen_intr_suspend() and xen_intr_resume() on !x86.

Huh, this was still lying around. After a while I figured out how to adjust things work rather better.

Sep 17 2022, 2:29 AM

Sep 16 2022

ehem_freebsd_m5p.com abandoned D28832: arm64: Rename pmap_kenter() to pmap_kenter_arm64().

I still think the renaming is appropriate, but since the crucial spot has been resolved by other means I'll simply drop this.

Sep 16 2022, 11:38 PM
ehem_freebsd_m5p.com added a comment to D32793: sys: declare bit sets unsigned.

Trying for more reviewers since D32793 has simply been sitting out here gathering dust. Seems pretty appropriate as unsigned is by far the more common use.

Sep 16 2022, 11:36 PM
ehem_freebsd_m5p.com added reviewers for D32793: sys: declare bit sets unsigned: se, jhibbits.
Sep 16 2022, 11:34 PM
ehem_freebsd_m5p.com abandoned D35602: intrng: always use atomic increment for interrupt counter.

Abandoning this, D36610 gets the issue of spreading the interrupt counters apart and likely onto separate cache lines.

Sep 16 2022, 11:26 PM
ehem_freebsd_m5p.com abandoned D35608: intrng: isrc_free_irq() don't fail if absent from table.

The real lurking issue is handled by D36610. Abandoning D35608 as it is unlikely to ever see the light of day.

Sep 16 2022, 11:23 PM
ehem_freebsd_m5p.com added a reviewer for D35418: intrng: convert to using resource manager for interrupt numbers: manu.
Sep 16 2022, 9:37 PM
ehem_freebsd_m5p.com added a comment to D35418: intrng: convert to using resource manager for interrupt numbers.

Huh, looking back at this I'm wondering why the mv_ap806 bits were needed.

Sep 16 2022, 9:36 PM
ehem_freebsd_m5p.com removed a reviewer for D36611: intrng: implement use of intr_isrc_deregister(): markj.
Sep 16 2022, 9:13 PM
ehem_freebsd_m5p.com added a reviewer for D36610: intr: remove intrcnt/intrnames and move counters to intr_event: markj.
Sep 16 2022, 9:13 PM
ehem_freebsd_m5p.com added a reviewer for D36611: intrng: implement use of intr_isrc_deregister(): markj.
Sep 16 2022, 9:12 PM
ehem_freebsd_m5p.com added a reviewer for D35607: intrng: purge INTR_SOLO: manu.
Sep 16 2022, 9:12 PM
ehem_freebsd_m5p.com updated the diff for D36609: intr: add interrupt FOREACH macro.

Ick, dropping the idea of a common maximum possible interrupt number variable. Had seemed worthwhile, but now I'm unsure.

Sep 16 2022, 8:09 PM
ehem_freebsd_m5p.com updated the diff for D32504: kern/intr: remove "irq" from kernel event API.

Fixing a distinct goof.

Sep 16 2022, 7:51 PM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Also note, I've confirmed this builds, I'm less than certain it actually works yet.

Sep 16 2022, 7:45 PM
ehem_freebsd_m5p.com added a comment to D36609: intr: add interrupt FOREACH macro.

Hmm, then I spot the error...

Sep 16 2022, 7:43 PM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

Guess I should also note how sys/dev/hyperv/vmbus/vmbus.c simply had to drop an entry on the floor if its initialization failed, whereas with this that can be cleaned up.

Sep 16 2022, 7:41 PM
ehem_freebsd_m5p.com requested review of D36611: intrng: implement use of intr_isrc_deregister().
Sep 16 2022, 7:36 PM
ehem_freebsd_m5p.com added a comment to D36610: intr: remove intrcnt/intrnames and move counters to intr_event.

The implementation for sys/kern/kern_clock.c is based on D36609. I have a suspicion that idea may be shot down, but it was the approach which came to mind (since necessary functionality almost existed). The approach for sys/kern/kern_intr.c seems better for that purpose, but that requires a different bit of functionality not presently provided (allowing the outside to enumerate the events).

Sep 16 2022, 7:31 PM
ehem_freebsd_m5p.com requested review of D36610: intr: remove intrcnt/intrnames and move counters to intr_event.
Sep 16 2022, 7:23 PM
ehem_freebsd_m5p.com requested review of D36609: intr: add interrupt FOREACH macro.
Sep 16 2022, 7:17 PM

Sep 9 2022

ehem_freebsd_m5p.com added inline comments to D35417: kern/intr: allow for event allocation outside kern_intr.c.
Sep 9 2022, 11:32 PM