All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org>
Cc: "mesa-dev-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<mesa-dev-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	"nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
Subject: Re: [PATCH mesa 2/3] nv30: Fix color resolving for nv3x cards
Date: Wed, 9 Sep 2015 15:42:50 +0200	[thread overview]
Message-ID: <55F0375A.5040608@redhat.com> (raw)
In-Reply-To: <55EE93EC.1090100-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Hi,

On 08-09-15 09:53, Hans de Goede wrote:
> Hi,
>
> On 07-09-15 21:55, Ilia Mirkin wrote:
>> May I ask why you're doing 512x512 instead of 1024x1024? These are
>> already scaled up coordinates, so 1024x1024 should work no? Or is it
>> because of the seams on the edges? Do those not also appear with
>> 512x512 or does it sample outside of the box?
>
> This is my bad because of the bug fixed by patch 1/3 I had 512x512 in there
> for a while, I've moved back and forth between 512 and 1024 a couple of times.
>
> I've also tried 2048 but the hardware does not like that.
>
> I will retest with 1024 and submit a fixed version.
>
> I've not noticed any seams on the edges, even though I've been actively looking
> for them.
>
>> Separately, why not use this approach on nv40 as well? I can't imagine
>> the blitter would be faster... does this result in lower quality?
>
> I've the feeling the sifm bilinear filtering is more "blurry" then the
> blitter one. I will do some more objective (ahem) tests on nv4x and get
> back to you on this.

Ok I've run some tests comparing the rendering between using the blitter
and the sifm and the results seem identical, so one v2 using 1024x1024
blocks and doing so on all nv3x/nv4x cards coming up.

Regards,

Hans
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

  parent reply	other threads:[~2015-09-09 13:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-07 19:50 [PATCH mesa 1/3] nv30: Fix max width / height checks in nv30 sifm code Hans de Goede
2015-09-07 19:50 ` [PATCH mesa 2/3] nv30: Fix color resolving for nv3x cards Hans de Goede
     [not found]   ` <1441655450-9336-2-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-07 19:55     ` Ilia Mirkin
     [not found]       ` <CAKb7Uvhk56knb4ovaSrMDrOXKQq4ODJm3n1xhwQ+q=2jiNGbvA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-09-08  7:53         ` Hans de Goede
     [not found]           ` <55EE93EC.1090100-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-09 13:42             ` Hans de Goede [this message]
2015-09-07 19:50 ` [PATCH mesa 3/3] nv30: Disable msaa for now because it causes lockups Hans de Goede
2015-09-07 20:00   ` Ilia Mirkin
2015-09-08  8:00     ` Hans de Goede
     [not found]       ` <55EE95A6.5040307-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-08  8:48         ` Ben Skeggs
     [not found] ` <1441655450-9336-1-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-09-07 19:52   ` [PATCH mesa 1/3] nv30: Fix max width / height checks in nv30 sifm code Ilia Mirkin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55F0375A.5040608@redhat.com \
    --to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org \
    --cc=mesa-dev-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.