On 2/28/2019 5:56 AM, Qi Zhang wrote: > Resolve static analysis reported issue in > ice_get_itr_intrl_gran and ice_ptg_find_ptype.
Same comment with previous patch, related to the commit log. Commit log says "static analysis reported issues" are solved, what are they really? If we know them we can know your intention and say something about the code below, otherwise is there a way to figure out if something wrong below? And "impact" part, "what is the impact of the change?" Will those issues create definite memory corruption? Or protection for possible issue. Knowing this helps maintainers and LTS maintainers to priorities the patch accordingly. Also please use "fix" instead of "resolve", fix is kind of keyword we tend to use, and provide a fixes line for whatever fixed. > > Signed-off-by: Bruce Allan <bruce.w.al...@intel.com> > Signed-off-by: Paul M Stillwell Jr <paul.m.stillwell...@intel.com> > Signed-off-by: Qi Zhang <qi.z.zh...@intel.com>