Re: [Mesa-dev] [Mesa-stable] [Fwd: Re: [PATCH 1/2] intel/isl: Use uint64_t to store total surface size]

2017-07-10 Thread Andres Gomez
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 Gomez  wrote:
> > 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]

2017-07-10 Thread Anuj Phogat
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 Gomez  wrote:
> 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]

2017-07-07 Thread Andres Gomez
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