Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-20 Thread Marek Olšák
FYI, I'm without my computer until Sunday, so I won't be able to push anything until then. Marek On Sep 20, 2017 7:36 AM, "Nicolai Hähnle" wrote: Looks good to me. I guess Emil didn't find any problems either. Feel free to push with Reviewed-by: Nicolai Hähnle On 18.09.2017 18:08, Marek Olš

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-20 Thread Nicolai Hähnle
Looks good to me. I guess Emil didn't find any problems either. Feel free to push with Reviewed-by: Nicolai Hähnle On 18.09.2017 18:08, Marek Olšák wrote: On Thu, Sep 14, 2017 at 4:52 PM, Nicolai Hähnle wrote: On 14.09.2017 16:19, Emil Velikov wrote: Hi Marek, On 14 September 2017 at 14:

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-18 Thread Marek Olšák
It wasn't copy-paste, it was "cp". Marek On Mon, Sep 18, 2017 at 9:42 PM, Grazvydas Ignotas wrote: > Has anyone ran full piglit on these with r600? Copy-paste bugs are > very easy to introduce. > > Gražvydas > > On Mon, Sep 18, 2017 at 7:32 PM, Marek Olšák wrote: >> Latest version with an updat

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-18 Thread Grazvydas Ignotas
Has anyone ran full piglit on these with r600? Copy-paste bugs are very easy to introduce. Gražvydas On Mon, Sep 18, 2017 at 7:32 PM, Marek Olšák wrote: > Latest version with an updated commit message: > https://cgit.freedesktop.org/~mareko/mesa/commit/?h=master&id=8671613ed19586ac1eceda48eb067a

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-18 Thread Marek Olšák
Latest version with an updated commit message: https://cgit.freedesktop.org/~mareko/mesa/commit/?h=master&id=8671613ed19586ac1eceda48eb067a416f9c93be Marek On Mon, Sep 18, 2017 at 6:18 PM, Emil Velikov wrote: > On 18 September 2017 at 17:08, Marek Olšák wrote: >> On Thu, Sep 14, 2017 at 4:52 PM

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-18 Thread Emil Velikov
On 18 September 2017 at 17:08, Marek Olšák wrote: > On Thu, Sep 14, 2017 at 4:52 PM, Nicolai Hähnle wrote: >> On 14.09.2017 16:19, Emil Velikov wrote: >>> >>> Hi Marek, >>> >>> On 14 September 2017 at 14:06, Marek Olšák wrote: From: Marek Olšák This marks the end of code sha

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-18 Thread Marek Olšák
On Thu, Sep 14, 2017 at 4:52 PM, Nicolai Hähnle wrote: > On 14.09.2017 16:19, Emil Velikov wrote: >> >> Hi Marek, >> >> On 14 September 2017 at 14:06, Marek Olšák wrote: >>> >>> From: Marek Olšák >>> >>> This marks the end of code sharing between r600 and radeonsi. >>> >> It has the "what" but i

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-14 Thread Dave Airlie
On 15 September 2017 at 02:10, Marek Olšák wrote: > On Thu, Sep 14, 2017 at 4:19 PM, Emil Velikov > wrote: >> Hi Marek, >> >> On 14 September 2017 at 14:06, Marek Olšák wrote: >>> From: Marek Olšák >>> >>> This marks the end of code sharing between r600 and radeonsi. >>> >> It has the "what" b

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-14 Thread Marek Olšák
On Thu, Sep 14, 2017 at 4:19 PM, Emil Velikov wrote: > Hi Marek, > > On 14 September 2017 at 14:06, Marek Olšák wrote: >> From: Marek Olšák >> >> This marks the end of code sharing between r600 and radeonsi. >> > It has the "what" but it's missing the "why". Can you please add some > information

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-14 Thread Nicolai Hähnle
On 14.09.2017 16:19, Emil Velikov wrote: Hi Marek, On 14 September 2017 at 14:06, Marek Olšák wrote: From: Marek Olšák This marks the end of code sharing between r600 and radeonsi. It has the "what" but it's missing the "why". Can you please add some information. Basically, it's getting

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-14 Thread Emil Velikov
Hi Marek, On 14 September 2017 at 14:06, Marek Olšák wrote: > From: Marek Olšák > > This marks the end of code sharing between r600 and radeonsi. > It has the "what" but it's missing the "why". Can you please add some information. From a quick look which will make each binary ~140KiB larger (dr

[Mesa-dev] [PATCH] r600: fork and import gallium/radeon

2017-09-14 Thread Marek Olšák
From: Marek Olšák This marks the end of code sharing between r600 and radeonsi. A lot of functions had to be renamed to prevent linker conflicts. There are also minor cleanups. --- This one is huge. Please review here: https://cgit.freedesktop.org/~mareko/mesa/commit/?h=master&id=858b2d1c8cec7