Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-07-03 Thread David Hendricks
Necromancing this thread...

Sage has a patch to *optionally* use a serial console log in
board_status.sh: http://review.coreboot.org/#/c/6094/

Earlier objections to such an approach seemed to stem from either:
- Desire to use cbmem console instead. A fine idea, but on some platforms
(especially those which use AGESA) a lot of information gets spit out to
the console before cbmem is available. Re-factoring to make cbmem init
happen earlier is unfeasible AFAICT.

- Avoid confusing cbmem console log and other logs. This can be easily
solved by using a different filename. I personally think it's best to only
upload one log (whichever is most useful) and avoid polluting the web UI
with redundant files. But I could live with multiple console logs if others
feel strongly.

Seeing as how only a small handful of people currently actually use this
utility anyway, I'm inclined to think it's best to make the utility more
useful for a major coreboot contributor get some more status reports
uploaded.

Thoughts?

On Wed, Feb 19, 2014 at 3:14 PM, Kyösti Mälkki kyosti.mal...@gmail.com
wrote:

 On 02/20/2014 12:56 AM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:



 You do not get raminit debug output printed in ramstage.

 Unfortunately, the case of incompatible DIMMs seems to be common one
 with recent AGESA ports so information from romstage what DIMMs have
 worked is actually relevant.

 Just read this data from registers and print it.




 Neither of us has probably looked closely what details we are missing from
 AGESA romstages. I very much doubt it would be just static register
 configuration on memory controllers that could be dumped afterwards.

 AMD has an amount of SMP init going on in romstage along with possibly
 multiple logical CPUs using CAR etc.




-- 
David Hendricks (dhendrix)
Systems Software Engineer, Google Inc.
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-07-03 Thread ron minnich
On Thu, Jul 3, 2014 at 4:18 PM, David Hendricks dhend...@google.com wrote:

 Necromancing this thread...

 Sage has a patch to *optionally* use a serial console log in
 board_status.sh: http://review.coreboot.org/#/c/6094/

 Earlier objections to such an approach seemed to stem from either:
 - Desire to use cbmem console instead. A fine idea, but on some platforms
 (especially those which use AGESA) a lot of information gets spit out to
 the console before cbmem is available. Re-factoring to make cbmem init
 happen earlier is unfeasible AFAICT.

 - Avoid confusing cbmem console log and other logs. This can be easily
 solved by using a different filename. I personally think it's best to only
 upload one log (whichever is most useful) and avoid polluting the web UI
 with redundant files. But I could live with multiple console logs if others
 feel strongly.

 Seeing as how only a small handful of people currently actually use this
 utility anyway, I'm inclined to think it's best to make the utility more
 useful for a major coreboot contributor get some more status reports
 uploaded.



Yes, please, let's just get this done. I think Sage's arguments make lots
of sense.

ron
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Paul Menzel
Am Mittwoch, den 19.02.2014, 00:47 +0100 schrieb Vladimir 'φ-coder/phcoder' 
Serbinenko:
 On 19.02.2014 00:18, Paul Menzel wrote:
But in general I think I agree with Vladimir. CBMEM console should be
supported and if not then that should be fixed.
  I also agree, but it’ll take more time and the above is a good
  work-around for the mean time.
 Strongly disagree workarounds are like glue: they stick around. This one
 is one that will stick around once implemented. It's better to avoid
 workarounds if sane solution is within reach.
 In this case you still haven't even named a single board where CBMEMC
 doesn't work.

