Re: [PATCH 2/3] drm/radeon: mask out WC from BO on unsupported arches

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 30.01.2016 14:59, Oded Gabbay wrote:
> Signed-off-by: Oded Gabbay <oded.gabbay@xxxxxxxxx>
> ---
>  drivers/gpu/drm/radeon/radeon_object.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
> index 84d4563..255faf6 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.c
> +++ b/drivers/gpu/drm/radeon/radeon_object.c
> @@ -33,6 +33,7 @@
>  #include <linux/slab.h>
>  #include <drm/drmP.h>
>  #include <drm/radeon_drm.h>
> +#include <drm/drm_cache.h>
>  #include "radeon.h"
>  #include "radeon_trace.h"
>  
> @@ -245,6 +246,13 @@ int radeon_bo_create(struct radeon_device *rdev,
>  		DRM_INFO_ONCE("Please enable CONFIG_MTRR and CONFIG_X86_PAT for "
>  			      "better performance thanks to write-combining\n");
>  	bo->flags &= ~(RADEON_GEM_GTT_WC | RADEON_GEM_GTT_UC);
> +#else
> +	/*
> +	 * For architectures that don't support WC memory,
> +	 * mask out the WC flag from the BO
> +	 */
> +	if (!drm_arch_can_wc_memory())
> +		bo->flags &= ~RADEON_GEM_GTT_WC;
>  #endif

For consistency with existing multi-line comments, please start the
comment on the opening line:

	/* For architectures that don't support WC memory,
	 * mask out the WC flag from the BO
	 */

With that fixed (in patch 3 as well), the series is

Reviewed-by: Michel Dänzer <michel.daenzer@xxxxxxx>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux