On Thu, 28 May 2009 21:44:05 +0400, Alexander Beregalov <[email protected]> 
wrote:

> >> kernel is 2.6.30-rc6-00065-g3633832
> >>
> >> It worked sometime ago, I do not remember,
> >> perhaps it was 2.6.29.
> >
> > Can you verify it?

> It works with 2.6.29 indeed.

Thanks. Just to make sure that it's the culprit, can you back out
the patch for the usblp from the 2.6.30-rc6-00065-g3633832, whatever
that is? We only had one since 2.6.29 (attached).

Also, I'd like to see a usbmon trace. I don't want to create more
quirks, so perhaps we can work out some kind of compromise that lets
BJC-3000 work with the general code. Failing that, we'll add a quirk
and that would hopefuly restore your case (HP CP1215, I take it).

-- Pete

--- linux-2.6.29/drivers/usb/class/usblp.c      2009-05-28 12:07:06.000000000 
-0600
+++ linux-2.6/drivers/usb/class/usblp.c 2009-04-05 11:27:00.000000000 -0600
@@ -880,16 +880,19 @@ static int usblp_wwait(struct usblp *usb
                if (rc <= 0)
                        break;
 
-               if (usblp->flags & LP_ABORT) {
-                       if (schedule_timeout(msecs_to_jiffies(5000)) == 0) {
+               if (schedule_timeout(msecs_to_jiffies(1500)) == 0) {
+                       if (usblp->flags & LP_ABORT) {
                                err = usblp_check_status(usblp, err);
                                if (err == 1) { /* Paper out */
                                        rc = -ENOSPC;
                                        break;
                                }
+                       } else {
+                               /* Prod the printer, Gentoo#251237. */
+                               mutex_lock(&usblp->mut);
+                               usblp_read_status(usblp, usblp->statusbuf);
+                               mutex_unlock(&usblp->mut);
                        }
-               } else {
-                       schedule();
                }
        }
        set_current_state(TASK_RUNNING);
--
To unsubscribe from this list: send the line "unsubscribe kernel-testers" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to