Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]
OK, thanks for the feedback! ☺ On Mon, 2017-07-10 at 09:46 -0700, Anuj Phogat wrote: > These patches are not fixing any known bug. But they are adding the > previously missing surface size limits for the hardware. It's really hard > to hit these limits. But, let's pick them to stable for the sake of > completion. > Thanks for marking them for mesa-stable. > > > On Fri, Jul 7, 2017 at 7:25 AM, Andres Gomezwrote: > > On Thu, 2017-07-06 at 18:21 +0100, Emil Velikov wrote: > > > On 3 July 2017 at 21:14, Andres Gomez wrote: > > > > Actually, forgot to add -stable into CC. > > > > > > > > Forwarded Message > > > > From: Andres Gomez > > > > To: Anuj Phogat , mesa-dev@lists.freedesktop.org > > > > Subject: Re: [Mesa-dev] [PATCH 1/2] intel/isl: Use uint64_t to store > > > > total surface size > > > > Date: Mon, 03 Jul 2017 23:02:31 +0300 > > > > > > > > It looks like we could want these 2 into -stable (?) > > > > > > > > > > Shouldn't hurt, despite that most of the > > > isl_surf_init/isl_surf_get_[a-z]_surf handling is a simple assert(ok). > > > I'll leave the call to the experts, but my take is "don't bother". > > > > OK, I'll wait to see if Anuj has anything to say before picking this > > one, then. > > > > Thanks for the feedback! > > > > -- > > Br, > > > > Andres > > ___ > mesa-stable mailing list > mesa-sta...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-stable -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]
These patches are not fixing any known bug. But they are adding the previously missing surface size limits for the hardware. It's really hard to hit these limits. But, let's pick them to stable for the sake of completion. Thanks for marking them for mesa-stable. On Fri, Jul 7, 2017 at 7:25 AM, Andres Gomezwrote: > On Thu, 2017-07-06 at 18:21 +0100, Emil Velikov wrote: >> On 3 July 2017 at 21:14, Andres Gomez wrote: >> > Actually, forgot to add -stable into CC. >> > >> > Forwarded Message >> > From: Andres Gomez >> > To: Anuj Phogat , mesa-dev@lists.freedesktop.org >> > Subject: Re: [Mesa-dev] [PATCH 1/2] intel/isl: Use uint64_t to store >> > total surface size >> > Date: Mon, 03 Jul 2017 23:02:31 +0300 >> > >> > It looks like we could want these 2 into -stable (?) >> > >> >> Shouldn't hurt, despite that most of the >> isl_surf_init/isl_surf_get_[a-z]_surf handling is a simple assert(ok). >> I'll leave the call to the experts, but my take is "don't bother". > > OK, I'll wait to see if Anuj has anything to say before picking this > one, then. > > Thanks for the feedback! > > -- > Br, > > Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]
On Thu, 2017-07-06 at 18:21 +0100, Emil Velikov wrote: > On 3 July 2017 at 21:14, Andres Gomezwrote: > > Actually, forgot to add -stable into CC. > > > > Forwarded Message > > From: Andres Gomez > > To: Anuj Phogat , mesa-dev@lists.freedesktop.org > > Subject: Re: [Mesa-dev] [PATCH 1/2] intel/isl: Use uint64_t to store > > total surface size > > Date: Mon, 03 Jul 2017 23:02:31 +0300 > > > > It looks like we could want these 2 into -stable (?) > > > > Shouldn't hurt, despite that most of the > isl_surf_init/isl_surf_get_[a-z]_surf handling is a simple assert(ok). > I'll leave the call to the experts, but my take is "don't bother". OK, I'll wait to see if Anuj has anything to say before picking this one, then. Thanks for the feedback! -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev