Chris, as we're ready to enable 48bit full ppgtt for guest, so patch like
https://patchwork.freedesktop.org/patch/162507/ will change this part too.
I'd like you to send this one, so Tina can refresh against this to apply.
Thanks
On 2017.07.26 15:44:00 +0100, Chris Wilson wrote:
>
> I was looki
Yep, looks better in the end. If you could send a proper patch I'd
review it.
On Wed, Jul 26, 2017 at 03:44:00PM +0100, Chris Wilson wrote:
> Quoting Imre Deak (2017-07-25 22:12:36)
> > From: Damien Lespiau
> >
> > We exit early if has_aliasing_ppgtt is 0, so towards the end of the
> > function
Quoting Imre Deak (2017-07-25 22:12:36)
> From: Damien Lespiau
>
> We exit early if has_aliasing_ppgtt is 0, so towards the end of the
> function has_aliasing_ppgtt can only be 1.
>
> Also:
>
> if (foo)
> return 1;
> else
> return 0;
>
> when foo
From: Damien Lespiau
We exit early if has_aliasing_ppgtt is 0, so towards the end of the
function has_aliasing_ppgtt can only be 1.
Also:
if (foo)
return 1;
else
return 0;
when foo is already a bool is really just:
return foo;
v2:
- Sim
On Wed, Jun 01, 2016 at 06:04:40PM +0100, Chris Wilson wrote:
> On Wed, Jun 01, 2016 at 05:01:13PM +0100, Damien Lespiau wrote:
> > We exit early if has_aliasing_ppgtt is 0, so towards the end of the
> > function has_aliasing_ppgtt can only be 1.
> >
> > Also:
> >
> > if (foo)
> >
On Wed, Jun 01, 2016 at 05:01:13PM +0100, Damien Lespiau wrote:
> We exit early if has_aliasing_ppgtt is 0, so towards the end of the
> function has_aliasing_ppgtt can only be 1.
>
> Also:
>
> if (foo)
> return 1;
> else
> return 0;
>
> when foo is already
We exit early if has_aliasing_ppgtt is 0, so towards the end of the
function has_aliasing_ppgtt can only be 1.
Also:
if (foo)
return 1;
else
return 0;
when foo is already a bool is really just:
return foo;
Signed-off-by: Damien Lespiau
-
