* [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers [not found] <20200928233041.GA23230@paulmck-ThinkPad-P72> @ 2020-09-28 23:30 ` paulmck 2020-10-01 7:17 ` Joonas Lahtinen 0 siblings, 1 reply; 4+ messages in thread From: paulmck @ 2020-09-28 23:30 UTC (permalink / raw) To: rcu Cc: peterz, fweisbec, jiangshanlai, dri-devel, oleg, dhowells, edumazet, joel, mingo, David Airlie, kernel-team, Paul E . McKenney, intel-gfx, josh, rostedt, mathieu.desnoyers, tglx, mhocko, mgorman, linux-kernel, akpm, torvalds From: Thomas Gleixner <tglx@linutronix.de> CONFIG_PREEMPT_COUNT is now unconditionally enabled and will be removed. Cleanup the leftovers before doing so. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: David Airlie <airlied@linux.ie> Cc: Daniel Vetter <daniel@ffwll.ch> Cc: intel-gfx@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org Signed-off-by: Paul E. McKenney <paulmck@kernel.org> --- drivers/gpu/drm/i915/Kconfig.debug | 1 - drivers/gpu/drm/i915/i915_utils.h | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug index 1cb28c2..17d9b00 100644 --- a/drivers/gpu/drm/i915/Kconfig.debug +++ b/drivers/gpu/drm/i915/Kconfig.debug @@ -20,7 +20,6 @@ config DRM_I915_DEBUG bool "Enable additional driver debugging" depends on DRM_I915 select DEBUG_FS - select PREEMPT_COUNT select I2C_CHARDEV select STACKDEPOT select DRM_DP_AUX_CHARDEV diff --git a/drivers/gpu/drm/i915/i915_utils.h b/drivers/gpu/drm/i915/i915_utils.h index 5477337..ecfed86 100644 --- a/drivers/gpu/drm/i915/i915_utils.h +++ b/drivers/gpu/drm/i915/i915_utils.h @@ -337,8 +337,7 @@ wait_remaining_ms_from_jiffies(unsigned long timestamp_jiffies, int to_wait_ms) (Wmax)) #define wait_for(COND, MS) _wait_for((COND), (MS) * 1000, 10, 1000) -/* If CONFIG_PREEMPT_COUNT is disabled, in_atomic() always reports false. */ -#if defined(CONFIG_DRM_I915_DEBUG) && defined(CONFIG_PREEMPT_COUNT) +#ifdef CONFIG_DRM_I915_DEBUG # define _WAIT_FOR_ATOMIC_CHECK(ATOMIC) WARN_ON_ONCE((ATOMIC) && !in_atomic()) #else # define _WAIT_FOR_ATOMIC_CHECK(ATOMIC) do { } while (0) -- 2.9.5 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers 2020-09-28 23:30 ` [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers paulmck @ 2020-10-01 7:17 ` Joonas Lahtinen 2020-10-01 8:25 ` Thomas Gleixner 0 siblings, 1 reply; 4+ messages in thread From: Joonas Lahtinen @ 2020-10-01 7:17 UTC (permalink / raw) To: paulmck, rcu Cc: peterz, fweisbec, jiangshanlai, dri-devel, oleg, dhowells, edumazet, joel, mingo, David Airlie, kernel-team, Paul E . McKenney, intel-gfx, josh, rostedt, mathieu.desnoyers, tglx, mhocko, mgorman, linux-kernel, akpm, torvalds Quoting paulmck@kernel.org (2020-09-29 02:30:58) > From: Thomas Gleixner <tglx@linutronix.de> > > CONFIG_PREEMPT_COUNT is now unconditionally enabled and will be > removed. Cleanup the leftovers before doing so. Change looks fine: Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Are you looking for us to merge or merge through another tree? If us, did the base patch always enabling PREEMPT_COUNT go into 5.9 or is it heading to 5.10? We can queue this earliest for 5.11 as drm-next closed for 5.10 at week of -rc5. Regards, Joonas > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > Cc: Jani Nikula <jani.nikula@linux.intel.com> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: David Airlie <airlied@linux.ie> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: intel-gfx@lists.freedesktop.org > Cc: dri-devel@lists.freedesktop.org > Signed-off-by: Paul E. McKenney <paulmck@kernel.org> > --- > drivers/gpu/drm/i915/Kconfig.debug | 1 - > drivers/gpu/drm/i915/i915_utils.h | 3 +-- > 2 files changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug > index 1cb28c2..17d9b00 100644 > --- a/drivers/gpu/drm/i915/Kconfig.debug > +++ b/drivers/gpu/drm/i915/Kconfig.debug > @@ -20,7 +20,6 @@ config DRM_I915_DEBUG > bool "Enable additional driver debugging" > depends on DRM_I915 > select DEBUG_FS > - select PREEMPT_COUNT > select I2C_CHARDEV > select STACKDEPOT > select DRM_DP_AUX_CHARDEV > diff --git a/drivers/gpu/drm/i915/i915_utils.h b/drivers/gpu/drm/i915/i915_utils.h > index 5477337..ecfed86 100644 > --- a/drivers/gpu/drm/i915/i915_utils.h > +++ b/drivers/gpu/drm/i915/i915_utils.h > @@ -337,8 +337,7 @@ wait_remaining_ms_from_jiffies(unsigned long timestamp_jiffies, int to_wait_ms) > (Wmax)) > #define wait_for(COND, MS) _wait_for((COND), (MS) * 1000, 10, 1000) > > -/* If CONFIG_PREEMPT_COUNT is disabled, in_atomic() always reports false. */ > -#if defined(CONFIG_DRM_I915_DEBUG) && defined(CONFIG_PREEMPT_COUNT) > +#ifdef CONFIG_DRM_I915_DEBUG > # define _WAIT_FOR_ATOMIC_CHECK(ATOMIC) WARN_ON_ONCE((ATOMIC) && !in_atomic()) > #else > # define _WAIT_FOR_ATOMIC_CHECK(ATOMIC) do { } while (0) > -- > 2.9.5 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers 2020-10-01 7:17 ` Joonas Lahtinen @ 2020-10-01 8:25 ` Thomas Gleixner 2020-10-01 16:03 ` Paul E. McKenney 0 siblings, 1 reply; 4+ messages in thread From: Thomas Gleixner @ 2020-10-01 8:25 UTC (permalink / raw) To: Joonas Lahtinen, paulmck, rcu Cc: peterz, fweisbec, jiangshanlai, dri-devel, oleg, dhowells, edumazet, joel, mingo, David Airlie, kernel-team, Paul E . McKenney, intel-gfx, josh, rostedt, mathieu.desnoyers, mhocko, mgorman, linux-kernel, akpm, torvalds On Thu, Oct 01 2020 at 10:17, Joonas Lahtinen wrote: > Quoting paulmck@kernel.org (2020-09-29 02:30:58) >> CONFIG_PREEMPT_COUNT is now unconditionally enabled and will be >> removed. Cleanup the leftovers before doing so. > > Change looks fine: > > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > > Are you looking for us to merge or merge through another tree? > > If us, did the base patch always enabling PREEMPT_COUNT go into 5.9 or is > it heading to 5.10? We can queue this earliest for 5.11 as drm-next closed > for 5.10 at week of -rc5. If at all it goes through rcu/tip because it depends on the earlier patches. Thanks, tglx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers 2020-10-01 8:25 ` Thomas Gleixner @ 2020-10-01 16:03 ` Paul E. McKenney 0 siblings, 0 replies; 4+ messages in thread From: Paul E. McKenney @ 2020-10-01 16:03 UTC (permalink / raw) To: Thomas Gleixner Cc: peterz, fweisbec, jiangshanlai, dri-devel, oleg, dhowells, edumazet, joel, mingo, David Airlie, kernel-team, intel-gfx, josh, rostedt, rcu, mathieu.desnoyers, mhocko, mgorman, linux-kernel, akpm, torvalds On Thu, Oct 01, 2020 at 10:25:06AM +0200, Thomas Gleixner wrote: > On Thu, Oct 01 2020 at 10:17, Joonas Lahtinen wrote: > > Quoting paulmck@kernel.org (2020-09-29 02:30:58) > >> CONFIG_PREEMPT_COUNT is now unconditionally enabled and will be > >> removed. Cleanup the leftovers before doing so. > > > > Change looks fine: > > > > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Applied, thank you! > > Are you looking for us to merge or merge through another tree? > > > > If us, did the base patch always enabling PREEMPT_COUNT go into 5.9 or is > > it heading to 5.10? We can queue this earliest for 5.11 as drm-next closed > > for 5.10 at week of -rc5. > > If at all it goes through rcu/tip because it depends on the earlier patches. I was figuring on sending a pull request later today, Pacific Time. Thanx, Paul _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-10-01 16:32 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20200928233041.GA23230@paulmck-ThinkPad-P72>
2020-09-28 23:30 ` [Intel-gfx] [PATCH tip/core/rcu 11/15] drm/i915: Cleanup PREEMPT_COUNT leftovers paulmck
2020-10-01 7:17 ` Joonas Lahtinen
2020-10-01 8:25 ` Thomas Gleixner
2020-10-01 16:03 ` Paul E. McKenney
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox