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
