From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Date: Tue, 24 Oct 2017 12:52:24 +0000 Subject: Re: [PATCH] drm/i915/gvt: Use common error handling code in shadow_workload_ring_buffer() Message-Id: <87efpsiu7r.fsf@intel.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: SF Markus Elfring , dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, intel-gvt-dev@lists.freedesktop.org, David Airlie , Joonas Lahtinen , Rodrigo Vivi , Zhenyu Wang , Zhi Wang Cc: kernel-janitors@vger.kernel.org, LKML On Tue, 24 Oct 2017, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 24 Oct 2017 14:20:06 +0200 > > Add a jump target so that a call of the function "gvt_vgpu_err" is stored > only once at the end of this function implementation. > Replace two calls by goto statements. > > This issue was detected by using the Coccinelle software. I don't think this is an issue or an improvement. BR, Jani. > > Signed-off-by: Markus Elfring > --- > drivers/gpu/drm/i915/gvt/cmd_parser.c | 18 ++++++++++-------- > 1 file changed, 10 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/gvt/cmd_parser.c > index 2c0ccbb817dc..caa181380958 100644 > --- a/drivers/gpu/drm/i915/gvt/cmd_parser.c > +++ b/drivers/gpu/drm/i915/gvt/cmd_parser.c > @@ -2640,10 +2640,9 @@ static int shadow_workload_ring_buffer(struct intel_vgpu_workload *workload) > if (gma_head > gma_tail) { > ret = copy_gma_to_hva(vgpu, vgpu->gtt.ggtt_mm, > gma_head, gma_top, shadow_ring_buffer_va); > - if (ret < 0) { > - gvt_vgpu_err("fail to copy guest ring buffer\n"); > - return ret; > - } > + if (ret < 0) > + goto report_failure; > + > shadow_ring_buffer_va += ret; > gma_head = workload->rb_start; > } > @@ -2651,11 +2650,14 @@ static int shadow_workload_ring_buffer(struct intel_vgpu_workload *workload) > /* copy head or start <-> tail */ > ret = copy_gma_to_hva(vgpu, vgpu->gtt.ggtt_mm, gma_head, gma_tail, > shadow_ring_buffer_va); > - if (ret < 0) { > - gvt_vgpu_err("fail to copy guest ring buffer\n"); > - return ret; > - } > + if (ret < 0) > + goto report_failure; > + > return 0; > + > +report_failure: > + gvt_vgpu_err("fail to copy guest ring buffer\n"); > + return ret; > } > > int intel_gvt_scan_and_shadow_ringbuffer(struct intel_vgpu_workload *workload) -- Jani Nikula, Intel Open Source Technology Center