CBMEM console does not work for romstage on all AMD based boards. Ask
Rudolf and Kyösti for more details, but as far as I understood them, the
solution is *not* easy and not within reach at all.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 19.02.2014 23:03, Paul Menzel wrote:
 Am Mittwoch, den 19.02.2014, 00:47 +0100 schrieb Vladimir 'φ-coder/phcoder' 
 Serbinenko:
 On 19.02.2014 00:18, Paul Menzel wrote:
 But in general I think I agree with Vladimir. CBMEM console should be
 supported and if not then that should be fixed.
 I also agree, but it’ll take more time and the above is a good
 work-around for the mean time.
 Strongly disagree workarounds are like glue: they stick around. This one
 is one that will stick around once implemented. It's better to avoid
 workarounds if sane solution is within reach.
 In this case you still haven't even named a single board where CBMEMC
 doesn't work.
 
 CBMEM console does not work for romstage on all AMD based boards. Ask
 Rudolf and Kyösti for more details, but as far as I understood them, the
 solution is *not* easy and not within reach at all.
 
As I already told the info from romstage is likely of minor importance
if the board boots. And if it doesn't, well no board status. If any of
the info from romstage is relevant it can be printed in ramstage as well
 
 Thanks,
 
 Paul
 
 
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread ron minnich
Paul, your idea is a non-starter, so let's wind this conversation up, ok?

ron
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Kyösti Mälkki

On 02/20/2014 12:28 AM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:

On 19.02.2014 23:03, Paul Menzel wrote:

Am Mittwoch, den 19.02.2014, 00:47 +0100 schrieb Vladimir 'φ-coder/phcoder' 
Serbinenko:

On 19.02.2014 00:18, Paul Menzel wrote:

But in general I think I agree with Vladimir. CBMEM console should be
supported and if not then that should be fixed.

I also agree, but it’ll take more time and the above is a good
work-around for the mean time.

Strongly disagree workarounds are like glue: they stick around. This one
is one that will stick around once implemented. It's better to avoid
workarounds if sane solution is within reach.
In this case you still haven't even named a single board where CBMEMC
doesn't work.


CBMEM console does not work for romstage on all AMD based boards. Ask
Rudolf and Kyösti for more details, but as far as I understood them, the
solution is *not* easy and not within reach at all.


As I already told the info from romstage is likely of minor importance
if the board boots. And if it doesn't, well no board status. If any of
the info from romstage is relevant it can be printed in ramstage as well


You do not get raminit debug output printed in ramstage.

Unfortunately, the case of incompatible DIMMs seems to be common one 
with recent AGESA ports so information from romstage what DIMMs have 
worked is actually relevant.


Kyösti


--
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Vladimir 'φ-coder/phcoder' Serbinenko

 
 You do not get raminit debug output printed in ramstage.
 
 Unfortunately, the case of incompatible DIMMs seems to be common one
 with recent AGESA ports so information from romstage what DIMMs have
 worked is actually relevant.
Just read this data from registers and print it.
 
 Kyösti
 
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Paul Menzel
Am Mittwoch, den 19.02.2014, 23:56 +0100 schrieb Vladimir 'φ-coder/phcoder' 
Serbinenko:
  
  You do not get raminit debug output printed in ramstage.
  
  Unfortunately, the case of incompatible DIMMs seems to be common one
  with recent AGESA ports so information from romstage what DIMMs have
  worked is actually relevant.

 Just read this data from registers and print it.

Until this is done, such data is lost and it is not nice to ask users to
rerun stuff again with such a patch. So please, could we just decide on
a name for the serial/USB debug log file and be done with this? Not a
lot of people are going to do this anyway.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread Kyösti Mälkki

On 02/20/2014 12:56 AM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:




You do not get raminit debug output printed in ramstage.

Unfortunately, the case of incompatible DIMMs seems to be common one
with recent AGESA ports so information from romstage what DIMMs have
worked is actually relevant.

Just read this data from registers and print it.






Neither of us has probably looked closely what details we are missing 
from AGESA romstages. I very much doubt it would be just static register 
configuration on memory controllers that could be dumped afterwards.


AMD has an amount of SMP init going on in romstage along with possibly 
multiple logical CPUs using CAR etc.


Kyösti

--
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-19 Thread mrnuke
On Thursday, February 20, 2014 12:11:58 AM Paul Menzel wrote:
 Until this is done, such data is lost and it is not nice to ask users to
 rerun stuff again with such a patch. So please, could we just decide on
 a name for the serial/USB debug log file and be done with this? Not a
 lot of people are going to do this anyway.
 
