On 02/27/14 11:59, Kohji Okuno wrote:
+ sc = kbd->kb_dev->si_drv1;
+ knlist_clear(&sc->gkb_rsel.si_note, 0);
+ knlist_destroy(&sc->gkb_rsel.si_note);
+
free(kbd->kb_dev->si_drv1, M_DEVBUF);
destroy_dev(kbd->kb_dev);
Hi,
You should put the "knlist_destroy()" af
Hi John-Mark,
Thank you for your comment.
I added knote_clear() and knote_destroy() in kbd_detach(). I attached
patch, again. But, maybe this patch can not resolve all cases you
pointed.
Regards,
Kohji Okuno
> Kohji Okuno wrote this message on Thu, Feb 27, 2014 at 14:24 +0900:
>> I tried to add
Kohji Okuno wrote this message on Thu, Feb 27, 2014 at 14:24 +0900:
> I tried to add kqueue I/F to kbd.c. I attached patch.
> What do you think about my patch?
So, knlist_destroy is missing in this patch too..
It also needs some style(9) loving in that some blank lines are missing
and there are s
Hi,
I tried to add kqueue I/F to kbd.c. I attached patch.
What do you think about my patch?
Best regards,
Kohji Okuno
diff --git a/sys/dev/kbd/kbd.c b/sys/dev/kbd/kbd.c
index 8036762..df000ab 100644
--- a/sys/dev/kbd/kbd.c
+++ b/sys/dev/kbd/kbd.c
@@ -59,6 +59,9 @@ typedef struct genkbd_softc {
Hi,
I tried add kqueue I/F to kbd.c. I attached my patch.
What do you think about my patch?
Best regards,
Kohji Okuno
diff --git a/sys/dev/kbd/kbd.c b/sys/dev/kbd/kbd.c
index 8036762..df000ab 100644
--- a/sys/dev/kbd/kbd.c
+++ b/sys/dev/kbd/kbd.c
@@ -59,6 +59,9 @@ typedef struct genkbd_softc {
