[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Nick Clifton changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|--- |FIXED --- Comment #31 from Nick Clifton --- The bug is fixed in the 2.41 release. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Pete Moore changed: What|Removed |Added CC||petemoore at gmx dot net --- Comment #30 from Pete Moore --- I have a simple test case which I think is consistent with this bug: ``` $ cat adrp.s .global _start _start: adrp x0, 0x8000 ``` After assembling and linking: ``` $ aarch64-none-elf-as -o adrp.o adrp.s $ aarch64-none-elf-ld -Ttext=0x0 -o adrp.elf adrp.o aarch64-none-elf-ld: adrp.o: in function `_start': (.text+0x0): undefined reference to `no symbol' ``` Note, the following trick works (if e.g. _start is known to be 0): ``` $ cat adrp.s .global _start _start: adrp x0, 0x8000 + _start ``` which assembles as desired: ``` $ aarch64-none-elf-objdump -d adrp.elf adrp.elf: file format elf64-littleaarch64 Disassembly of section .text: <_start>: 0: 9040adrpx0, 8000 <_start+0x8000> ``` However, this only works if _start is fixed and known. Being able to specify an absolute (rather than relative) value for the immediate of the adrp instruction is useful for e.g. referencing peripherals at fixed locations which are known to be in the +/- 4GB range of the current address. My particular use case is referencing Raspberry Pi peripheral base addresses. Let me know if I should raise a separate bug, or if the above test case is is consistent with the current bug. Many thanks! ``` $ aarch64-none-elf-ld --version GNU ld (GNU Binutils) 2.39 ``` -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #29 from Kinsey Moore --- Ah, great! Thanks so much for looking into this. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #28 from Jan Beulich --- (In reply to Jan Beulich from comment #26) > Quoting from the description of r_info in the ELF spec: "If the index is > STN_UNDEF, the undefined symbol index, the relocation uses 0 as the ``symbol > value''." Which makes me think we're dealing with a linker issue here, as > this is precisely the situation we're in. I'll see to find time to go hunt, > but I'm far less familiar with ld than with gas. (If others agree this is a > separate issue, I guess this would better be handled in a fresh bug report.) See https://sourceware.org/pipermail/binutils/2023-March/126759.html -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #27 from Jan Beulich --- Another question is: Can't we suppress emitting of relocations when the value is absolute? (Of course really the relocation in the testcase should reference "bar", but as we've seen arranging for that by simply avoiding to evaluate expressions produces other fallout. In the corresponding email conversation I did suggest yet another expression evaluation mode as a possible route. But that would look more like another hack than a solution, to me at least.) -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #26 from Jan Beulich --- (In reply to Kinsey Moore from comment #25) > The original test case should show it provided that you also attempt to link > it as per Nick's comment: > https://sourceware.org/bugzilla/show_bug.cgi?id=27217#c4 Oh, I'm sorry for not paying attention. I can indeed observe the issue there. Quoting from the description of r_info in the ELF spec: "If the index is STN_UNDEF, the undefined symbol index, the relocation uses 0 as the ``symbol value''." Which makes me think we're dealing with a linker issue here, as this is precisely the situation we're in. I'll see to find time to go hunt, but I'm far less familiar with ld than with gas. (If others agree this is a separate issue, I guess this would better be handled in a fresh bug report.) -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #25 from Kinsey Moore --- The original test case should show it provided that you also attempt to link it as per Nick's comment: https://sourceware.org/bugzilla/show_bug.cgi?id=27217#c4 -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Jan Beulich changed: What|Removed |Added Resolution|FIXED |--- Status|RESOLVED|REOPENED --- Comment #24 from Jan Beulich --- (In reply to Kinsey Moore from comment #22) > To be clear, the "no symbol" error has returned. Since the testcase that was originally added doesn't cover this case (which is why it wasn't noticed), can you please supply an example code fragment where the bad behavior is observed? -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Kinsey Moore changed: What|Removed |Added CC||jbeulich at suse dot com -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #23 from Kinsey Moore --- I have verified that commit c1723a8118f0d02b01a061095f48e75264b2ca4f is the cause of the regression. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #22 from Kinsey Moore --- To be clear, the "no symbol" error has returned. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #21 from Kinsey Moore --- This has broken again in 2.40, presumably due to the rework that occurred. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #20 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Jan Beulich : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c1723a8118f0d02b01a061095f48e75264b2ca4f commit c1723a8118f0d02b01a061095f48e75264b2ca4f Author: Jan Beulich Date: Fri Jul 29 09:26:47 2022 +0200 Arm64: re-work PR gas/27217 fix The original approach has resulted in anomalies when . is involved in an operand of one of the affected insns. We cannot leave . unresolved, or else it'll be resolved at the end of assembly, then pointing to the address of a section rather than at the insn of interest. Undo part of the original change and instead check whether a relocation cannot be omitted in md_apply_fix(). By resolving the expressions again, equates (see the adjustment of the respective testcase) will now be evaluated, and hence relocations against absolute addresses be emitted. This ought to be okay as long as the equates aren't global (and hence can't be overridden). If a need for such arises, quite likely the only way to address this would be to invent yet another expression evaluation mode, leaving everything _except_ . un-evaluated. There's a further anomaly in how transitive equates are handled. In .set x, 0x12345678 .eqv bar, x foo: adrpx0, x add x0, x0, :lo12:x adrpx0, bar add x0, x0, :lo12:bar the first two relocations are now against *ABS*:0x12345678 (as said above), whereas the latter two relocations would be against x. (Before the change here, the first two relocations are against x and the latter two against bar.) But this is an issue seen elsewhere as well, and would likely require adjustments in the target-independent parts of the assembler instead of trying to hack around this for every target. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #19 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Jan Beulich : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=15b7af6c874610d802b64e1778202b7653d5fa08 commit 15b7af6c874610d802b64e1778202b7653d5fa08 Author: Jan Beulich Date: Thu May 19 12:46:21 2022 +0200 Arm64: force emission of ILP32-dependent relocs Like the placeholder types added in 04dfe7aa5217 ("Arm64: follow-on to PR gas/27217 fix"), these are also placeholders which are subsequently resolved (albeit later, hence this being a separate issue). As for the resolved types 1 is returned, these pseudo-relocs should also have 1 returned to force retaining of the [eventual] relocations. This is also spelled out individually for each of them in md_apply_fix(). -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #18 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Jan Beulich : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=04dfe7aa52171d110db813bce67c0eea5f4b18cd commit 04dfe7aa52171d110db813bce67c0eea5f4b18cd Author: Jan Beulich Date: Wed May 18 17:55:55 2022 +0200 Arm64: follow-on to PR gas/27217 fix PR gas/27217 Prior to trying to address PR gas/2 I noticed anomalies in how certain insns would / wouldn't be affected in similar ways. Commit eac4eb8ecb26 ("Fix a problem assembling AArch64 sources when a relocation is generated against a symbol that has a defined value") had two copy-and-paste mistakes, passing the wrong type to aarch64_force_reloc(). It further failed to add placeholder relocation types to that function's block of case labels leading to a return of 1. While not of interest for aarch64_force_relocation() (these placeholders are resolved right in parse_operands()), calls to aarch64_force_reloc() happen before that resolution would take place. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #17 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Alan Modra : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=b12389f219facfb4aa29b97fdcbc2664a5b0732a commit b12389f219facfb4aa29b97fdcbc2664a5b0732a Author: Alan Modra Date: Wed Apr 7 18:12:38 2021 +0930 Fix pr27217 testcase failure aarch64_be-linux-gnu_ilp32 +FAIL: PR27212 PR 27217 * testsuite/gas/aarch64/pr27217.d: Correct name. Accept ilp32 relocs. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #15 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Nick Clifton : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=eac4eb8ecb2626ef7711d8f6bee9e870ae435604 commit eac4eb8ecb2626ef7711d8f6bee9e870ae435604 Author: Nick Clifton Date: Tue Apr 6 13:27:50 2021 +0100 Fix a problem assembling AArch64 sources when a relocation is generated against a symbol that has a defined value. PR 27217 * config/tc-aarch64.c (my_get_expression): Rename to aarch64_get_expression. Add a fifth argument to enable deferring of expression resolution. (parse_typed_reg): Update calls to my_get_expression. (parse_vector_reg_list): Likewise. (parse_immediate_expression): Likewise. (parse_big_immediate): Likewise. (parse_shift): Likewise. (parse_shifter_operand_imm): Likewise. (parse_operands): Likewise. (parse_shifter_operand_reloc): Update calls to my_get_expression and call aarch64_force_reloc to determine the value of the new fifth argument. (parse_address_main): Likewise. (parse_half): Likewise. (parse_adrp): Likewise. (aarch64_force_reloc): New function. Contains code extracted from... (aarch64_force_relocation): ... here. * testsuite/gas/aarch64/pr27217.s: New test case. * testsuite/gas/aarch64/pr27217.d: New test driver. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Nick Clifton changed: What|Removed |Added Resolution|--- |FIXED Status|REOPENED|RESOLVED --- Comment #16 from Nick Clifton --- OK, patch applied. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #14 from Kinsey Moore --- I had to tweak the patch a little to compile (no bool) and apply (probably because I'm working off of a slightly older revision), but everything seems to be working as expected with this latest patch. Thanks for diving into this! -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Nick Clifton changed: What|Removed |Added Attachment #13329|0 |1 is obsolete|| Attachment #13335|0 |1 is obsolete|| --- Comment #13 from Nick Clifton --- Created attachment 13341 --> https://sourceware.org/bugzilla/attachment.cgi?id=13341=edit Proposed patch Hi Guys, OK, please try out this third patch. (It should be used on its own - it supplants the other two). This patch tries to stop the assembler's expression evaluator from converting the references to the "bar" symbol into references to a constant value. Instead the symbol is preserved intact in the object file, and the relocations are resolved by the linker as normal. I *think* that this is the right way to resolve the problem, since looking at the generated assembler code, and C source, it appears that the intention is provide a fixed value for the "bar" symbol, but not to eliminate it completely. Cheers Nick -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #12 from Kinsey Moore --- (In reply to Nick Clifton from comment #11) > Ah - and change the ".set bar,0" to something else, like ".set bar,1". > Right ? >From the two tests, one was 0x2800 and one was 0xabc, but I think 1 there should suffice just as well. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #11 from Nick Clifton --- (In reply to Joel Sherrill from comment #10) > Nick. Go back to the first comment when I posted this. I put the C code > along with assembly from two different optimization levels. Ah - and change the ".set bar,0" to something else, like ".set bar,1". Right ? -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #10 from Joel Sherrill --- Nick. Go back to the first comment when I posted this. I put the C code along with assembly from two different optimization levels. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #9 from Nick Clifton --- (In reply to Kinsey Moore from comment #8) > I'll see if I can come up with a reduced test case. Do you prefer an > executable output that runs on QEMU or just something that compiles to an > object file to pick apart? Preferably an assembler source file. Then I can check that the object file either contains the necessary relocations, or else that the correct values have been stored in the constant part of the relevant instructions. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #8 from Kinsey Moore --- (In reply to Nick Clifton from comment #7) > (In reply to Kinsey Moore from comment #6) > > I've tested the new patch and, while everything now compiles and links, > > there seems to be some weird behavior with global symbols. > > Yeah - this is the kind of thing that I was worried about. > > > This bug was > > previously affecting two tests in RTEMS under AArch64 and both tests now > > exhibit global symbols not yielding their assigned values. The remainder of > > the tests still pass appropriately. > > Am I correct in thinking that these assigned values are non-zero ? You are correct, the assigned values are non-zero in both cases. > One effect of removing the relocations entirely is that the effected > symbol's value is always treated as 0... > > Do you have a test case that I can look at to see if I can improve the > patch ? I'll see if I can come up with a reduced test case. Do you prefer an executable output that runs on QEMU or just something that compiles to an object file to pick apart? -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #7 from Nick Clifton --- (In reply to Kinsey Moore from comment #6) > I've tested the new patch and, while everything now compiles and links, > there seems to be some weird behavior with global symbols. Yeah - this is the kind of thing that I was worried about. > This bug was > previously affecting two tests in RTEMS under AArch64 and both tests now > exhibit global symbols not yielding their assigned values. The remainder of > the tests still pass appropriately. Am I correct in thinking that these assigned values are non-zero ? One effect of removing the relocations entirely is that the effected symbol's value is always treated as 0... Do you have a test case that I can look at to see if I can improve the patch ? -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Kinsey Moore changed: What|Removed |Added CC||nyphbl8d+sourceware at gmail dot c ||om --- Comment #6 from Kinsey Moore --- I've tested the new patch and, while everything now compiles and links, there seems to be some weird behavior with global symbols. This bug was previously affecting two tests in RTEMS under AArch64 and both tests now exhibit global symbols not yielding their assigned values. The remainder of the tests still pass appropriately. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #5 from Nick Clifton --- Created attachment 13335 --> https://sourceware.org/bugzilla/attachment.cgi?id=13335=edit Proposed patch -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Nick Clifton changed: What|Removed |Added Ever confirmed|0 |1 Status|RESOLVED|REOPENED Last reconfirmed||2021-03-29 Resolution|DUPLICATE |--- --- Comment #4 from Nick Clifton --- (In reply to Joel Sherrill from comment #3) Hi Joel, > It changes the failure to one I've never see before. :) > > /home/opticron/rtems-development/tools/lib/gcc/aarch64-rtems6/10.2.1/../../.. > /../aarch64-rtems6/bin/ld: testsuites/sptests/spconfig01/init.c.568.o: in > function `test_stack_config': > /home/opticron/rtems-development/rtems/build/aarch64/xilinx_zynqmp_lp64_qemu/ > ../../../testsuites/sptests/spconfig01/init.c:95: undefined reference to `no > symbol' Ah- that should not happen. Your test case shows it too, although I only ever assembled it, I did not try to link it. The reloc is for the ADRP instruction's reference to the zero'ed "bar" symbol. I am not sure of the best thing to do here. I am going to upload a patch that stops the relocation from being generated, but that might not be the right thing to do. Since the symbol has been, effectively, deleted, maybe generating a linker error is correct. > This PR is marked as resolved/duplicate but is that right? Nope - that was a snafu. I am reopening the PR. Cheers Nick -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 --- Comment #3 from Joel Sherrill --- (In reply to Nick Clifton from comment #2) > Created attachment 13329 [details] > Proposed patch > > Hi Joel, > > Please could you try out this patch and let me know if it works for you ? > > Cheers > Nick It changes the failure to one I've never see before. :) /home/opticron/rtems-development/tools/lib/gcc/aarch64-rtems6/10.2.1/../../../../aarch64-rtems6/bin/ld: testsuites/sptests/spconfig01/init.c.568.o: in function `test_stack_config': /home/opticron/rtems-development/rtems/build/aarch64/xilinx_zynqmp_lp64_qemu/../../../testsuites/sptests/spconfig01/init.c:95: undefined reference to `no symbol' collect2: error: ld returned 1 exit status This PR is marked as resolved/duplicate but is that right? -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Nick Clifton changed: What|Removed |Added CC||nickc at redhat dot com --- Comment #2 from Nick Clifton --- Created attachment 13329 --> https://sourceware.org/bugzilla/attachment.cgi?id=13329=edit Proposed patch Hi Joel, Please could you try out this patch and let me know if it works for you ? Cheers Nick -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Alex Coplan changed: What|Removed |Added CC||acoplan at gcc dot gnu.org Status|UNCONFIRMED |RESOLVED Resolution|--- |DUPLICATE --- Comment #1 from Alex Coplan --- Dup. *** This bug has been marked as a duplicate of bug 26395 *** -- You are receiving this mail because: You are on the CC list for the bug.
[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.
https://sourceware.org/bugzilla/show_bug.cgi?id=27217 Joel Sherrill changed: What|Removed |Added Target||aarch64-rtems aarch64-elf -- You are receiving this mail because: You are on the CC list for the bug.