[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/3] nvram: at24c: fix problems related to "r
From: |
Wolfram Sang |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/3] nvram: at24c: fix problems related to "rom-size" |
Date: |
Sun, 8 Apr 2018 09:39:42 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Tue, Mar 20, 2018 at 05:18:47PM +0100, Wolfram Sang wrote:
> I used this driver as a template for a custom one. While hacking on my own, I
> noticed some problems in this driver, too. This series fixes the first set of
> them, related to the "rom-size" parameter. It fixes a segfault.
>
> I think the first patch is clearly suitable for stable. I think the second
> one,
> too, but not as clearly. The third one is a cleanup and not for stable. Still,
> I am open for opinions about these thoughts.
Through which tree should these patches go? get_maintainer doesn't list
a person specifically. Anything else I could do?
>
> Thanks,
>
> Wolfram
>
> Changes since v2:
>
> * removed '\n' from error_report-strings
> * made sure checkpatch is happy
> * added tags from Philippe (thanks!)
>
> Changes since v1:
>
> * reordered patches according to significance for stable
> * use AT24C_ROMSIZE_DEFAULT instead of magic value
> * patch 3 doesn't improve the ERR macro anymore but replaces
> it completely with error_report().
>
>
> Wolfram Sang (3):
> nvram: at24c: prevent segfault by checking "rom-size"
> nvram: at24c: use a sane default for "rom-size"
> nvram: at24c: use standard error reporting
>
> hw/nvram/eeprom_at24c.c | 25 ++++++++++++++-----------
> 1 file changed, 14 insertions(+), 11 deletions(-)
>
> --
> 2.11.0
>
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 0/3] nvram: at24c: fix problems related to "rom-size",
Wolfram Sang <=