lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1421337444.12868.4.camel@x220>
Date:	Thu, 15 Jan 2015 16:57:24 +0100
From:	Paul Bolle <pebolle@...cali.nl>
To:	Jerome Glisse <j.glisse@...il.com>
Cc:	Alex Deucher <alexdeucher@...il.com>,
	Rickard Strandqvist <rickard_strandqvist@...ctrumdigital.se>,
	Alex Deucher <alexander.deucher@....com>,
	Christian König <christian.koenig@....com>,
	Maling list - DRI developers 
	<dri-devel@...ts.freedesktop.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] drm/radeon/rv515: Remove unused function

On Thu, 2015-01-15 at 10:43 -0500, Jerome Glisse wrote:
> This one looks highly suspicious, i need to check, but i would think
> that this function should be use !

Have a look at commit 76a0df859def ("drm/radeon: rework ring function
handling"), which removed the two users of rv515_ring_start.

I don't think Rickard ever included information like that in this
seemingly unending series. I'd guess looking for information like that
might slow things down, but it would aid the reviewers and improve the
overall quality of the series.

> On Wed, Jan 14, 2015 at 4:44 PM, Alex Deucher <alexdeucher@...il.com> wrote:
> > On Tue, Jan 13, 2015 at 1:55 PM, Rickard Strandqvist
> > <rickard_strandqvist@...ctrumdigital.se> wrote:
> >> Remove the function rv515_ring_start() that is not used anywhere.
> >>
> >> This was partially found by using a static code analysis program called cppcheck.
> >>
> >> Signed-off-by: Rickard Strandqvist <rickard_strandqvist@...ctrumdigital.se>
> >
> > Applied.  thanks!
> >
> > Alex
> >
> >> ---
> >>  drivers/gpu/drm/radeon/radeon_asic.h |    1 -
> >>  drivers/gpu/drm/radeon/rv515.c       |   68 ----------------------------------
> >>  2 files changed, 69 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/radeon/radeon_asic.h b/drivers/gpu/drm/radeon/radeon_asic.h
> >> index d8ace5b..52667b3 100644
> >> --- a/drivers/gpu/drm/radeon/radeon_asic.h
> >> +++ b/drivers/gpu/drm/radeon/radeon_asic.h
> >> @@ -280,7 +280,6 @@ int rv515_init(struct radeon_device *rdev);
> >>  void rv515_fini(struct radeon_device *rdev);
> >>  uint32_t rv515_mc_rreg(struct radeon_device *rdev, uint32_t reg);
> >>  void rv515_mc_wreg(struct radeon_device *rdev, uint32_t reg, uint32_t v);
> >> -void rv515_ring_start(struct radeon_device *rdev, struct radeon_ring *ring);
> >>  void rv515_bandwidth_update(struct radeon_device *rdev);
> >>  int rv515_resume(struct radeon_device *rdev);
> >>  int rv515_suspend(struct radeon_device *rdev);
> >> diff --git a/drivers/gpu/drm/radeon/rv515.c b/drivers/gpu/drm/radeon/rv515.c
> >> index 8a477bf..cb966e6 100644
> >> --- a/drivers/gpu/drm/radeon/rv515.c
> >> +++ b/drivers/gpu/drm/radeon/rv515.c
> >> @@ -59,74 +59,6 @@ void rv515_debugfs(struct radeon_device *rdev)
> >>         }
> >>  }
> >>
> >> -void rv515_ring_start(struct radeon_device *rdev, struct radeon_ring *ring)
> >> -{
> >> -       int r;
> >> -
> >> -       r = radeon_ring_lock(rdev, ring, 64);
> >> -       if (r) {
> >> -               return;
> >> -       }
> >> -       radeon_ring_write(ring, PACKET0(ISYNC_CNTL, 0));
> >> -       radeon_ring_write(ring,
> >> -                         ISYNC_ANY2D_IDLE3D |
> >> -                         ISYNC_ANY3D_IDLE2D |
> >> -                         ISYNC_WAIT_IDLEGUI |
> >> -                         ISYNC_CPSCRATCH_IDLEGUI);
> >> -       radeon_ring_write(ring, PACKET0(WAIT_UNTIL, 0));
> >> -       radeon_ring_write(ring, WAIT_2D_IDLECLEAN | WAIT_3D_IDLECLEAN);
> >> -       radeon_ring_write(ring, PACKET0(R300_DST_PIPE_CONFIG, 0));
> >> -       radeon_ring_write(ring, R300_PIPE_AUTO_CONFIG);
> >> -       radeon_ring_write(ring, PACKET0(GB_SELECT, 0));
> >> -       radeon_ring_write(ring, 0);
> >> -       radeon_ring_write(ring, PACKET0(GB_ENABLE, 0));
> >> -       radeon_ring_write(ring, 0);
> >> -       radeon_ring_write(ring, PACKET0(R500_SU_REG_DEST, 0));
> >> -       radeon_ring_write(ring, (1 << rdev->num_gb_pipes) - 1);
> >> -       radeon_ring_write(ring, PACKET0(VAP_INDEX_OFFSET, 0));
> >> -       radeon_ring_write(ring, 0);
> >> -       radeon_ring_write(ring, PACKET0(RB3D_DSTCACHE_CTLSTAT, 0));
> >> -       radeon_ring_write(ring, RB3D_DC_FLUSH | RB3D_DC_FREE);
> >> -       radeon_ring_write(ring, PACKET0(ZB_ZCACHE_CTLSTAT, 0));
> >> -       radeon_ring_write(ring, ZC_FLUSH | ZC_FREE);
> >> -       radeon_ring_write(ring, PACKET0(WAIT_UNTIL, 0));
> >> -       radeon_ring_write(ring, WAIT_2D_IDLECLEAN | WAIT_3D_IDLECLEAN);
> >> -       radeon_ring_write(ring, PACKET0(GB_AA_CONFIG, 0));
> >> -       radeon_ring_write(ring, 0);
> >> -       radeon_ring_write(ring, PACKET0(RB3D_DSTCACHE_CTLSTAT, 0));
> >> -       radeon_ring_write(ring, RB3D_DC_FLUSH | RB3D_DC_FREE);
> >> -       radeon_ring_write(ring, PACKET0(ZB_ZCACHE_CTLSTAT, 0));
> >> -       radeon_ring_write(ring, ZC_FLUSH | ZC_FREE);
> >> -       radeon_ring_write(ring, PACKET0(GB_MSPOS0, 0));
> >> -       radeon_ring_write(ring,
> >> -                         ((6 << MS_X0_SHIFT) |
> >> -                          (6 << MS_Y0_SHIFT) |
> >> -                          (6 << MS_X1_SHIFT) |
> >> -                          (6 << MS_Y1_SHIFT) |
> >> -                          (6 << MS_X2_SHIFT) |
> >> -                          (6 << MS_Y2_SHIFT) |
> >> -                          (6 << MSBD0_Y_SHIFT) |
> >> -                          (6 << MSBD0_X_SHIFT)));
> >> -       radeon_ring_write(ring, PACKET0(GB_MSPOS1, 0));
> >> -       radeon_ring_write(ring,
> >> -                         ((6 << MS_X3_SHIFT) |
> >> -                          (6 << MS_Y3_SHIFT) |
> >> -                          (6 << MS_X4_SHIFT) |
> >> -                          (6 << MS_Y4_SHIFT) |
> >> -                          (6 << MS_X5_SHIFT) |
> >> -                          (6 << MS_Y5_SHIFT) |
> >> -                          (6 << MSBD1_SHIFT)));
> >> -       radeon_ring_write(ring, PACKET0(GA_ENHANCE, 0));
> >> -       radeon_ring_write(ring, GA_DEADLOCK_CNTL | GA_FASTSYNC_CNTL);
> >> -       radeon_ring_write(ring, PACKET0(GA_POLY_MODE, 0));
> >> -       radeon_ring_write(ring, FRONT_PTYPE_TRIANGE | BACK_PTYPE_TRIANGE);
> >> -       radeon_ring_write(ring, PACKET0(GA_ROUND_MODE, 0));
> >> -       radeon_ring_write(ring, GEOMETRY_ROUND_NEAREST | COLOR_ROUND_NEAREST);
> >> -       radeon_ring_write(ring, PACKET0(0x20C8, 0));
> >> -       radeon_ring_write(ring, 0);
> >> -       radeon_ring_unlock_commit(rdev, ring, false);
> >> -}
> >> -
> >>  int rv515_mc_wait_for_idle(struct radeon_device *rdev)
> >>  {
> >>         unsigned i;
> >> --
> >> 1.7.10.4


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