Paul! It's been decided! It's dead! Drop it!

Alex


-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot


Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-18 Thread Paul Menzel
Am Dienstag, den 11.02.2014, 00:04 +0100 schrieb Vladimir 'φ-coder/phcoder' 
Serbinenko:
 On 10.02.2014 23:47, David Hendricks wrote:
  On Sun, Feb 9, 2014 at 4:50 AM, Paul Menzel wrote:

  currently no coreboot messages are stored for boards not supporting
  CBMEM console (or where this option is disabled (currently by default))
  or no coreboot *romstage* messages are stored for boards, where the data
  cannot be preserved (passed to ramstage).
  
  Using the serial (or USB) console all these messages can be captured
  with no problem, so I propose to just add these captured messages into
  the file `serial_console.txt`. Of course this file probably contains
  also the payload and (Linux) kernel log, but I think that is fine.
  
  SeaBIOS’ `readserial.py` should be used for capturing the messages as it
  adds time stamps.
  
  Scripting this is going to be hard, as the log is captured on a
  different system. So for now I propose to add it manually.
  
  
  I don't think the script itself should be responsible for collecting
  serial output. Instead, how about adding an argument to override the
  default behavior of running cbmem -c on the target so that the user
  can pass in a filename? The user will simply capture the serial output
  using whatever tool they like, dump the output to a text file, and run
  the script with an argument to use the file instead of calling cbmem
  -c. Here is a proof-of-concept: http://review.coreboot.org/#/c/5191 .

David, thanks a lot for implementing this.

 This requires user to do right manipulations. While keyboard and chair
 are usually fine, the space between them exhibits strong bug-inducing
 properties. The idea of the script is to reduce a possibility of user
 error creating strange reports. In this case the common error I expect is
 using a stale file from some other version. It's a particularly nasty one
 as at first glance in may look fine but would be almost useless to track
 how details changed from one submit to the next. If we let user supply
 files at all, it should be added to report, not replace files, and it
 should have some prefix to clearly indicate that user was involved in
 creating them. E.g. user_serial_log.txt

I agree with Vladimir that the file should be put there with a separate
name. I am not sure about a common name though, as it could be also
captured using a USB debug device or spkmodem(?).

  But in general I think I agree with Vladimir. CBMEM console should be
  supported and if not then that should be fixed.

I also agree, but it’ll take more time and the above is a good
work-around for the mean time.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-18 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 19.02.2014 00:18, Paul Menzel wrote:
   But in general I think I agree with Vladimir. CBMEM console should be
   supported and if not then that should be fixed.
 I also agree, but it’ll take more time and the above is a good
 work-around for the mean time.
Strongly disagree workarounds are like glue: they stick around. This one
is one that will stick around once implemented. It's better to avoid
workarounds if sane solution is within reach.
In this case you still haven't even named a single board where CBMEMC
doesn't work.



signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-18 Thread ron minnich
I 'm with Vladimir on this one.

ron

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot


Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-10 Thread David Hendricks
On Sun, Feb 9, 2014 at 4:50 AM, Paul Menzel 
paulepan...@users.sourceforge.net wrote:

 Dear coreboot folks,


 currently no coreboot messages are stored for boards not supporting
 CBMEM console (or where this option is disabled (currently by default))
 or no coreboot *romstage* messages are stored for boards, where the data
 cannot be preserved (passed to ramstage).

 Using the serial (or USB) console all these messages can be captured
 with no problem, so I propose to just add these captured messages into
 the file `serial_console.txt`. Of course this file probably contains
 also the payload and (Linux) kernel log, but I think that is fine.

 SeaBIOS’ `readserial.py` should be used for capturing the messages as it
 adds time stamps.

 Scripting this is going to be hard, as the log is captured on a
 different system. So for now I propose to add it manually.


