#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
aed32395 |
| 11-Jan-2022 |
Jiasheng Jiang <jiasheng@iscas.ac.cn> |
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). The
fsi: core: Check error number after calling ida_simple_get
If allocation fails, the ida_simple_get() will return error number. So master->idx could be error number and be used in dev_set_name(). Therefore, it should be better to check it and return error if fails, like the ida_simple_get() in __fsi_get_new_minor().
OpenBMC-Staging-Count: 1 Fixes: 09aecfab93b8 ("drivers/fsi: Add fsi master definition") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Reviewed-by: Eddie James <eajames@linux.ibm.com> Link: https://lore.kernel.org/r/20220111073411.614138-1-jiasheng@iscas.ac.cn Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
e23f44ee |
| 03-Apr-2022 |
Tom Rix <trix@redhat.com> |
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: wa
fsi: cleanup extern usage in function definition
Smatch reports these issues fsi-core.c:395:12: warning: function 'fsi_slave_claim_range' with external linkage has definition fsi-core.c:409:13: warning: function 'fsi_slave_release_range' with external linkage has definition
The storage-class-specifier extern is not needed in a definition, so remove it.
Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20220403140937.3833578-1-trix@redhat.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|
#
ab53623d |
| 07-Feb-2022 |
Eddie James <eajames@linux.ibm.com> |
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Ste
fsi: Add trace events in initialization path
Add definitions for trace events to show the scanning flow.
OpenBMC-Staging-Count: 1 Signed-off-by: Eddie James <eajames@linux.ibm.com> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org> Link: https://lore.kernel.org/r/20220207161640.35605-1-eajames@linux.ibm.com Signed-off-by: Joel Stanley <joel@jms.id.au>
show more ...
|