diff mbox series

UBUNTU: [Config] Re-enable CONFIG_DRM_BOCHS

Message ID 20181003112220.820-1-tjaalton@ubuntu.com
State New
Headers show
Series UBUNTU: [Config] Re-enable CONFIG_DRM_BOCHS | expand

Commit Message

Timo Aaltonen Oct. 3, 2018, 11:22 a.m. UTC
From: Timo Aaltonen <timo.aaltonen@canonical.com>

BugLink: https://bugs.launchpad.net/bugs/1795857

This is required for xserver to work on QEMU with dummy VGA.

Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
---
 debian.master/config/annotations          | 2 --
 debian.master/config/config.common.ubuntu | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

Comments

Seth Forshee Oct. 3, 2018, 12:51 p.m. UTC | #1
On Wed, Oct 03, 2018 at 02:22:20PM +0300, Timo Aaltonen wrote:
> From: Timo Aaltonen <timo.aaltonen@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1795857
> 
> This is required for xserver to work on QEMU with dummy VGA.
> 
> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>

Can you confirm that the problem from LP #1378648 has been fixed when
this option is turned on?

> ---
>  debian.master/config/annotations          | 2 --
>  debian.master/config/config.common.ubuntu | 2 +-
>  2 files changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index 93813c874627..4d11ae71c875 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -1822,7 +1822,6 @@ CONFIG_DRM_SHMOBILE                             policy<{'armhf': 'm'}>
>  CONFIG_DRM_OMAP                                 policy<{'armhf': 'n'}>
>  CONFIG_DRM_TILCDC                               policy<{'armhf': 'm'}>
>  CONFIG_DRM_QXL                                  policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
> -CONFIG_DRM_BOCHS                                policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_DRM_VIRTIO_GPU                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>  CONFIG_DRM_FSL_DCU                              policy<{'armhf': 'm'}>
>  CONFIG_DRM_TEGRA                                policy<{'armhf-generic': 'm'}>
> @@ -1848,7 +1847,6 @@ CONFIG_DRM_PL111                                policy<{'arm64': 'm', 'armhf': '
>  CONFIG_DRM_TVE200                               policy<{'armhf': 'm'}>
>  #
>  CONFIG_DRM_MGAG200                              note<LP#1693337>
> -CONFIG_DRM_BOCHS                                note<LP#1378648>
>  CONFIG_DRM_STI                                  note<LP#1398458>
>  
>  # Menu: Device Drivers >> Graphics support >> Direct Rendering Manager (XFree86 4.1.0 and higher DRI support) >> AMD GPU
> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> index bdece4b92c89..e6e1f5d34ea1 100644
> --- a/debian.master/config/config.common.ubuntu
> +++ b/debian.master/config/config.common.ubuntu
> @@ -2315,7 +2315,7 @@ CONFIG_DRM_ARM=y
>  CONFIG_DRM_ARMADA=m
>  CONFIG_DRM_AST=m
>  CONFIG_DRM_ATMEL_HLCDC=m
> -# CONFIG_DRM_BOCHS is not set
> +CONFIG_DRM_BOCHS=m
>  CONFIG_DRM_BRIDGE=y
>  CONFIG_DRM_CDNS_DSI=m
>  CONFIG_DRM_CIRRUS_QEMU=m
> -- 
> 2.17.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Timo Aaltonen Oct. 3, 2018, 8:32 p.m. UTC | #2
On 03.10.2018 15:51, Seth Forshee wrote:
> On Wed, Oct 03, 2018 at 02:22:20PM +0300, Timo Aaltonen wrote:
>> From: Timo Aaltonen <timo.aaltonen@canonical.com>
>>
>> BugLink: https://bugs.launchpad.net/bugs/1795857
>>
>> This is required for xserver to work on QEMU with dummy VGA.
>>
>> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
> 
> Can you confirm that the problem from LP #1378648 has been fixed when
> this option is turned on?

Nope, X is still started too early for this to help. But adding this
could be targeted for next release.
Kleber Sacilotto de Souza Oct. 11, 2018, 11:52 a.m. UTC | #3
On 10/03/18 13:22, Timo Aaltonen wrote:
> From: Timo Aaltonen <timo.aaltonen@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1795857
> 
> This is required for xserver to work on QEMU with dummy VGA.
> 
> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
> ---
>  debian.master/config/annotations          | 2 --
>  debian.master/config/config.common.ubuntu | 2 +-
>  2 files changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index 93813c874627..4d11ae71c875 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -1822,7 +1822,6 @@ CONFIG_DRM_SHMOBILE                             policy<{'armhf': 'm'}>
>  CONFIG_DRM_OMAP                                 policy<{'armhf': 'n'}>
>  CONFIG_DRM_TILCDC                               policy<{'armhf': 'm'}>
>  CONFIG_DRM_QXL                                  policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
> -CONFIG_DRM_BOCHS                                policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>  CONFIG_DRM_VIRTIO_GPU                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>  CONFIG_DRM_FSL_DCU                              policy<{'armhf': 'm'}>
>  CONFIG_DRM_TEGRA                                policy<{'armhf-generic': 'm'}>
> @@ -1848,7 +1847,6 @@ CONFIG_DRM_PL111                                policy<{'arm64': 'm', 'armhf': '
>  CONFIG_DRM_TVE200                               policy<{'armhf': 'm'}>
>  #
>  CONFIG_DRM_MGAG200                              note<LP#1693337>
> -CONFIG_DRM_BOCHS                                note<LP#1378648>
>  CONFIG_DRM_STI                                  note<LP#1398458>
>  
>  # Menu: Device Drivers >> Graphics support >> Direct Rendering Manager (XFree86 4.1.0 and higher DRI support) >> AMD GPU
> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
> index bdece4b92c89..e6e1f5d34ea1 100644
> --- a/debian.master/config/config.common.ubuntu
> +++ b/debian.master/config/config.common.ubuntu
> @@ -2315,7 +2315,7 @@ CONFIG_DRM_ARM=y
>  CONFIG_DRM_ARMADA=m
>  CONFIG_DRM_AST=m
>  CONFIG_DRM_ATMEL_HLCDC=m
> -# CONFIG_DRM_BOCHS is not set
> +CONFIG_DRM_BOCHS=m
>  CONFIG_DRM_BRIDGE=y
>  CONFIG_DRM_CDNS_DSI=m
>  CONFIG_DRM_CIRRUS_QEMU=m
> 

Hi Timo,

Which is the release target for this patch?


Thanks,
Kleber
Timo Aaltonen Oct. 11, 2018, 1:57 p.m. UTC | #4
On 11.10.2018 14:52, Kleber Souza wrote:
> On 10/03/18 13:22, Timo Aaltonen wrote:
>> From: Timo Aaltonen <timo.aaltonen@canonical.com>
>>
>> BugLink: https://bugs.launchpad.net/bugs/1795857
>>
>> This is required for xserver to work on QEMU with dummy VGA.
>>
>> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
>> ---
>>  debian.master/config/annotations          | 2 --
>>  debian.master/config/config.common.ubuntu | 2 +-
>>  2 files changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
>> index 93813c874627..4d11ae71c875 100644
>> --- a/debian.master/config/annotations
>> +++ b/debian.master/config/annotations
>> @@ -1822,7 +1822,6 @@ CONFIG_DRM_SHMOBILE                             policy<{'armhf': 'm'}>
>>  CONFIG_DRM_OMAP                                 policy<{'armhf': 'n'}>
>>  CONFIG_DRM_TILCDC                               policy<{'armhf': 'm'}>
>>  CONFIG_DRM_QXL                                  policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>> -CONFIG_DRM_BOCHS                                policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>>  CONFIG_DRM_VIRTIO_GPU                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>>  CONFIG_DRM_FSL_DCU                              policy<{'armhf': 'm'}>
>>  CONFIG_DRM_TEGRA                                policy<{'armhf-generic': 'm'}>
>> @@ -1848,7 +1847,6 @@ CONFIG_DRM_PL111                                policy<{'arm64': 'm', 'armhf': '
>>  CONFIG_DRM_TVE200                               policy<{'armhf': 'm'}>
>>  #
>>  CONFIG_DRM_MGAG200                              note<LP#1693337>
>> -CONFIG_DRM_BOCHS                                note<LP#1378648>
>>  CONFIG_DRM_STI                                  note<LP#1398458>
>>  
>>  # Menu: Device Drivers >> Graphics support >> Direct Rendering Manager (XFree86 4.1.0 and higher DRI support) >> AMD GPU
>> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
>> index bdece4b92c89..e6e1f5d34ea1 100644
>> --- a/debian.master/config/config.common.ubuntu
>> +++ b/debian.master/config/config.common.ubuntu
>> @@ -2315,7 +2315,7 @@ CONFIG_DRM_ARM=y
>>  CONFIG_DRM_ARMADA=m
>>  CONFIG_DRM_AST=m
>>  CONFIG_DRM_ATMEL_HLCDC=m
>> -# CONFIG_DRM_BOCHS is not set
>> +CONFIG_DRM_BOCHS=m
>>  CONFIG_DRM_BRIDGE=y
>>  CONFIG_DRM_CDNS_DSI=m
>>  CONFIG_DRM_CIRRUS_QEMU=m
>>
> 
> Hi Timo,
> 
> Which is the release target for this patch?

It's for DD at this point.
Timo Aaltonen Nov. 13, 2018, 11:19 a.m. UTC | #5
On 11.10.2018 16.57, Timo Aaltonen wrote:
> On 11.10.2018 14:52, Kleber Souza wrote:
>> On 10/03/18 13:22, Timo Aaltonen wrote:
>>> From: Timo Aaltonen <timo.aaltonen@canonical.com>
>>>
>>> BugLink: https://bugs.launchpad.net/bugs/1795857
>>>
>>> This is required for xserver to work on QEMU with dummy VGA.
>>>
>>> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
>>> ---
>>>  debian.master/config/annotations          | 2 --
>>>  debian.master/config/config.common.ubuntu | 2 +-
>>>  2 files changed, 1 insertion(+), 3 deletions(-)
>>>
>>> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
>>> index 93813c874627..4d11ae71c875 100644
>>> --- a/debian.master/config/annotations
>>> +++ b/debian.master/config/annotations
>>> @@ -1822,7 +1822,6 @@ CONFIG_DRM_SHMOBILE                             policy<{'armhf': 'm'}>
>>>  CONFIG_DRM_OMAP                                 policy<{'armhf': 'n'}>
>>>  CONFIG_DRM_TILCDC                               policy<{'armhf': 'm'}>
>>>  CONFIG_DRM_QXL                                  policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>>> -CONFIG_DRM_BOCHS                                policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
>>>  CONFIG_DRM_VIRTIO_GPU                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
>>>  CONFIG_DRM_FSL_DCU                              policy<{'armhf': 'm'}>
>>>  CONFIG_DRM_TEGRA                                policy<{'armhf-generic': 'm'}>
>>> @@ -1848,7 +1847,6 @@ CONFIG_DRM_PL111                                policy<{'arm64': 'm', 'armhf': '
>>>  CONFIG_DRM_TVE200                               policy<{'armhf': 'm'}>
>>>  #
>>>  CONFIG_DRM_MGAG200                              note<LP#1693337>
>>> -CONFIG_DRM_BOCHS                                note<LP#1378648>
>>>  CONFIG_DRM_STI                                  note<LP#1398458>
>>>  
>>>  # Menu: Device Drivers >> Graphics support >> Direct Rendering Manager (XFree86 4.1.0 and higher DRI support) >> AMD GPU
>>> diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
>>> index bdece4b92c89..e6e1f5d34ea1 100644
>>> --- a/debian.master/config/config.common.ubuntu
>>> +++ b/debian.master/config/config.common.ubuntu
>>> @@ -2315,7 +2315,7 @@ CONFIG_DRM_ARM=y
>>>  CONFIG_DRM_ARMADA=m
>>>  CONFIG_DRM_AST=m
>>>  CONFIG_DRM_ATMEL_HLCDC=m
>>> -# CONFIG_DRM_BOCHS is not set
>>> +CONFIG_DRM_BOCHS=m
>>>  CONFIG_DRM_BRIDGE=y
>>>  CONFIG_DRM_CDNS_DSI=m
>>>  CONFIG_DRM_CIRRUS_QEMU=m
>>>
>>
>> Hi Timo,
>>
>> Which is the release target for this patch?
> 
> It's for DD at this point.

Ping, I don't think this has been applied anywhere yet? Fixed the subject..
Thadeu Lima de Souza Cascardo Nov. 13, 2018, 12:01 p.m. UTC | #6
On Wed, Oct 03, 2018 at 11:32:25PM +0300, Timo Aaltonen wrote:
> On 03.10.2018 15:51, Seth Forshee wrote:
> > On Wed, Oct 03, 2018 at 02:22:20PM +0300, Timo Aaltonen wrote:
> >> From: Timo Aaltonen <timo.aaltonen@canonical.com>
> >>
> >> BugLink: https://bugs.launchpad.net/bugs/1795857
> >>
> >> This is required for xserver to work on QEMU with dummy VGA.
> >>
> >> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
> > 
> > Can you confirm that the problem from LP #1378648 has been fixed when
> > this option is turned on?
> 
> Nope, X is still started too early for this to help. But adding this
> could be targeted for next release.
> 
> 
> -- 
> t

This is an old bug (from 14.10), and we don't know whether it has been
addressed. I will add a card so I can test it on a ppc64el system, and get back
to it by next week.

Cascardo.
Timo Aaltonen March 7, 2019, 7:47 a.m. UTC | #7
On 13.11.2018 14.01, Thadeu Lima de Souza Cascardo wrote:
> On Wed, Oct 03, 2018 at 11:32:25PM +0300, Timo Aaltonen wrote:
>> On 03.10.2018 15:51, Seth Forshee wrote:
>>> On Wed, Oct 03, 2018 at 02:22:20PM +0300, Timo Aaltonen wrote:
>>>> From: Timo Aaltonen <timo.aaltonen@canonical.com>
>>>>
>>>> BugLink: https://bugs.launchpad.net/bugs/1795857
>>>>
>>>> This is required for xserver to work on QEMU with dummy VGA.
>>>>
>>>> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
>>>
>>> Can you confirm that the problem from LP #1378648 has been fixed when
>>> this option is turned on?
>>
>> Nope, X is still started too early for this to help. But adding this
>> could be targeted for next release.
>>
>>
>> -- 
>> t
> 
> This is an old bug (from 14.10), and we don't know whether it has been
> addressed. I will add a card so I can test it on a ppc64el system, and get back
> to it by next week.

Hi, did you have a chance to test? :)
Thadeu Lima de Souza Cascardo March 8, 2019, 3:32 p.m. UTC | #8
On Thu, Mar 07, 2019 at 09:47:12AM +0200, Timo Aaltonen wrote:
> On 13.11.2018 14.01, Thadeu Lima de Souza Cascardo wrote:
> > On Wed, Oct 03, 2018 at 11:32:25PM +0300, Timo Aaltonen wrote:
> >> On 03.10.2018 15:51, Seth Forshee wrote:
> >>> On Wed, Oct 03, 2018 at 02:22:20PM +0300, Timo Aaltonen wrote:
> >>>> From: Timo Aaltonen <timo.aaltonen@canonical.com>
> >>>>
> >>>> BugLink: https://bugs.launchpad.net/bugs/1795857
> >>>>
> >>>> This is required for xserver to work on QEMU with dummy VGA.
> >>>>
> >>>> Signed-off-by: Timo Aaltonen <timo.aaltonen@canonical.com>
> >>>
> >>> Can you confirm that the problem from LP #1378648 has been fixed when
> >>> this option is turned on?
> >>
> >> Nope, X is still started too early for this to help. But adding this
> >> could be targeted for next release.
> >>
> >>
> >> -- 
> >> t
> > 
> > This is an old bug (from 14.10), and we don't know whether it has been
> > addressed. I will add a card so I can test it on a ppc64el system, and get back
> > to it by next week.
> 
> Hi, did you have a chance to test? :)
> 

It seems to be working fine. Loaded bochs-drm on a ppc64el KVM guest with a
bochs card and it loaded/probe OK. VNC worked fine too.

Cascardo.

> 
> -- 
> t
diff mbox series

Patch

diff --git a/debian.master/config/annotations b/debian.master/config/annotations
index 93813c874627..4d11ae71c875 100644
--- a/debian.master/config/annotations
+++ b/debian.master/config/annotations
@@ -1822,7 +1822,6 @@  CONFIG_DRM_SHMOBILE                             policy<{'armhf': 'm'}>
 CONFIG_DRM_OMAP                                 policy<{'armhf': 'n'}>
 CONFIG_DRM_TILCDC                               policy<{'armhf': 'm'}>
 CONFIG_DRM_QXL                                  policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
-CONFIG_DRM_BOCHS                                policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'i386': 'n', 'ppc64el': 'n'}>
 CONFIG_DRM_VIRTIO_GPU                           policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
 CONFIG_DRM_FSL_DCU                              policy<{'armhf': 'm'}>
 CONFIG_DRM_TEGRA                                policy<{'armhf-generic': 'm'}>
@@ -1848,7 +1847,6 @@  CONFIG_DRM_PL111                                policy<{'arm64': 'm', 'armhf': '
 CONFIG_DRM_TVE200                               policy<{'armhf': 'm'}>
 #
 CONFIG_DRM_MGAG200                              note<LP#1693337>
-CONFIG_DRM_BOCHS                                note<LP#1378648>
 CONFIG_DRM_STI                                  note<LP#1398458>
 
 # Menu: Device Drivers >> Graphics support >> Direct Rendering Manager (XFree86 4.1.0 and higher DRI support) >> AMD GPU
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index bdece4b92c89..e6e1f5d34ea1 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -2315,7 +2315,7 @@  CONFIG_DRM_ARM=y
 CONFIG_DRM_ARMADA=m
 CONFIG_DRM_AST=m
 CONFIG_DRM_ATMEL_HLCDC=m
-# CONFIG_DRM_BOCHS is not set
+CONFIG_DRM_BOCHS=m
 CONFIG_DRM_BRIDGE=y
 CONFIG_DRM_CDNS_DSI=m
 CONFIG_DRM_CIRRUS_QEMU=m