ich777/linux_media
1
0
Fork 0
mirror of https://github.com/tbsdtv/linux_media.git synced 2025-07-23 20:51:03 +02:00
Code Issues Packages Projects Releases Wiki Activity

misc: sgi-gru: use explicitly signed char

Browse Source 
With char becoming unsigned by default, and with `char` alone being
ambiguous and based on architecture, signed chars need to be marked
explicitly as such. This fixes warnings like:

drivers/misc/sgi-gru/grumain.c:711 gru_check_chiplet_assignment() warn: 'gts->ts_user_chiplet_id' is unsigned

Cc: Dimitri Sivanich <dimitri.sivanich@hpe.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Link: https://lore.kernel.org/r/20221025025223.573543-1-Jason@zx2c4.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Jason A. Donenfeld
2022-10-25 04:52:23 +02:00
committed by Greg Kroah-Hartman
parent 1deac35b2d
commit 6770473832
2 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/misc/sgi-gru/grumain.c
View File

@@ -152,7 +152,7 @@ static int gru_assign_asid(struct gru_state *gru)
* Optionally, build an array of chars that contain the bit numbers allocated. * Optionally, build an array of chars that contain the bit numbers allocated.
*/ */
static unsigned long reserve_resources(unsigned long *p, int n, int mmax, static unsigned long reserve_resources(unsigned long *p, int n, int mmax,
char *idx) signed char *idx)
{ {
unsigned long bits = 0; unsigned long bits = 0;
int i; int i;
@@ -170,14 +170,14 @@ static unsigned long reserve_resources(unsigned long *p, int n, int mmax,
} }
unsigned long gru_reserve_cb_resources(struct gru_state *gru, int cbr_au_count, unsigned long gru_reserve_cb_resources(struct gru_state *gru, int cbr_au_count,
char *cbmap) signed char *cbmap)
{ {
return reserve_resources(&gru->gs_cbr_map, cbr_au_count, GRU_CBR_AU, return reserve_resources(&gru->gs_cbr_map, cbr_au_count, GRU_CBR_AU,
cbmap); cbmap);
} }
unsigned long gru_reserve_ds_resources(struct gru_state *gru, int dsr_au_count, unsigned long gru_reserve_ds_resources(struct gru_state *gru, int dsr_au_count,
char *dsmap) signed char *dsmap)
{ {
return reserve_resources(&gru->gs_dsr_map, dsr_au_count, GRU_DSR_AU, return reserve_resources(&gru->gs_dsr_map, dsr_au_count, GRU_DSR_AU,
dsmap); dsmap);

14
drivers/misc/sgi-gru/grutables.h
View File

@@ -351,7 +351,7 @@ struct gru_thread_state {
pid_t ts_tgid_owner; /* task that is using the pid_t ts_tgid_owner; /* task that is using the
context - for migration */ context - for migration */
short ts_user_blade_id;/* user selected blade */ short ts_user_blade_id;/* user selected blade */
char ts_user_chiplet_id;/* user selected chiplet */ signed char ts_user_chiplet_id;/* user selected chiplet */
unsigned short ts_sizeavail; /* Pagesizes in use */ unsigned short ts_sizeavail; /* Pagesizes in use */
int ts_tsid; /* thread that owns the int ts_tsid; /* thread that owns the
structure */ structure */
@@ -364,11 +364,11 @@ struct gru_thread_state {
required for contest */ required for contest */
unsigned char ts_cbr_au_count;/* Number of CBR resources unsigned char ts_cbr_au_count;/* Number of CBR resources
required for contest */ required for contest */
char ts_cch_req_slice;/* CCH packet slice */ signed char ts_cch_req_slice;/* CCH packet slice */
char ts_blade; /* If >= 0, migrate context if signed char ts_blade; /* If >= 0, migrate context if
ref from different blade */ ref from different blade */
char ts_force_cch_reload; signed char ts_force_cch_reload;
char ts_cbr_idx[GRU_CBR_AU];/* CBR numbers of each signed char ts_cbr_idx[GRU_CBR_AU];/* CBR numbers of each
allocated CB */ allocated CB */
int ts_data_valid; /* Indicates if ts_gdata has int ts_data_valid; /* Indicates if ts_gdata has
valid data */ valid data */
@@ -643,9 +643,9 @@ extern struct gru_thread_state *gru_alloc_gts(struct vm_area_struct *vma,
int cbr_au_count, int dsr_au_count, int cbr_au_count, int dsr_au_count,
unsigned char tlb_preload_count, int options, int tsid); unsigned char tlb_preload_count, int options, int tsid);
extern unsigned long gru_reserve_cb_resources(struct gru_state *gru, extern unsigned long gru_reserve_cb_resources(struct gru_state *gru,
int cbr_au_count, char *cbmap); int cbr_au_count, signed char *cbmap);
extern unsigned long gru_reserve_ds_resources(struct gru_state *gru, extern unsigned long gru_reserve_ds_resources(struct gru_state *gru,
int dsr_au_count, char *dsmap); int dsr_au_count, signed char *dsmap);
extern vm_fault_t gru_fault(struct vm_fault *vmf); extern vm_fault_t gru_fault(struct vm_fault *vmf);
extern struct gru_mm_struct *gru_register_mmu_notifier(void); extern struct gru_mm_struct *gru_register_mmu_notifier(void);
extern void gru_drop_mmu_notifier(struct gru_mm_struct *gms); extern void gru_drop_mmu_notifier(struct gru_mm_struct *gms);