summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarcus Glocker <mglocker@cvs.openbsd.org>2009-09-19 20:49:54 +0000
committerMarcus Glocker <mglocker@cvs.openbsd.org>2009-09-19 20:49:54 +0000
commite8be2c58e91023fe36720327dd981408b81e4786 (patch)
tree59eaf4754ef13d4d651965efe69f46674d4c7aab
parentdc3a8ded80720d43840f949cc2140741d4bf734c (diff)
If UDLIO_DAMAGE fails to queue the damage drawing request, undo the
operation and tell the X11 driver to retry later (same as for rasops).
-rw-r--r--sys/dev/usb/udl.c21
-rw-r--r--sys/dev/usb/udl.h5
2 files changed, 18 insertions, 8 deletions
diff --git a/sys/dev/usb/udl.c b/sys/dev/usb/udl.c
index 6960caf7234..a5afd8f983e 100644
--- a/sys/dev/usb/udl.c
+++ b/sys/dev/usb/udl.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: udl.c,v 1.42 2009/09/19 11:54:16 mglocker Exp $ */
+/* $OpenBSD: udl.c,v 1.43 2009/09/19 20:49:53 mglocker Exp $ */
/*
* Copyright (c) 2009 Marcus Glocker <mglocker@openbsd.org>
@@ -463,8 +463,9 @@ udl_ioctl(void *v, u_long cmd, caddr_t data, int flag, struct proc *p)
d = (struct udl_ioctl_damage *)data;
r = udl_damage(sc, sc->sc_fbmem, d->x1, d->x2, d->y1, d->y2);
if (r != 0) {
- /* XXX we need to inform X11 when we failed to draw */
- printf("%s: %s: damage draw failed!\n", DN(sc), FUNC);
+ d->status = UDLIO_STATUS_FAILED;
+ } else {
+ d->status = UDLIO_STATUS_OK;
}
break;
default:
@@ -934,6 +935,9 @@ udl_damage(struct udl_softc *sc, uint8_t *image,
{
int r;
int x, y, width, height;
+ usbd_status error;
+
+ udl_cmd_get_offset(sc);
x = x1;
y = y1;
@@ -942,11 +946,14 @@ udl_damage(struct udl_softc *sc, uint8_t *image,
r = udl_draw_image(sc, image, x, y, width, height);
if (r != 0)
- return (r);
+ goto fail;
- r = udl_cmd_send_async(sc);
- if (r != 0)
- return (r);
+ error = udl_cmd_send_async(sc);
+ if (error != USBD_NORMAL_COMPLETION) {
+fail:
+ udl_cmd_set_offset(sc);
+ return (EAGAIN);
+ }
return (0);
}
diff --git a/sys/dev/usb/udl.h b/sys/dev/usb/udl.h
index 32f8fa1d036..28c934e727c 100644
--- a/sys/dev/usb/udl.h
+++ b/sys/dev/usb/udl.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: udl.h,v 1.9 2009/09/19 11:54:16 mglocker Exp $ */
+/* $OpenBSD: udl.h,v 1.10 2009/09/19 20:49:53 mglocker Exp $ */
/*
* Copyright (c) 2009 Marcus Glocker <mglocker@openbsd.org>
@@ -108,6 +108,9 @@ struct udl_ioctl_damage {
int x2;
int y1;
int y2;
+#define UDLIO_STATUS_OK 0
+#define UDLIO_STATUS_FAILED 1
+ int status;
};
#define UDLIO_DAMAGE _IOWR('W', 0, struct udl_ioctl_damage)