public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Takashi Sakamoto <o-takashi@sakamocchi.jp>
To: torvalds@linux-foundation.org
Cc: linux-kernel@vger.kernel.org
Subject: [GIT PULL] firewire updates for 6.7
Date: Sun, 5 Nov 2023 23:48:52 +0900	[thread overview]
Message-ID: <20231105144852.GA165906@workstation.local> (raw)

Hi Linus,

This is the pull request from FireWire subsystem. It contains a slight
change in core function itself. I'm glad if you accept it to your tree.

Allow me to report a regression of PCI driver for 1394 OHCI hardware in
kernel 6.5 or later. When probing the hardware with following combination
of chips on Linux system running on AMD Ryzen machines, the system
unexpectedly reboots.

* Asmedia ASM1083 (PCI bridge)
* VIA VT6306/6307/6308 (1394 OHCI controller)

The issue has been reported by users from OpenSUSE 
(https://bugzilla.suse.com/show_bug.cgi?id=1215436) and Fedora
(https://bugzilla.kernel.org/show_bug.cgi?id=217994), and I can reproduce
it on my local machine. The superficial cause of issue is accesses to
Isochronous Cycle Timer register in OHCI control register space during the
PCI device probing. However, the exact mechanism triggering the reboot is
currently unclear, as the system does not provide any useful output. In my
experience, this issue does not occur on Intel machine, AMD Sempron 145
machine, or with different 1394 OHCI hardware from other vendors such as
Texas Instruments. While it's possible that there may be code issues
within the PCI device driver, I think that specific hardware quirks exist
in the combination of the Ryzen machines, the PCI bridge, and the 1394
OHCI controller, since I can see another issue of cache-coherence over
DMA memory mapping in Ryzen machine with VT6315. Anyway I'm still under
investigation and wish to find any workaround as soon.

----------------------------------------------------------------

The following changes since commit ffc253263a1375a65fa6c9f62a893e9767fbebfa:

  Linux 6.6 (2023-10-29 16:31:08 -1000)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394.git tags/firewire-updates-6.7

for you to fetch changes up to c12d7aa7ffa4c61443241fbc1ee405acf4aa17de:

  firewire: Annotate struct fw_node with __counted_by (2023-11-05 21:15:17 +0900)

----------------------------------------------------------------
firewire updates for 6.7

The pull request includes a slight change for flexible length of array
in core function.

Kees Cook provides a patch to annotate the array embedded in fw_node
structure referring to structure member for the length of array. The
annotation would be defined by future extension of C compilers, and
used for access bound-check at run-time enabled by UBSAN and
FORTIFY_SOURCE.

----------------------------------------------------------------
Kees Cook (1):
      firewire: Annotate struct fw_node with __counted_by

 drivers/firewire/core.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


Regards

Takashi Sakamoto

             reply	other threads:[~2023-11-05 14:49 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-05 14:48 Takashi Sakamoto [this message]
2023-11-05 18:53 ` [GIT PULL] firewire updates for 6.7 pr-tracker-bot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231105144852.GA165906@workstation.local \
    --to=o-takashi@sakamocchi.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox