https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238138

Mark Johnston <ma...@freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|b...@freebsd.org            |ma...@freebsd.org

--- Comment #4 from Mark Johnston <ma...@freebsd.org> ---
(In reply to Muhammad Kaisar Arkhan from comment #3)
Thanks.  I applied the patches to my tree.  A couple more comments:
- Please use tabs for indentation next time.  The style(9) man page documents
the kernel coding style.  Admittedly, some of the ACPI code does not follow it
very well.
- Why do you check for a running devd?  Other ACPI subsystems don't bother. 
The event message is non-blocking and we will simply free the message if devd
isn't running.  We save some unnecessary work by checking first, but I'd rather
be consistent with the rest of the kernel unless there's a good reason here.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

Reply via email to