Skip to content

Commit e181523

Browse files
committed
minor changes related to logging
1 parent af8fff7 commit e181523

File tree

4 files changed

+14
-4
lines changed

4 files changed

+14
-4
lines changed

internal/usbgadget/hid_keyboard.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,8 @@ func (u *UsbGadget) GetKeysDownState() KeysDownState {
146146
}
147147

148148
func (u *UsbGadget) updateKeyDownState(state KeysDownState) {
149+
u.log.Trace().Interface("old", u.keysDownState).Interface("new", state).Msg("acquiring keyboardStateLock for updateKeyDownState")
150+
149151
u.keyboardStateLock.Lock()
150152
defer u.keyboardStateLock.Unlock()
151153

@@ -242,7 +244,7 @@ func (u *UsbGadget) keyboardWriteHidFile(modifier byte, keys []byte) error {
242244
return err
243245
}
244246

245-
_, err := writeWithTimeout(u.keyboardHidFile, append([]byte{modifier, 0x00}, keys[:hidKeyBufferSize]...))
247+
_, err := u.writeWithTimeout(u.keyboardHidFile, append([]byte{modifier, 0x00}, keys[:hidKeyBufferSize]...))
246248
if err != nil {
247249
u.logWithSuppression("keyboardWriteHidFile", 100, u.log, err, "failed to write to hidg0")
248250
u.keyboardHidFile.Close()

internal/usbgadget/hid_mouse_absolute.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ func (u *UsbGadget) absMouseWriteHidFile(data []byte) error {
7474
}
7575
}
7676

77-
_, err := writeWithTimeout(u.absMouseHidFile, data)
77+
_, err := u.writeWithTimeout(u.absMouseHidFile, data)
7878
if err != nil {
7979
u.logWithSuppression("absMouseWriteHidFile", 100, u.log, err, "failed to write to hidg1")
8080
u.absMouseHidFile.Close()

internal/usbgadget/hid_mouse_relative.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ func (u *UsbGadget) relMouseWriteHidFile(data []byte) error {
6464
}
6565
}
6666

67-
_, err := writeWithTimeout(u.relMouseHidFile, data)
67+
_, err := u.writeWithTimeout(u.relMouseHidFile, data)
6868
if err != nil {
6969
u.logWithSuppression("relMouseWriteHidFile", 100, u.log, err, "failed to write to hidg2")
7070
u.relMouseHidFile.Close()

internal/usbgadget/utils.go

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ func compareFileContent(oldContent []byte, newContent []byte, looserMatch bool)
110110
return false
111111
}
112112

113-
func writeWithTimeout(file *os.File, data []byte) (n int, err error) {
113+
func (u *UsbGadget) writeWithTimeout(file *os.File, data []byte) (n int, err error) {
114114
if err := file.SetWriteDeadline(time.Now().Add(hidWriteTimeout)); err != nil {
115115
return -1, err
116116
}
@@ -121,6 +121,14 @@ func writeWithTimeout(file *os.File, data []byte) (n int, err error) {
121121
}
122122

123123
if errors.Is(err, os.ErrDeadlineExceeded) {
124+
u.logWithSuppression(
125+
fmt.Sprintf("writeWithTimeout_%s", file.Name()),
126+
1000,
127+
u.log,
128+
err,
129+
"write timed out: %s",
130+
file.Name(),
131+
)
124132
err = nil
125133
}
126134

0 commit comments

Comments
 (0)