rrcrtc: brackets are hard, lets go shopping.
Slaving two outputs on a secondary GPU to a primary GPU testing picked this up, in that we'd try to resize to the totally the wrong thing, then as usual segfault in the rotation code. Signed-off-by: Dave Airlie <airlied@redhat.com> Reviewed-by: Aaron Plattner <aplattner@nvidia.com>
This commit is contained in:
		
							parent
							
								
									6dd86f3cba
								
							
						
					
					
						commit
						01e18af17f
					
				| 
						 | 
				
			
			@ -476,7 +476,7 @@ rrCheckPixmapBounding(ScreenPtr pScreen,
 | 
			
		|||
 | 
			
		||||
    xorg_list_for_each_entry(slave, &pScreen->output_slave_list, output_head) {
 | 
			
		||||
        rrScrPrivPtr    slave_priv = rrGetScrPriv(slave);
 | 
			
		||||
        for (c = 0; c < slave_priv->numCrtcs; c++)
 | 
			
		||||
        for (c = 0; c < slave_priv->numCrtcs; c++) {
 | 
			
		||||
            if (slave_priv->crtcs[c] == rr_crtc) {
 | 
			
		||||
                newbox.x1 = x;
 | 
			
		||||
                newbox.x2 = x + w;
 | 
			
		||||
| 
						 | 
				
			
			@ -491,8 +491,9 @@ rrCheckPixmapBounding(ScreenPtr pScreen,
 | 
			
		|||
                newbox.y1 = slave_priv->crtcs[c]->y;
 | 
			
		||||
                newbox.y2 = slave_priv->crtcs[c]->y + slave_priv->crtcs[c]->mode->mode.height;
 | 
			
		||||
            }
 | 
			
		||||
        RegionInit(&new_crtc_region, &newbox, 1);
 | 
			
		||||
        RegionUnion(&total_region, &total_region, &new_crtc_region);
 | 
			
		||||
            RegionInit(&new_crtc_region, &newbox, 1);
 | 
			
		||||
            RegionUnion(&total_region, &total_region, &new_crtc_region);
 | 
			
		||||
        }
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    newsize = RegionExtents(&total_region);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue