Re: ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Interface to get GFX shmem usage stats per process

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

 



On Thu, 22 Mar 2018, Patchwork <patchwork@xxxxxxxxxxxxxxxxxxxxxx> wrote:
> == Series Details ==
>
> Series: drm/i915: Interface to get GFX shmem usage stats per process
> URL   : https://patchwork.freedesktop.org/series/40464/
> State : warning
>
> == Summary ==
>
> $ dim checkpatch origin/drm-tip

I don't mind the foo == NULL comparisons leading to warnings
CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!foo"

But everything else here is valid, and should be fixed before submitting
this for real.

BR,
Jani.


> a2507d62bb4c drm/i915: Sysfs interface to get GFX shmem usage stats per process
> -:228: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
> #228: FILE: drivers/gpu/drm/i915/i915_gem.c:6118:
> +	if (res > 0 && buffer[res-1] != '\0' && len < PAGE_SIZE)
>  	                         ^
>
> -:229: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
> #229: FILE: drivers/gpu/drm/i915/i915_gem.c:6119:
> +		buffer[res-1] = '\0';
>  		          ^
>
> -:275: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!entry"
> #275: FILE: drivers/gpu/drm/i915/i915_gem.c:6165:
> +		if (entry == NULL) {
>
> -:315: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #315: FILE: drivers/gpu/drm/i915/i915_gem.c:6205:
> +		DRM_DEBUG("Couldn't find matching tgid %d for obj %p\n",
> +				current_tgid, obj);
>
> -:329: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #329: FILE: drivers/gpu/drm/i915/i915_gem.c:6219:
> +static int i915_obj_find_insert_in_hash(struct drm_i915_gem_object *obj,
> +				struct pid_stat_entry *pid_entry,
>
> -:336: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #336: FILE: drivers/gpu/drm/i915/i915_gem.c:6226:
> +	ret = drm_ht_find_item(&pid_entry->namelist,
> +				(unsigned long)&obj->base, &hash_item);
>
> -:338: CHECK:BRACES: braces {} should be used on all arms of this statement
> #338: FILE: drivers/gpu/drm/i915/i915_gem.c:6228:
> +	if (ret) {
> [...]
> +	} else
> [...]
>
> -:341: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!entry"
> #341: FILE: drivers/gpu/drm/i915/i915_gem.c:6231:
> +		if (entry == NULL) {
>
> -:350: CHECK:BRACES: Unbalanced braces around else statement
> #350: FILE: drivers/gpu/drm/i915/i915_gem.c:6240:
> +	} else
>
> -:357: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #357: FILE: drivers/gpu/drm/i915/i915_gem.c:6247:
> +static int i915_obj_shared_count(struct drm_i915_gem_object *obj,
> +				struct pid_stat_entry *pid_entry,
>
> -:371: CHECK:BRACES: braces {} should be used on all arms of this statement
> #371: FILE: drivers/gpu/drm/i915/i915_gem.c:6261:
> +	if (!obj->base.name && !obj->base.dma_buf)
> [...]
> +	else if (obj->base.import_attach) {
> [...]
>
> -:429: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #429: FILE: drivers/gpu/drm/i915/i915_gem.c:6319:
> +			i915_obj_get_shmem_pages_alloced(obj)*PAGE_SIZE;
>  			                                     ^
>
> -:439: CHECK:BRACES: braces {} should be used on all arms of this statement
> #439: FILE: drivers/gpu/drm/i915/i915_gem.c:6329:
> +			if (obj_shared_count > 1) {
> [...]
> +			} else
> [...]
>
> -:443: CHECK:BRACES: Unbalanced braces around else statement
> #443: FILE: drivers/gpu/drm/i915/i915_gem.c:6333:
> +			} else
>
> -:480: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!new_entry"
> #480: FILE: drivers/gpu/drm/i915/i915_gem.c:6370:
> +			if (new_entry == NULL) {
>
> -:490: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #490: FILE: drivers/gpu/drm/i915/i915_gem.c:6380:
> +			ret = drm_ht_create(&new_entry->namelist,
> +					DRM_DEBUG_MAGIC_HASH_ORDER);
>
> -:519: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #519: FILE: drivers/gpu/drm/i915/i915_gem.c:6409:
> +
> +	}
>
> -:541: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #541: FILE: drivers/gpu/drm/i915/i915_gem.c:6431:
> +	err_puts(m,
> +		"\n\n  pid   Total  Shared  Priv   Purgeable  Alloced  SharedPHYsize   SharedPHYprop    PrivPHYsize   PurgeablePHYsize   process\n");
>
> -:561: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #561: FILE: drivers/gpu/drm/i915/i915_gem.c:6451:
> +		list_for_each_entry_safe(new_pid_entry, new_temp_entry,
> +			&sorted_pid_stats, head) {
>
> -:567: WARNING:MULTILINE_DEREFERENCE: Avoid multiple line dereference - prefer 'new_pid_entry->stats.phys_space_shared_proportion'
> #567: FILE: drivers/gpu/drm/i915/i915_gem.c:6457:
> +				new_pid_entry->
> +				stats.phys_space_shared_proportion +
>
> -:572: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #572: FILE: drivers/gpu/drm/i915/i915_gem.c:6462:
> +				list_add_tail(&pid_entry->head,
> +					&new_pid_entry->head);
>
> -:576: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #576: FILE: drivers/gpu/drm/i915/i915_gem.c:6466:
> +			if (list_is_last(&new_pid_entry->head,
> +				&sorted_pid_stats)) {
>
> -:579: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #579: FILE: drivers/gpu/drm/i915/i915_gem.c:6469:
> +				list_add_tail(&pid_entry->head,
> +						&sorted_pid_stats);
>
> -:585: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #585: FILE: drivers/gpu/drm/i915/i915_gem.c:6475:
> +	list_for_each_entry_safe(pid_entry, temp_entry,
> +				&sorted_pid_stats, head) {
>
> -:589: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #589: FILE: drivers/gpu/drm/i915/i915_gem.c:6479:
> +		err_printf(m,
> +			"%5d %6d %6d %6d %9d %8d %14zdK %14zdK %14zdK  %14zdK     %s",
>
> -:596: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #596: FILE: drivers/gpu/drm/i915/i915_gem.c:6486:
> +			   pid_entry->stats.phys_space_allocated_shared/1024,
>  			                                               ^
>
> -:597: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #597: FILE: drivers/gpu/drm/i915/i915_gem.c:6487:
> +			   pid_entry->stats.phys_space_shared_proportion/1024,
>  			                                                ^
>
> -:598: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #598: FILE: drivers/gpu/drm/i915/i915_gem.c:6488:
> +			   pid_entry->stats.phys_space_allocated_priv/1024,
>  			                                             ^
>
> -:599: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #599: FILE: drivers/gpu/drm/i915/i915_gem.c:6489:
> +			   pid_entry->stats.phys_space_purgeable/1024,
>  			                                        ^
>
> -:602: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!task"
> #602: FILE: drivers/gpu/drm/i915/i915_gem.c:6492:
> +		if (task == NULL)
>
> -:608: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #608: FILE: drivers/gpu/drm/i915/i915_gem.c:6498:
> +			pid_entry->stats.phys_space_shared_proportion/1024;
>  			                                             ^
>
> -:610: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #610: FILE: drivers/gpu/drm/i915/i915_gem.c:6500:
> +			pid_entry->stats.phys_space_allocated_priv/1024;
>  			                                          ^
>
> -:614: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #614: FILE: drivers/gpu/drm/i915/i915_gem.c:6504:
> +		list_for_each_entry_safe(entry, next,
> +					&pid_entry->namefree, head) {
>
> -:617: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #617: FILE: drivers/gpu/drm/i915/i915_gem.c:6507:
> +			drm_ht_remove_item(&pid_entry->namelist,
> +					&entry->hash_item);
>
> -:628: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #628: FILE: drivers/gpu/drm/i915/i915_gem.c:6518:
> +	err_puts(m,
> +		"\t\t\t\t\t\t\t\t--------------\t-------------\t--------\n");
>
> -:630: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #630: FILE: drivers/gpu/drm/i915/i915_gem.c:6520:
> +	err_printf(m,
> +		"\t\t\t\t\t\t\t\t%13zdK\t%12zdK\tTotal\n",
>
> -:754: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
> #754: FILE: drivers/gpu/drm/i915/i915_sysfs.c:555:
>  }
> +static ssize_t i915_gem_clients_state_read(struct file *filp,
>
> -:755: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #755: FILE: drivers/gpu/drm/i915/i915_sysfs.c:556:
> +static ssize_t i915_gem_clients_state_read(struct file *filp,
> +				struct kobject *memtrack_kobj,
>
> -:791: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
> #791: FILE: drivers/gpu/drm/i915/i915_sysfs.c:593:
>  };
> +static struct bin_attribute i915_gem_client_state_attr = {
>
> -:793: WARNING:SYMBOLIC_PERMS: Symbolic permissions 'S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH' are not preferred. Consider using octal permissions '0644'.
> #793: FILE: drivers/gpu/drm/i915/i915_sysfs.c:595:
> +	.attr.mode = S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH,
>
> -:819: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #819: FILE: drivers/gpu/drm/i915/i915_sysfs.c:621:
> +		ret = kobject_init_and_add(
>
> -:823: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #823: FILE: drivers/gpu/drm/i915/i915_sysfs.c:625:
> +			DRM_ERROR(
>
> -:829: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #829: FILE: drivers/gpu/drm/i915/i915_sysfs.c:631:
> +			ret = sysfs_create_bin_file(&dev_priv->memtrack_kobj,
> +					    &i915_gem_client_state_attr);
>
> -:831: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #831: FILE: drivers/gpu/drm/i915/i915_sysfs.c:633:
> +				DRM_ERROR(
>
> -:847: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #847: FILE: drivers/gpu/drm/i915/i915_sysfs.c:718:
> +		sysfs_remove_bin_file(&dev_priv->memtrack_kobj,
> +					&i915_gem_client_state_attr);
>
> total: 0 errors, 2 warnings, 43 checks, 766 lines checked
> 27bfa55dd165 drm/i915: Calculate total shmem GFX memory consumed
> -:84: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'obj->has_backing_pages == 1'
> #84: FILE: drivers/gpu/drm/i915/i915_gem.c:4884:
> +		if (!obj->stolen && (obj->has_backing_pages == 1)) {
>
> -:126: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #126: FILE: drivers/gpu/drm/i915/i915_gem.c:6554:
> +	err_printf(m, "\nTotal used GFX Shmem Physical space %8zdK\n",
> +		  dev_priv->mm.phys_mem_total/1024);
>
> -:126: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #126: FILE: drivers/gpu/drm/i915/i915_gem.c:6554:
> +		  dev_priv->mm.phys_mem_total/1024);
>  		                             ^
>
> total: 0 errors, 0 warnings, 3 checks, 101 lines checked
> 0d8d278defab drm/i915: Sysfs interface to get detailed GFX buffer info per process
> -:167: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!client_pid"
> #167: FILE: drivers/gpu/drm/i915/i915_gem.c:3689:
> +		if (client_pid == NULL) {
>
> -:175: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #175: FILE: drivers/gpu/drm/i915/i915_gem.c:3697:
> +		ret = i915_gem_create_sysfs_file_entry(&i915->drm,
> +						 &client_pid->obj_attr);
>
> -:187: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
> #187: FILE: drivers/gpu/drm/i915/i915_gem.c:3709:
> +
> +	}
>
> -:201: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'client_pid->tgid == tgid'
> #201: FILE: drivers/gpu/drm/i915/i915_gem.c:3723:
> +		if ((client_pid->tgid == tgid) &&
> +		    (--client_pid->open_obj_count == 0)) {
>
> -:249: CHECK:LINE_SPACING: Please don't use multiple blank lines
> #249: FILE: drivers/gpu/drm/i915/i915_gem.c:3790:
>  
> +
>
> -:322: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #322: FILE: drivers/gpu/drm/i915/i915_gem.c:6436:
> +		nr_bytes = i915_obj_get_shmem_pages_alloced(obj)*PAGE_SIZE;
>  		                                                ^
>
> -:324: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'obj->mm.madv != __I915_MADV_PURGED'
> #324: FILE: drivers/gpu/drm/i915/i915_gem.c:6438:
> +	if (!discard && !obj->stolen &&
> +			(obj->mm.madv != __I915_MADV_PURGED) &&
> +			(nr_bytes != 0)) {
>
> -:324: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'nr_bytes != 0'
> #324: FILE: drivers/gpu/drm/i915/i915_gem.c:6438:
> +	if (!discard && !obj->stolen &&
> +			(obj->mm.madv != __I915_MADV_PURGED) &&
> +			(nr_bytes != 0)) {
>
> -:325: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #325: FILE: drivers/gpu/drm/i915/i915_gem.c:6439:
> +	if (!discard && !obj->stolen &&
> +			(obj->mm.madv != __I915_MADV_PURGED) &&
>
> -:335: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #335: FILE: drivers/gpu/drm/i915/i915_gem.c:6449:
> +	i915_error_printf(m,
> +		"%p: %7zdK  %10zdK     %c      %c     %s      %s      %s     %s     %s",
>
> -:368: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #368: FILE: drivers/gpu/drm/i915/i915_gem.c:6482:
> +		i915_error_printf(m, " (%d: %d)",
> +			   pid_info_entry->tgid,
>
> -:389: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
> #389: FILE: drivers/gpu/drm/i915/i915_gem.c:6503:
> +	list_for_each_entry(pid_info, &obj->pid_info, head) {
> +
>
> -:435: CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements
> #435: FILE: drivers/gpu/drm/i915/i915_gem.c:6792:
> +	i915_error_printf(m, "%"SPACES_STR(NUM_SPACES)"s\n", " ");
>
> -:438: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #438: FILE: drivers/gpu/drm/i915/i915_gem.c:6795:
> +	err_puts(m,
> +		"\n Obj Identifier   Obj-Size Resident-Size Pin Tiling Dirty Shared Vmap Stolen Mappable  AllocState Global/PP  GttOffset (PID: handle count)\n");
>
> -:448: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #448: FILE: drivers/gpu/drm/i915/i915_gem.c:6805:
> +		ret = idr_for_each(&file->object_idr,
> +				&i915_drm_gem_obj_info, &obj_stat_buf);
>
> -:460: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #460: FILE: drivers/gpu/drm/i915/i915_gem.c:6817:
> +	i915_error_printf(m, "%5d %8zdK ", pid_nr(tgid),
> +			(pid_entry.stats.phys_space_shared_proportion +
>
> -:461: CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
> #461: FILE: drivers/gpu/drm/i915/i915_gem.c:6818:
> +			 pid_entry.stats.phys_space_allocated_priv)/1024);
>  			                                           ^
>
> -:468: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #468: FILE: drivers/gpu/drm/i915/i915_gem.c:6825:
> +		drm_ht_remove_item(&pid_entry.namelist,
> +				&entry->hash_item);
>
> -:489: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #489: FILE: drivers/gpu/drm/i915/i915_gem.c:6863:
> +int i915_gem_get_obj_info(struct drm_i915_error_state_buf *m,
> +			struct drm_device *dev, struct pid *tgid)
>
> -:512: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #512: FILE: drivers/gpu/drm/i915/i915_gpu_error.c:863:
> +int i915_obj_state_buf_init(struct drm_i915_error_state_buf *ebuf,
> +				size_t count)
>
> -:518: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!ebuf->buf"
> #518: FILE: drivers/gpu/drm/i915/i915_gpu_error.c:869:
> +	if (ebuf->buf == NULL)
>
> -:549: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
> #549: FILE: drivers/gpu/drm/i915/i915_sysfs.c:585:
>  }
> +#define GEM_OBJ_STAT_BUF_SIZE (4*1024) /* 4KB */
>
> -:549: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #549: FILE: drivers/gpu/drm/i915/i915_sysfs.c:585:
> +#define GEM_OBJ_STAT_BUF_SIZE (4*1024) /* 4KB */
>                                  ^
>
> -:550: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
> #550: FILE: drivers/gpu/drm/i915/i915_sysfs.c:586:
> +#define GEM_OBJ_STAT_BUF_SIZE_MAX (1024*1024) /* 1MB */
>                                         ^
>
> -:559: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #559: FILE: drivers/gpu/drm/i915/i915_sysfs.c:595:
> +static ssize_t i915_gem_read_objects(struct file *filp,
> +				struct kobject *memtrack_kobj,
>
> -:601: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #601: FILE: drivers/gpu/drm/i915/i915_sysfs.c:637:
> +		DRM_ERROR(
>
> -:610: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #610: FILE: drivers/gpu/drm/i915/i915_sysfs.c:646:
> +			ret = i915_obj_state_buf_init(&attr_priv->buf,
> +				buf_size);
>
> -:612: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #612: FILE: drivers/gpu/drm/i915/i915_sysfs.c:648:
> +				DRM_ERROR(
>
> -:643: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #643: FILE: drivers/gpu/drm/i915/i915_sysfs.c:679:
> +			ret = i915_obj_state_buf_init(&attr_priv->buf,
> +						buf_size);
>
> -:645: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #645: FILE: drivers/gpu/drm/i915/i915_sysfs.c:681:
> +				DRM_ERROR(
>
> -:657: CHECK:BRACES: braces {} should be used on all arms of this statement
> #657: FILE: drivers/gpu/drm/i915/i915_sysfs.c:693:
> +	if (bytes_available > 0) {
> [...]
> +	} else
> [...]
>
> -:660: CHECK:BRACES: Unbalanced braces around else statement
> #660: FILE: drivers/gpu/drm/i915/i915_sysfs.c:696:
> +	} else
>
> -:690: WARNING:SYMBOLIC_PERMS: Symbolic permissions 'S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH' are not preferred. Consider using octal permissions '0644'.
> #690: FILE: drivers/gpu/drm/i915/i915_sysfs.c:726:
> +	obj_attr->attr.mode = S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH;
>
> -:700: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
> #700: FILE: drivers/gpu/drm/i915/i915_sysfs.c:736:
> +		DRM_ERROR(
>
> -:723: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!obj_attr"
> #723: FILE: drivers/gpu/drm/i915/i915_sysfs.c:759:
> +	if (WARN_ON(obj_attr == NULL))
>
> total: 0 errors, 1 warnings, 34 checks, 664 lines checked
> 486204b0dac9 drm/i915: Output the user virtual addresses for GFX buffers
> -:97: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #97: FILE: drivers/gpu/drm/i915/i915_gem.c:6345:
> +				list_for_each_entry_safe(virt_entry,
> +						virt_next,
>
> -:165: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "!new_entry"
> #165: FILE: drivers/gpu/drm/i915/i915_gem.c:6426:
> +			if (new_entry == NULL) {
>
> -:172: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #172: FILE: drivers/gpu/drm/i915/i915_gem.c:6433:
> +			list_add_tail(&new_entry->head,
> +				&pid_entry->virt_addr_head);
>
> -:212: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #212: FILE: drivers/gpu/drm/i915/i915_gem.c:6579:
> +				err_printf(m, " %p",
> +				(void *)(virt_entry->user_virt_addr & ~1));
>
> -:215: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #215: FILE: drivers/gpu/drm/i915/i915_gem.c:6582:
> +				err_printf(m, " %p*",
> +				(void *)virt_entry->user_virt_addr);
>
> total: 0 errors, 0 warnings, 5 checks, 199 lines checked
> 05a4782cc77f drm/i915: Update process mm stats for Gen GFX memory
> -:80: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #80: FILE: drivers/gpu/drm/i915/i915_gem.c:6306:
> +		add_mm_counter(pid_entry->attr_priv->mm, MM_FILEPAGES,
> +				new_size - old_size);
>
> -:248: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
> #248: FILE: drivers/gpu/drm/i915/i915_sysfs.c:770:
> +		add_mm_counter(attr_priv->mm, MM_FILEPAGES,
> +				-attr_priv->rss_size);
>
> total: 0 errors, 0 warnings, 2 checks, 192 lines checked
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux