[Intel-gfx] [PATCH] drm/i915: Use hash tables for the command parser

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Thu May 8 11:56:05 CEST 2014


Hi Brad,

On 04/28/2014 04:22 PM, bradley.d.volkin at intel.com wrote:
[snip]
> -	BUG_ON(!validate_cmds_sorted(ring));
> +	BUG_ON(!validate_cmds_sorted(ring, cmd_tables, cmd_table_count));
>   	BUG_ON(!validate_regs_sorted(ring));
> +
> +	BUG_ON(init_hash_table(ring, cmd_tables, cmd_table_count));

Is a BUG_ON a bit harsh since the above fails only on ENOMEM condition?

If the concern is not allowing any command execution if parser setup has 
failed, it would be nicer to the system as whole to just keep rejecting 
everything, but let the rest of the kernel live to enable debug or whatever?

I know it won't happen almost ever so it's a minor point really. I just 
dislike actively hosing the whole system if it is avoidable.

Regards,

Tvrtko



More information about the Intel-gfx mailing list