I don't think the script itself should be responsible for collecting serial
output. Instead, how about adding an argument to override the default
behavior of running cbmem -c on the target so that the user can pass in a
filename? The user will simply capture the serial output using whatever
tool they like, dump the output to a text file, and run the script with an
argument to use the file instead of calling cbmem -c. Here is a
proof-of-concept: http://review.coreboot.org/#/c/5191 .

But in general I think I agree with Vladimir. CBMEM console should be
supported and if not then that should be fixed.

-- 
David Hendricks (dhendrix)
Systems Software Engineer, Google Inc.
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-10 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 10.02.2014 23:47, David Hendricks wrote:
 On Sun, Feb 9, 2014 at 4:50 AM, Paul Menzel
 paulepan...@users.sourceforge.net
 mailto:paulepan...@users.sourceforge.net wrote:
 
 Dear coreboot folks,
 
 
 currently no coreboot messages are stored for boards not supporting
 CBMEM console (or where this option is disabled (currently by default))
 or no coreboot *romstage* messages are stored for boards, where the data
 cannot be preserved (passed to ramstage).
 
 Using the serial (or USB) console all these messages can be captured
 with no problem, so I propose to just add these captured messages into
 the file `serial_console.txt`. Of course this file probably contains
 also the payload and (Linux) kernel log, but I think that is fine.
 
 SeaBIOS’ `readserial.py` should be used for capturing the messages as it
 adds time stamps.
 
 Scripting this is going to be hard, as the log is captured on a
 different system. So for now I propose to add it manually.
 
 
 I don't think the script itself should be responsible for collecting
 serial output. Instead, how about adding an argument to override the
 default behavior of running cbmem -c on the target so that the user
 can pass in a filename? The user will simply capture the serial output
 using whatever tool they like, dump the output to a text file, and run
 the script with an argument to use the file instead of calling cbmem
 -c. Here is a proof-of-concept: http://review.coreboot.org/#/c/5191 .
 
This requires user to do right manipulations. While keyboard and chair
are usually fine, the space between them exhibits strong bug-inducing
properties. The idea of the script is to reduce a possibility of user
error creating strange reports. In this case the common erro I expect is
using a stale file fom some other version. It's a particularly nasty one
as at first glance in may look fine but would be almost useless to track
how details changed from one submit to the next. If we let user supply
files at all, it should be added to report, not replace files, and it
should have some prefix to clearly indicate that user was involved in
creating them. E.g. user_serial_log.txt
 But in general I think I agree with Vladimir. CBMEM console should be
 supported and if not then that should be fixed.
 
 -- 
 David Hendricks (dhendrix)
 Systems Software Engineer, Google Inc.
 
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-10 Thread mrnuke
On Tuesday, February 11, 2014 12:04:33 AM Vladimir 'φ-coder/phcoder' 
Serbinenko wrote:
 If we let user supply
 files at all, it should be added to report, not replace files, and it
 should have some prefix to clearly indicate that user was involved in
 creating them. E.g. user_serial_log.txt
 
There is a point at which the information becomes too much. What is really 
relevant, in my opinion, are the following three points:

1. Is it an unmodified commit in master (AKA, not -dirty).
2. Does it boot?
3. What does not work, if any ?

Point 1 and 2 are answered by cbmem -c | head. It answers point 1 with the 
first line of output, and point 2 by the fact that getting anything from cbmem 
means you've already booted your system.

Point 3 is more complicated, and it seems it is because of this point that we 
want to collect any and all information we can possibly get our hands on.

Other than cbmem -c | head and .config, a majority of people just don't care 
about all the pesky details. They're only useful when debugging problems some 
poor guy or gal is having. In that case, we can ask them for this info 
manually. It's not of any use in tracking down which revision + config works 
on _this_ board.

