summaryrefslogtreecommitdiffstats
path: root/doc/release-notes/skiboot-5.2.0.rst
blob: 28d977d98dfb29eb9b819b523292097a62fee21a (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
skiboot-5.2.0
=============

skiboot-5.2.0 was released on Wednesday March 16th, 2016.

skiboot-5.2.0 is the first stable release of skiboot 5.2, the new stable
release of skiboot, which will take over from the 5.1.x series which was
first released August 17th, 2015.

skiboot-5.2.0 contains all bug fixes as of skiboot-5.1.15.

This is the second release that will follow the (now documented) Skiboot
stable rules - see :ref:`stable-rules`.

Changes since rc2
-----------------
Over skiboot-5.2.0-rc2, the following fixes are included:

- Include 'extract-gcov' in make clean.
- ipmi-sel: Fix esel event logger to handle early boot PANIC events
- IPMI: Enable synchronous eSEL logging option (for PANIC events)
- libflash/libffs: Reporting seeing all 0xFF bytes during init.
- ipmi-sel: Fix memory leak in error path

Changes since rc1
-----------------
Over skiboot-5.2.0-rc1, we have the following changes:

- Add Barreleye platform

Generic
^^^^^^^

- hw/p8-i2c: Speed up SMBUS_WRITE
- Fix early backtraces

FSP Platforms
^^^^^^^^^^^^^

- fsp-sensor: rework device tree for sensors
- platforms/firenze: Fix I2C clock source frequency

Simics simulator
^^^^^^^^^^^^^^^^

- Enable Simics UART console

Mambo simulator
^^^^^^^^^^^^^^^

- platforms/mambo: Add terminate callback

  - fix hang in multi-threaded mambo
  - add multithreaded mambo tests

IPMI
^^^^

- hw/ipmi: fix event data 1 for System Firmware Progress sensor
- ipmi: Log exact NetFn value in OPAL logs

AST BMC based platforms
^^^^^^^^^^^^^^^^^^^^^^^

- hw/bt: allow BT driver to use different buffer size

opal-prd utility
^^^^^^^^^^^^^^^^

- opal-prd: Add debug output for firmware-driven OCC events
    We indicate when we have a user-driven event, so add corresponding
    outputs for firmware-driven ones too.

getscom utility
^^^^^^^^^^^^^^^

- Add Naples chip support

New Features
^^^^^^^^^^^^
Over skiboot-5.1, the following features have been added:

- Naples (P8', i.e. P8 with NVLINK) processor support, including NVLINK.
- Improvements in gard, libflash/pflash and opal-prd utilities

  - increased testing
  - increased usability
  - systemd scripts for opal-prd
  - pflash can now use the /dev/mtd device to access BMC flash rather than
    accessing it directly. It is *important* that you use --mtd if your
    BMC may otherwise know how to interact with its own flash.
- support for Micron N25Q256Ax and N25Qx256Ax NOR flash.
- support for Winbond W25Q256BV NOR flash
- support for an emulated ("fake") RTC clock, useful in simulators
  and during bringup
- Explicit 1:1 mapping in ranges properties have been added to PCI
  bridges. This allows a neat trick with offb and VGA ports that should
  probably not be told to young children.
- Added support to read the V2 format of the OCC-OPAL memory region,
  which supports Workload Optimized Frequency (WOF)

Changes in behavior
^^^^^^^^^^^^^^^^^^^

- Assigning OPAL IDs to PHBs is now fixed and based on the chip id and PHB
  index on that chip. On POWER7, we continue to use allocated numbers.
- We now query the BMC for BT capabilities rather than making assumptions

Removed support
^^^^^^^^^^^^^^^

- p5ioc2 is no longer supported.
  This affects a grand total of two POWER7 systems in the world.

**NOTE**: It is planned that skiboot-5.2 will be the last release supporting
POWER7 machines.

Bugs fixed
^^^^^^^^^^

- PHB3: Fix unexpected ER (all) on errinjct by PCI config
- hw/bt: timeout messages when BT interface isn't functional
- On Habanero, Slot3 should have been "Slot 3".
- We now completely flush the console buffer before power down and reboot
- For chips with ibm,occ-functional-state set to false, we don't wait
  for the OCC to start. This caused needless delay in booting on simulators
  which did not simulate OCCs.
- Change OCC reset order to always reset slave OCCs first.
- slw: Remove overwrites for EX_PM_CORE_ECO_VRET and EX_PM_CORE_PFET_VRET
  (these were already initialized in hostboot)
- p8-i2c: send stop bit on timeouts.
  Some devices can otherwise leave the bus in a held state.

Other improvements
^^^^^^^^^^^^^^^^^^

- many fixes of compiler and static analysis warnings
- increased unit test coverage
- Unit test of "boot debian jessie installer"
- ability to plug in other simulators to run existing tests (e.g. simulator for
  non pegasus p8)
- Support using (patched) Qemu with PowerNV platform support for running
  unit tests.
- increased support for running with sparse
- We now build with -fstack-protector-strong if supported by the compiler
- We now build with -Werror for -Wformat
- pflash is now built as part of travis-ci and for Coverity Scan.
- There is now a RPM SPEC file that can be used as the basis for packaging
  skiboot and associated utilities.

Contributors
------------

We have had a number of improvements in workflow over skiboot-5.1.0. Looking
back, we have roughly the same number of changesets (372 for 5.1.0, 334 for
5.2.0-rc1 - even closer for 5.1.0-beta1) which indicates a relatively stable
rate of development.

Complete statistics are included below (generated by gitdm), but I'd like to
draw attention to a couple of stats:

======== ====== ======= ======= ======  ========
Release	 csets	Ack	Reviews	Tested	Reported
======== ====== ======= ======= ======  ========
5.0	 329	 15	     20	     1	       0
5.1	 372	 13	     38	     1	       4
5.2-rc1	 334	 20	     34	     6	      11
======== ====== ======= ======= ======  ========

Overall, it looks like we're on the right trajectory for increasing the number
of eyeballs looking at code before it heads in tree, especially around testing.
Largely, this increase in Tested-by can be attributed to encouraging the
existing test teams to start commenting on the patches themselves.

Anyway, here's the full stats from skiboot 5.1.0 to 5.2.0-rc1:

Processed 334 csets from 27 developers
2 employers found
A total of 46172 lines added, 23274 removed (delta 22898)

Developers with the most changesets

========================== ===========
========================== ===========
Stewart Smith              146 (43.7%)
Cyril Bur                   52 (15.6%)
Benjamin Herrenschmidt      15 (4.5%)
Joel Stanley                12 (3.6%)
Gavin Shan                  12 (3.6%)
Alistair Popple             10 (3.0%)
Vasant Hegde                10 (3.0%)
Michael Neuling             10 (3.0%)
Russell Currey               9 (2.7%)
Cédric Le Goater             8 (2.4%)
Jeremy Kerr                  8 (2.4%)
Samuel Mendoza-Jonas         6 (1.8%)
Neelesh Gupta                6 (1.8%)
Shilpasri G Bhat             4 (1.2%)
Oliver O'Halloran            4 (1.2%)
Mahesh Salgaonkar            4 (1.2%)
Vipin K Parashar             3 (0.9%)
Daniel Axtens                3 (0.9%)
Andrew Donnellan             2 (0.6%)
Philippe Bergheaud           2 (0.6%)
Ananth N Mavinakayanahalli   2 (0.6%)
Vaibhav Jain                 1 (0.3%)
Sam Mendoza-Jonas            1 (0.3%)
Adriana Kobylak              1 (0.3%)
Shreyas B. Prabhu            1 (0.3%)
Vaidyanathan Srinivasan      1 (0.3%)
Ian Munsie                   1 (0.3%)
========================== ===========

Developers with the most changed lines


========================== =============
========================== =============
Stewart Smith              19533 (39.4%)
Oliver O'Halloran          17920 (36.1%)
Alistair Popple             3285 (6.6%)
Daniel Axtens               2154 (4.3%)
Cyril Bur                   2028 (4.1%)
Benjamin Herrenschmidt       941 (1.9%)
Neelesh Gupta                434 (0.9%)
Gavin Shan                   294 (0.6%)
Russell Currey               261 (0.5%)
Vasant Hegde                 245 (0.5%)
Cédric Le Goater             209 (0.4%)
Vipin K Parashar             155 (0.3%)
Shilpasri G Bhat             153 (0.3%)
Joel Stanley                 140 (0.3%)
Vaidyanathan Srinivasan      135 (0.3%)
Michael Neuling              111 (0.2%)
Samuel Mendoza-Jonas          81 (0.2%)
Jeremy Kerr                   60 (0.1%)
Mahesh Salgaonkar             58 (0.1%)
Vaibhav Jain                  50 (0.1%)
Ananth N Mavinakayanahalli    43 (0.1%)
Shreyas B. Prabhu             17 (0.0%)
Sam Mendoza-Jonas             12 (0.0%)
Andrew Donnellan              10 (0.0%)
Ian Munsie                     8 (0.0%)
Philippe Bergheaud             6 (0.0%)
Adriana Kobylak                6 (0.0%)
========================== =============

Developers with the most lines removed

========================= =============
========================= =============
Daniel Axtens             2149 (9.2%)
Shreyas B. Prabhu           17 (0.1%)
Andrew Donnellan             9 (0.0%)
Vipin K Parashar             2 (0.0%)
========================= =============

Developers with the most signoffs (total 190)

========================= =============
========================= =============
Stewart Smith              188 (98.9%)
Gavin Shan                   1 (0.5%)
Neelesh Gupta                1 (0.5%)
========================= =============

Developers with the most reviews (total 34)

========================= =============
========================= =============
Patrick Williams             5 (14.7%)
Joel Stanley                 5 (14.7%)
Cédric Le Goater            5 (14.7%)
Vasant Hegde                 4 (11.8%)
Alistair Popple              4 (11.8%)
Sam Mendoza-Jonas            3 (8.8%)
Samuel Mendoza-Jonas         3 (8.8%)
Andrew Donnellan             2 (5.9%)
Cyril Bur                    2 (5.9%)
Vaibhav Jain                 1 (2.9%)
========================= =============

Developers with the most test credits (total 6)

========================= =============
========================= =============
Vipin K Parashar             3 (50.0%)
Vaibhav Jain                 2 (33.3%)
Gajendra B Bandhu1           1 (16.7%)
========================= =============

Developers who gave the most tested-by credits (total 6)

=========================== =============
=========================== =============
Gavin Shan                   2 (33.3%)
Ananth N Mavinakayanahalli    2 (33.3%)
Alistair Popple              1 (16.7%)
Stewart Smith                1 (16.7%)
=========================== =============

Developers with the most report credits (total 11)

========================= =============
========================= =============
Vaibhav Jain                 2 (18.2%)
Paul Nguyen                  2 (18.2%)
Alistair Popple              1 (9.1%)
Cédric Le Goater            1 (9.1%)
Aneesh Kumar K.V             1 (9.1%)
Dionysius d. Bell            1 (9.1%)
Pradeep Ramanna              1 (9.1%)
John Walthour                1 (9.1%)
Benjamin Herrenschmidt       1 (9.1%)
========================= =============

Developers who gave the most report credits (total 11)

========================= =============
========================= =============
Gavin Shan                   6 (54.5%)
Stewart Smith                3 (27.3%)
Samuel Mendoza-Jonas         1 (9.1%)
Shilpasri G Bhat             1 (9.1%)
========================= =============
OpenPOWER on IntegriCloud