Message ID | 20211119084919.83906-1-damien.lemoal@opensource.wdc.com |
---|---|
State | New |
Headers | show |
Series | [GIT,PULL] libata fixes for 5.16-rc2 | expand |
Hello! On 19.11.2021 11:49, Damien Le Moal wrote: > Linus, > > The following changes since commit fa55b7dcdc43c1aa1ba12bca9d2dd4318c2a0dbf: > > Linux 5.16-rc1 (2021-11-14 13:56:52 -0800) > > are available in the Git repository at: > > ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/dlemoal/libata tags/libata-5.16-rc2 > > for you to fetch changes up to 6873c30c1b797997f302fe1707bee9a7f247dfe2: > > sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl (2021-11-19 13:57:18 +0900) > > ---------------------------------------------------------------- > libata fixes for 5.16-rc2 > > * Prevent accesses to unsupported log pages as that causes device scan > failures with LLDDs using libsas (from me). > * A couple of fixes for AMD AHCI adapters handling of low power modes > and resume (from Mario). > * Fix a compilation warning (from me). > * Fix sata_fsl driver problems on PPC64 (from Baokun). > > ---------------------------------------------------------------- > > Baokun Li (2): > sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl > sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl At least this one needs fixing! You were too fast at merging... :-/ Well, I guess this will need a follow-up fix now... [...] MBR, Sergey
On Fri, Nov 19, 2021 at 7:46 AM Sergei Shtylyov <sergei.shtylyov@gmail.com> wrote: > On 19.11.2021 11:49, Damien Le Moal wrote: > > Baokun Li (2): > > sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl > > sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl > > At least this one needs fixing! You were too fast at merging... :-/ > Well, I guess this will need a follow-up fix now... Hmm. I'm missing the context, but considering this note I have skipped this libata pull request. Damien? Linus
On 19.11.2021 22:06, Linus Torvalds wrote: >> On 19.11.2021 11:49, Damien Le Moal wrote: >>> Baokun Li (2): >>> sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl >>> sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl >> >> At least this one needs fixing! You were too fast at merging... :-/ >> Well, I guess this will need a follow-up fix now... > > Hmm. I'm missing the context, but considering this note I have skipped > this libata pull request. The patch replaced irq_of_parse_and_map() with platform_get_irq() but didn't update the error check from (!irq) to (irq < 0)... Quite frequent mistake in the past; it actually helps if you look at the function itself before starting to use it. :-) [...] MBR, Sergei
On 11/20/21 00:46, Sergei Shtylyov wrote: > Hello! > > On 19.11.2021 11:49, Damien Le Moal wrote: > >> Linus, >> >> The following changes since commit fa55b7dcdc43c1aa1ba12bca9d2dd4318c2a0dbf: >> >> Linux 5.16-rc1 (2021-11-14 13:56:52 -0800) >> >> are available in the Git repository at: >> >> ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/dlemoal/libata tags/libata-5.16-rc2 >> >> for you to fetch changes up to 6873c30c1b797997f302fe1707bee9a7f247dfe2: >> >> sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl (2021-11-19 13:57:18 +0900) >> >> ---------------------------------------------------------------- >> libata fixes for 5.16-rc2 >> >> * Prevent accesses to unsupported log pages as that causes device scan >> failures with LLDDs using libsas (from me). >> * A couple of fixes for AMD AHCI adapters handling of low power modes >> and resume (from Mario). >> * Fix a compilation warning (from me). >> * Fix sata_fsl driver problems on PPC64 (from Baokun). >> >> ---------------------------------------------------------------- >> >> Baokun Li (2): >> sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl >> sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl > > At least this one needs fixing! You were too fast at merging... :-/ Arg... And I did read about that function to check before applying the patch. But I missed the error check. I am going to drop these patches for now. They can go into rc3 after fixing. > Well, I guess this will need a follow-up fix now... I will request a V2 to the author so that it gets tested (I do not have that hardware, so I cannot test myself). > > [...] > > MBR, Sergey >
On 11/20/21 04:06, Linus Torvalds wrote: > On Fri, Nov 19, 2021 at 7:46 AM Sergei Shtylyov > <sergei.shtylyov@gmail.com> wrote: >> On 19.11.2021 11:49, Damien Le Moal wrote: >>> Baokun Li (2): >>> sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl >>> sata_fsl: fix warning in remove_proc_entry when rmmod sata_fsl >> >> At least this one needs fixing! You were too fast at merging... :-/ >> Well, I guess this will need a follow-up fix now... > > Hmm. I'm missing the context, but considering this note I have skipped > this libata pull request. > > Damien? My apologies about that. I am going to update the PR and resend. The read log patches need to go in. Without them, some drives fail to be detected at boot time with some HBAs. > > Linus >