Alex

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-10 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 11.02.2014 01:56, mrnuke wrote:
 On Tuesday, February 11, 2014 12:04:33 AM Vladimir 'φ-coder/phcoder' 
 Serbinenko wrote:
 If we let user supply
 files at all, it should be added to report, not replace files, and it
 should have some prefix to clearly indicate that user was involved in
 creating them. E.g. user_serial_log.txt

 There is a point at which the information becomes too much. What is really 
 relevant, in my opinion, are the following three points:
 
 1. Is it an unmodified commit in master (AKA, not -dirty).
 2. Does it boot?
 3. What does not work, if any ?
 
 Point 1 and 2 are answered by cbmem -c | head. It answers point 1 with the 
 first line of output, and point 2 by the fact that getting anything from 
 cbmem 
 means you've already booted your system.
 
 Point 3 is more complicated, and it seems it is because of this point that we 
 want to collect any and all information we can possibly get our hands on.
 
 Other than cbmem -c | head and .config, a majority of people just don't 
 care 
 about all the pesky details. They're only useful when debugging problems some 
 poor guy or gal is having. In that case, we can ask them for this info 
 manually. It's not of any use in tracking down which revision + config works 
 on _this_ board.
 
This is not the only uses of board-status. It's also useful to see if
some boards have an issue which one meets in particular board. This
information is important in tracking bugs down but hard to collect
unless it's in the board-status.
 Alex
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

[coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-09 Thread Paul Menzel
Dear coreboot folks,


currently no coreboot messages are stored for boards not supporting
CBMEM console (or where this option is disabled (currently by default))
or no coreboot *romstage* messages are stored for boards, where the data
cannot be preserved (passed to ramstage).

Using the serial (or USB) console all these messages can be captured
with no problem, so I propose to just add these captured messages into
the file `serial_console.txt`. Of course this file probably contains
also the payload and (Linux) kernel log, but I think that is fine.

SeaBIOS’ `readserial.py` should be used for capturing the messages as it
adds time stamps.

Scripting this is going to be hard, as the log is captured on a
different system. So for now I propose to add it manually.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 09.02.2014 13:50, Paul Menzel wrote:
 Dear coreboot folks,
 
 
 currently no coreboot messages are stored for boards not supporting
 CBMEM console
Is there any remaining? If so it's an issue to be fixed
 (or where this option is disabled (currently by default))
Change the defaults.
 or no coreboot *romstage* messages are stored for boards, where the data
 cannot be preserved (passed to ramstage).
 
which boards are concerned? Which boards use ROMCC for romstage? Do they
output anything useful in romstage? If so, can't it be compensated by
ramstage?
 Using the serial (or USB) console all these messages can be captured
 with no problem, so I propose to just add these captured messages into
 the file `serial_console.txt`. Of course this file probably contains
 also the payload and (Linux) kernel log, but I think that is fine.
 
 SeaBIOS’ `readserial.py` should be used for capturing the messages as it
 adds time stamps.
 
 Scripting this is going to be hard, as the log is captured on a
 different system. So for now I propose to add it manually.
 
Having to capture serial makes setup needed for board-status more
difficult. Certainely, one needs to have a recovery method for case of
brick but having a complete serial setup will heavily reduce
board-status input which already isn't huge.
 
 Thanks,
 
 Paul
 
 
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-09 Thread Paul Menzel
Am Sonntag, den 09.02.2014, 14:18 +0100 schrieb Vladimir 'φ-coder/phcoder' 
Serbinenko:
 On 09.02.2014 13:50, Paul Menzel wrote:

  currently no coreboot messages are stored for boards not supporting
  CBMEM console

 Is there any remaining? If so it's an issue to be fixed

Sorry, no idea. I thought there were some. At least not all boards have
been tested to my knowledge.

  (or where this option is disabled (currently by default))

 Change the defaults.

It is not safe or wanted for all boards I believe. But others are more
knowledgeable to comment on that issue.

  or no coreboot *romstage* messages are stored for boards, where the data
  cannot be preserved (passed to ramstage).

 which boards are concerned? Which boards use ROMCC for romstage? Do they
 output anything useful in romstage? If so, can't it be compensated by
 ramstage?

In my case it is the Asus M2V-MX SE (AMD, pre-AGESA).

  Using the serial (or USB) console all these messages can be captured
  with no problem, so I propose to just add these captured messages into
  the file `serial_console.txt`. Of course this file probably contains
  also the payload and (Linux) kernel log, but I think that is fine.
  
  SeaBIOS’ `readserial.py` should be used for capturing the messages as it
  adds time stamps.
  
  Scripting this is going to be hard, as the log is captured on a
  different system. So for now I propose to add it manually.
 
 Having to capture serial makes setup needed for board-status more
 difficult.

Surely it will not be a requirement to capture the serial output. My
proposal is about adding it, when it is captured already and what file
name to use.

 Certainly, one needs to have a recovery method for case of
 brick

You always need that in case of a brick. What is special about serial
log?

 but having a complete serial setup will heavily reduce board-status
 input which already isn't huge.

Again, it is not meant to be a requirement in addition to what is
currently captured.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 09.02.2014 14:34, Paul Menzel wrote:
 Am Sonntag, den 09.02.2014, 14:18 +0100 schrieb Vladimir 'φ-coder/phcoder' 
 Serbinenko:
 On 09.02.2014 13:50, Paul Menzel wrote:
 
 currently no coreboot messages are stored for boards not supporting
 CBMEM console

 Is there any remaining? If so it's an issue to be fixed
 
 Sorry, no idea. I thought there were some. At least not all boards have
 been tested to my knowledge.
 
board-status is all about testing. When we see incomplete
coreboot_console.txt it means there is a bug to be fixed, not workarounded.
 (or where this option is disabled (currently by default))

 Change the defaults.
 
 It is not safe or wanted for all boards I believe. But others are more
 knowledgeable to comment on that issue.
 
I don't see any problem with enabling cbmemc by default on all boards.
 or no coreboot *romstage* messages are stored for boards, where the data
 cannot be preserved (passed to ramstage).

 which boards are concerned? Which boards use ROMCC for romstage? Do they
 output anything useful in romstage? If so, can't it be compensated by
 ramstage?
 
 In my case it is the Asus M2V-MX SE (AMD, pre-AGESA).
 
What is the info we need from it? ROMCC boards usually don't output much.
 Using the serial (or USB)
Your argument with USB console is completele broken as EHCI debug is not
supported on ROMCC boards at all.
 console all these messages can be captured
 with no problem, so I propose to just add these captured messages into
 the file `serial_console.txt`. Of course this file probably contains
 also the payload and (Linux) kernel log, but I think that is fine.

 SeaBIOS’ `readserial.py` should be used for capturing the messages as it
 adds time stamps.

 Scripting this is going to be hard, as the log is captured on a
 different system. So for now I propose to add it manually.

 Having to capture serial makes setup needed for board-status more
 difficult.
 
 Surely it will not be a requirement to capture the serial output. My
 proposal is about adding it, when it is captured already and what file
 name to use.
 
And who will care about it then?
 Certainly, one needs to have a recovery method for case of
 brick
 
 You always need that in case of a brick. What is special about serial
 log?
 
I use debricking setup perhaps every 20 flashes on a supported board.
Not every time.
 but having a complete serial setup will heavily reduce board-status
 input which already isn't huge.
 
 Again, it is not meant to be a requirement in addition to what is
 currently captured.
 
 
 Thanks,
 
 Paul
 
 
 




signature.asc
Description: OpenPGP digital signature
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [RFC] board_status: Add serial log to `serial_console.txt`

2014-02-09 Thread Alex G.
On 02/09/2014 07:34 AM, Paul Menzel wrote:
 Sorry, no idea. I thought there were some. At least not all boards 
 have been tested to my knowledge.
 
Please stop just throwing half-thoght proposals out there when you
have no idea (your words, not mine).

Alex


-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot