X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~yarrgweb/git?p=ypp-sc-tools.main.git;a=blobdiff_plain;f=pctb%2Fpages.c;h=1a9970162ce10c8fd667208b5476f1ca9cde44f8;hp=3effde51ce9e7f1bcad338c2f4584b9a4dc73ac1;hb=c66cfd9d90974888907605f6d9888bcadd5fa433;hpb=bbcbb52822264bc83cd2c10bc5c2408483b8acb7

diff --git a/pctb/pages.c b/pctb/pages.c
index 3effde5..1a99701 100644
--- a/pctb/pages.c
+++ b/pctb/pages.c
@@ -254,6 +254,7 @@ static void snapshot(Snapshot **output) {
     memcpy(im_free->data, shmim->data, dsz);
     rtimestamp(&begin, "mmalloc/memcpy");
   } else {
+    rtimestamp(&begin, "XGetImage before");
     xassert( im_use= im_free=
 	     XGetImage(disp,id, 0,0, wwidth,wheight, AllPlanes, ZPixmap) );
     rtimestamp(&begin, "XGetImage");
@@ -320,6 +321,7 @@ static void wait_for_stability(Snapshot **output,
   va_start(al,fmt);
 
   Snapshot *last=0;
+  int nidentical=0;
   /* waits longer if we're going to return an image identical to previously
    * if previously==0, all images are considered identical to it */
 
@@ -332,33 +334,33 @@ static void wait_for_stability(Snapshot **output,
 	  last_input, previously, doing);
 
   for (;;) {
-    double at_snapshot= timestamp();
-    double need_sleep= min_update_allowance - (at_snapshot - last_input);
-    if (need_sleep > 0) { delay(need_sleep); continue; }
-
+    usleep(250000);
     snapshot(output);
 
-    if (!with_keypress &&
-	!(previously && identical(*output,previously))) {
-      debugf("PAGING  wait_for_stability  simple\n");
-      break;
-    }
-
-    if (last && identical(*output,last)) {
-      debugf("PAGING  wait_for_stability  stabilised\n");
-      break;
+    if (!last) {
+      debugf("PAGING  wait_for_stability first...\n");
+      last=*output; *output=0;
+    } else if (!identical(*output,last)) {
+      debugf("PAGING  wait_for_stability changed...\n");
+      free(last); last=*output; *output=0;
+      nidentical=0;
+    } else {
+      nidentical++;
+      int threshold=
+	!previously ? 3 :
+	identical(*output,previously) ? 3
+	: 1;
+      debugf("PAGING  wait_for_stability nidentical=%d threshold=%d\n",
+	      nidentical, threshold);
+      if (nidentical >= threshold)
+	break;
     }
-    
     progress_spinner("%s",doing);
 
-    debugf("PAGING  wait_for_stability  retry\n");
-
-    free_snapshot(&last); last=*output; *output=0;
-
     if (with_keypress)
       with_keypress();
 
-    delay(0.5);
+    usleep(250000);
   }
 
   free_snapshot(&last);
@@ -402,8 +404,10 @@ static void raise_and_get_details(void) {
 
   check_client_window_all_on_screen();
 
-  int shm= XShmQueryExtension(disp);
-  debugf("PAGING shm=%d\n",shm);
+  Bool shmpixmaps=0;
+  int major=0,minor=0;
+  int shm= XShmQueryVersion(disp, &major,&minor,&shmpixmaps);
+  debugf("PAGING shm=%d %d.%d pixmaps=%d\n",shm,major,minor,shmpixmaps);
   if (shm) {
     xassert( shmim= XShmCreateImage(disp, attr.visual, attr.depth, ZPixmap,
 				    0,&shminfo, wwidth,wheight) );