* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
[not found] ` <20121116184603.GA13453@morn.localdomain>
@ 2012-11-19 10:22 ` Alexander Graf
2012-11-19 11:33 ` Gleb Natapov
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2012-11-19 10:22 UTC (permalink / raw)
To: Kevin O'Connor
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Gerd Hoffmann
On 16.11.2012, at 19:46, Kevin O'Connor wrote:
> On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
>> ping
>>
>> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
>>> Updated _CRS method for HPET, bringing it in line with the way it is
>>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
>>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
>>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
>>>
>>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
>
> I'm okay with the patch. I'm looking to see an Ack from one of the
> kvm/qemu developers. (I saw Gerd was okay with the last version of
> the patch.)
IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
Alex
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
2012-11-19 10:22 ` [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method Alexander Graf
@ 2012-11-19 11:33 ` Gleb Natapov
2012-11-19 11:35 ` Alexander Graf
0 siblings, 1 reply; 6+ messages in thread
From: Gleb Natapov @ 2012-11-19 11:33 UTC (permalink / raw)
To: Alexander Graf
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Kevin O'Connor, Gerd Hoffmann
On Mon, Nov 19, 2012 at 11:22:48AM +0100, Alexander Graf wrote:
>
> On 16.11.2012, at 19:46, Kevin O'Connor wrote:
>
> > On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
> >> ping
> >>
> >> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
> >>> Updated _CRS method for HPET, bringing it in line with the way it is
> >>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
> >>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
> >>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
> >>>
> >>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
> >
> > I'm okay with the patch. I'm looking to see an Ack from one of the
> > kvm/qemu developers. (I saw Gerd was okay with the last version of
> > the patch.)
>
> IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
>
> Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
>
I think you are misremembering. Git shows that HPET entry was taken from
pcbios code and my old pcbios git tree says that the dsdt code was
contributed by Beth Kon and it is the same as we have it now in SeaBIOS.
The patch looks OK.
--
Gleb.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
2012-11-19 11:33 ` Gleb Natapov
@ 2012-11-19 11:35 ` Alexander Graf
2012-11-19 11:51 ` Gleb Natapov
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2012-11-19 11:35 UTC (permalink / raw)
To: Gleb Natapov
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Kevin O'Connor, Gerd Hoffmann
On 19.11.2012, at 12:33, Gleb Natapov wrote:
> On Mon, Nov 19, 2012 at 11:22:48AM +0100, Alexander Graf wrote:
>>
>> On 16.11.2012, at 19:46, Kevin O'Connor wrote:
>>
>>> On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
>>>> ping
>>>>
>>>> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
>>>>> Updated _CRS method for HPET, bringing it in line with the way it is
>>>>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
>>>>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
>>>>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
>>>>>
>>>>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
>>>
>>> I'm okay with the patch. I'm looking to see an Ack from one of the
>>> kvm/qemu developers. (I saw Gerd was okay with the last version of
>>> the patch.)
>>
>> IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
>>
>> Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
>>
> I think you are misremembering. Git shows that HPET entry was taken from
> pcbios code and my old pcbios git tree says that the dsdt code was
> contributed by Beth Kon and it is the same as we have it now in SeaBIOS.
Hrm. I'm 100% sure that from my code (which Beth based on) to the code that we have, someone had completely rewritten the DSDT entry. Either way, not complaining as long as we get it sorted out eventually :).
> The patch looks OK.
Mind to make this a formal acked-by? :)
Alex
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
2012-11-19 11:35 ` Alexander Graf
@ 2012-11-19 11:51 ` Gleb Natapov
2012-11-19 12:00 ` Alexander Graf
0 siblings, 1 reply; 6+ messages in thread
From: Gleb Natapov @ 2012-11-19 11:51 UTC (permalink / raw)
To: Alexander Graf
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Kevin O'Connor, Gerd Hoffmann
On Mon, Nov 19, 2012 at 12:35:47PM +0100, Alexander Graf wrote:
>
> On 19.11.2012, at 12:33, Gleb Natapov wrote:
>
> > On Mon, Nov 19, 2012 at 11:22:48AM +0100, Alexander Graf wrote:
> >>
> >> On 16.11.2012, at 19:46, Kevin O'Connor wrote:
> >>
> >>> On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
> >>>> ping
> >>>>
> >>>> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
> >>>>> Updated _CRS method for HPET, bringing it in line with the way it is
> >>>>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
> >>>>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
> >>>>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
> >>>>>
> >>>>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
> >>>
> >>> I'm okay with the patch. I'm looking to see an Ack from one of the
> >>> kvm/qemu developers. (I saw Gerd was okay with the last version of
> >>> the patch.)
> >>
> >> IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
> >>
> >> Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
> >>
> > I think you are misremembering. Git shows that HPET entry was taken from
> > pcbios code and my old pcbios git tree says that the dsdt code was
> > contributed by Beth Kon and it is the same as we have it now in SeaBIOS.
>
> Hrm. I'm 100% sure that from my code (which Beth based on) to the code that we have, someone had completely rewritten the DSDT entry. Either way, not complaining as long as we get it sorted out eventually :).
>
Here is Beth's commit in bochs :)
http://0x0badc0.de/gitweb?p=bochs/.git;a=commitdiff;h=874040e817e49811c30405d7c5d14b4c42161ca1;hp=1d2bac352688c75e8fdf8010f012b8ec85ba68d1
I added _STA logic in SeaBIOS but otherwise the code is the same.
> > The patch looks OK.
>
> Mind to make this a formal acked-by? :)
>
Sure, will do.
--
Gleb.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
2012-11-19 11:51 ` Gleb Natapov
@ 2012-11-19 12:00 ` Alexander Graf
2012-11-19 12:11 ` Gleb Natapov
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2012-11-19 12:00 UTC (permalink / raw)
To: Gleb Natapov
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Kevin O'Connor, Gerd Hoffmann
On 19.11.2012, at 12:51, Gleb Natapov wrote:
> On Mon, Nov 19, 2012 at 12:35:47PM +0100, Alexander Graf wrote:
>>
>> On 19.11.2012, at 12:33, Gleb Natapov wrote:
>>
>>> On Mon, Nov 19, 2012 at 11:22:48AM +0100, Alexander Graf wrote:
>>>>
>>>> On 16.11.2012, at 19:46, Kevin O'Connor wrote:
>>>>
>>>>> On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
>>>>>> ping
>>>>>>
>>>>>> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
>>>>>>> Updated _CRS method for HPET, bringing it in line with the way it is
>>>>>>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
>>>>>>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
>>>>>>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
>>>>>>>
>>>>>>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
>>>>>
>>>>> I'm okay with the patch. I'm looking to see an Ack from one of the
>>>>> kvm/qemu developers. (I saw Gerd was okay with the last version of
>>>>> the patch.)
>>>>
>>>> IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
>>>>
>>>> Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
>>>>
>>> I think you are misremembering. Git shows that HPET entry was taken from
>>> pcbios code and my old pcbios git tree says that the dsdt code was
>>> contributed by Beth Kon and it is the same as we have it now in SeaBIOS.
>>
>> Hrm. I'm 100% sure that from my code (which Beth based on) to the code that we have, someone had completely rewritten the DSDT entry. Either way, not complaining as long as we get it sorted out eventually :).
>>
> Here is Beth's commit in bochs :)
> http://0x0badc0.de/gitweb?p=bochs/.git;a=commitdiff;h=874040e817e49811c30405d7c5d14b4c42161ca1;hp=1d2bac352688c75e8fdf8010f012b8ec85ba68d1
Ah, apparently Beth's DSDT bits were a rewrite done by Ryan. My original version looked like this:
http://lists.gnu.org/archive/html/qemu-devel/2008-01/msg00180.html
but obviously broke quite a few things along the way ;). Also, I do remember that I extracted the HPET bits from there and submitted them individually, but I might really just be misremembering and it's not like that makes any difference.
>
> I added _STA logic in SeaBIOS but otherwise the code is the same.
>
>>> The patch looks OK.
>>
>> Mind to make this a formal acked-by? :)
>>
> Sure, will do.
Just reply with the line ;)
Alex
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method
2012-11-19 12:00 ` Alexander Graf
@ 2012-11-19 12:11 ` Gleb Natapov
0 siblings, 0 replies; 6+ messages in thread
From: Gleb Natapov @ 2012-11-19 12:11 UTC (permalink / raw)
To: Alexander Graf
Cc: René Rebe, Marcelo Tosatti, seabios, qemu-devel qemu-devel,
Gabriel L. Somlo, Kevin O'Connor, Gerd Hoffmann
On Mon, Nov 19, 2012 at 01:00:56PM +0100, Alexander Graf wrote:
>
> On 19.11.2012, at 12:51, Gleb Natapov wrote:
>
> > On Mon, Nov 19, 2012 at 12:35:47PM +0100, Alexander Graf wrote:
> >>
> >> On 19.11.2012, at 12:33, Gleb Natapov wrote:
> >>
> >>> On Mon, Nov 19, 2012 at 11:22:48AM +0100, Alexander Graf wrote:
> >>>>
> >>>> On 16.11.2012, at 19:46, Kevin O'Connor wrote:
> >>>>
> >>>>> On Fri, Nov 16, 2012 at 01:02:18PM -0500, Gabriel L. Somlo wrote:
> >>>>>> ping
> >>>>>>
> >>>>>> On Thu, Nov 08, 2012 at 12:35:17PM -0500, Gabriel L. Somlo wrote:
> >>>>>>> Updated _CRS method for HPET, bringing it in line with the way it is
> >>>>>>> presented on recent hardware (e.g. Dell Latitude D630, MacPro5,1, etc);
> >>>>>>> Allows it to be detected and utilized from Mac OS X; Also tested OK on
> >>>>>>> Linux (F16 64-bit install DVD) and Windows (Win7 64-bit install DVD).
> >>>>>>>
> >>>>>>> Signed-off-by: Gabriel Somlo <somlo@cmu.edu>
> >>>>>
> >>>>> I'm okay with the patch. I'm looking to see an Ack from one of the
> >>>>> kvm/qemu developers. (I saw Gerd was okay with the last version of
> >>>>> the patch.)
> >>>>
> >>>> IIRC Marcelo was the one changing the DSDT code from what this patch produces to what we have today. My initial HPET DSDT entry was also copied from real hardware.
> >>>>
> >>>> Marcelo, any memory on this? :) Keep in mind I might be misremembering - maybe it was someone else after all ;).
> >>>>
> >>> I think you are misremembering. Git shows that HPET entry was taken from
> >>> pcbios code and my old pcbios git tree says that the dsdt code was
> >>> contributed by Beth Kon and it is the same as we have it now in SeaBIOS.
> >>
> >> Hrm. I'm 100% sure that from my code (which Beth based on) to the code that we have, someone had completely rewritten the DSDT entry. Either way, not complaining as long as we get it sorted out eventually :).
> >>
> > Here is Beth's commit in bochs :)
> > http://0x0badc0.de/gitweb?p=bochs/.git;a=commitdiff;h=874040e817e49811c30405d7c5d14b4c42161ca1;hp=1d2bac352688c75e8fdf8010f012b8ec85ba68d1
>
> Ah, apparently Beth's DSDT bits were a rewrite done by Ryan. My original version looked like this:
>
> http://lists.gnu.org/archive/html/qemu-devel/2008-01/msg00180.html
>
> but obviously broke quite a few things along the way ;). Also, I do remember that I extracted the HPET bits from there and submitted them individually, but I might really just be misremembering and it's not like that makes any difference.
>
Just wanted to make sure that we do not revert someone's change without
understanding why it was done in the first place.
> >
> > I added _STA logic in SeaBIOS but otherwise the code is the same.
> >
> >>> The patch looks OK.
> >>
> >> Mind to make this a formal acked-by? :)
> >>
> > Sure, will do.
>
> Just reply with the line ;)
>
Did it in a most formal way by answering to the patch itself :)
--
Gleb.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-11-19 12:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20121108173516.GA2131@hedwig.ini.cmu.edu>
[not found] ` <20121116180217.GD10606@hedwig.ini.cmu.edu>
[not found] ` <20121116184603.GA13453@morn.localdomain>
2012-11-19 10:22 ` [Qemu-devel] [PATCH v3] DSDT: Fix HPET _CRS Method Alexander Graf
2012-11-19 11:33 ` Gleb Natapov
2012-11-19 11:35 ` Alexander Graf
2012-11-19 11:51 ` Gleb Natapov
2012-11-19 12:00 ` Alexander Graf
2012-11-19 12:11 ` Gleb Natapov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).