[Bug 265974] SMR has several missing barriers

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265974

--- Comment #12 from Pierre Habouzit  ---
wow TIL, that's the same on Darwin huh

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 265974] SMR has several missing barriers

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265974

--- Comment #11 from Mark Johnston  ---
I do think the deferred advance might still be incorrect on platforms where
FreeBSD's atomic_thread_fence_rel() does not provide a store/load barrier, such
as POWER.

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 265974] SMR has several missing barriers

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265974

--- Comment #10 from Mark Johnston  ---
(In reply to Pierre Habouzit from comment #9)
dmb st only orders stores, but that's not sufficient to implement a release
fence.  For what it's worth, LLVM on FreeBSD 14/arm64 compiles
atomic_thread_fence(memory_order_release) to a "dmb ish", same as
atomic_thread_fence(memory_order_seq_cst).

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 265974] SMR has several missing barriers

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265974

--- Comment #9 from Pierre Habouzit  ---
`atomic_thread_fence_rel()` is a `dmb sy` ? I would have expected that to be a
`dmb st` ..

but indeed it shows I'm not familiar with with FreeBSD:

```C
static __inline void
atomic_thread_fence_rel(void)
{

dmb(sy);
}
```

and indeed it makes the deferred advance correct then.

It also means smr_synchronize() has a stronger semantics than I expected
already and makes the smr_poll barrier on entry useless because the
`atomic_load_acq_int()` in smr_poll() already give the semantics I was after on
entry.

I think you might be right that the one on exit of smr_poll() isn't quite
needed. I think I just didn't think it through and just took
`ck_epoch_synchronize_wait` as an example and just was like "it needs one so
smr_scan does too" but trying to come up with a reason why it breaks yields
nothing..

-- 
You are receiving this mail because:
You are the assignee for the bug.


Problem reports for b...@freebsd.org that need special attention

2022-08-28 Thread bugzilla-noreply
To view an individual PR, use:
  https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=(Bug Id).

The following is a listing of current problems submitted by FreeBSD users,
which need special attention. These represent problem reports covering
all versions including experimental development code and obsolete releases.

Status  |Bug Id | Description
+---+---
New |197876 | [devfs] an error in devfs leads to data loss and  
New |198797 | [PATCH] Added an option to install BSDstats to bs 
New |202362 | ntp: restore refclocks selection (10.2-RELEASE re 
New |202740 | vi/ex string substitution problem when there is m 
New |204097 | witness_initialize() does not perform bound check 
New |206336 | [patch] usr.sbin/freebsd-update allow proxy confi 
New |209213 | UEFI Loader shows only black screen with Nvidia G 
New |210804 | installerconfig - using ZFS create in custom scri 
New |223470 | freebsd-update: Cannot identify running kernel (/ 
New |230620 | "install -d" issue
New |235085 | [PATCH] Option to make rc.d/sysctl more verbose ( 
New |252123 | fetch(3): Fix wrong usage of proxy when request i 
Open|177821 | sysctl: Some security.jail nodes are funky, dupli 
Open|182466 | [headers] [patch] make  self-contained  
Open|183618 | [panic] Dell PowerEdge R620 -- PERC H710 Mini (mf 
Open|187015 | agpgart: Panic make_dev_credv: bad si_name (error 
Open|192573 | Add ps(1) option: Print process start time in sec 
Open|194925 | [pf] [ifconfig] interface group keywords do not w 
Open|197921 | scheduler: Allow non-migratable threads to bind t 
Open|206528 | Emulex LPe 16002 FC HBA Not Recognized by oce(4)  
Open|206649 | cyapa(4): Add common gestures for Cypress APA I2C 
Open|207940 | stand/efi/boot1: Add boot partition selection 
Open|212608 | sockstat(1) and lsof(8) can not identity the owne 
Open|220246 | syslogd does not send RFC3164-conformant messages 
Open|221305 | Mouse cursor loss when moving cursor while loadin 
Open|221550 | kern.bootfile returns only /kernel on mips64 (ERL 
Open|221854 | makefs: Reject UFS labels that are too long to fi 
Open|226893 | freebsd-update: Support patchlevel argument for f 
Open|231810 | [build] release always fails with "mkimg: partiti 
Open|233578 | Unprivileged local user can prevent other users l 
Open|233988 | freebsd-update: Improve progress output on termin 
Open|236718 | system panics with message: vm_fault_hold: fault  
Open|237287 | moused(8) ignores button release events in virtua 
Open|237924 | Possible infinite loop in function empty_aux_buff 
Open|238183 | cam/scsi/scsi_sa.c: warnings issued by static ana 
Open|238486 | Possible buffer overflow bug in sc_allocate_keybo 
Open|238550 | Touchpad (via SMBus) not working: Synaptics (SYN1 
Open|238638 | mfi: Remove unnecessary pointer printing in mfi.c 
Open|238837 | init: Remove P_SYSTEM flag from PID 1 to allow ea 
Open|241697 | i915kms: Kernel panic loading module on custom ke 
Open|247132 | Fix build error: use of undeclared identifier 'cp 
Open|248352 | mfi(4): Remove RAID map sync functionality
Open|257149 | CFLAGS not passed to whole build  
New |260138 | TPM2 Support in bootloader / kernel in order to r 
New |261306 | Geli rc.d script does not support insertion of US 
New |262957 | mpr(4), mps(4): change formally to formerly   
Open|179832 | manual page of mac_from_text suggests incorrect f 

47 problems total for which you should take action.


[Bug 127230] [ipfw] [patch] Feature request to add UID and/or GID logging data to ipfw logging with uid rules.

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=127230

Dan Mahoney  changed:

   What|Removed |Added

 CC||free...@gushi.org

--- Comment #4 from Dan Mahoney  ---
Hey there all,

Every once in a while a problem comes up many years later (in this case, it's
trying to track down a user with a compromised wordpress install) and you say
"Gee, I remember seeing something on a mailing list about this".

Oh wait, it's me.   I posted the thing to the mailing list.

This was a fairly trivial change.  Assuming I can get the original patches
again, any chance it can be included?

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 263620] beinstall.sh with a port: installkernel failed, chrooted make in /tmp/beinstall.⋯/mnt failed

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=263620

Graham Perrin  changed:

   What|Removed |Added

   See Also|https://bugs.freebsd.org/bu |
   |gzilla/show_bug.cgi?id=2632 |
   |08  |

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 263208] beinstall.sh NO_PKG_UPGRADE="YES" seems to not skip. Updating FreeBSD repository catalogue...

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=263208

Graham Perrin  changed:

   What|Removed |Added

   See Also|https://bugs.freebsd.org/bu |
   |gzilla/show_bug.cgi?id=2634 |
   |94, |
   |https://bugs.freebsd.org/bu |
   |gzilla/show_bug.cgi?id=2636 |
   |20  |

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 265632] sound: add Lenovo Legion 5 speakers sound output

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265632

Nuno Teixeira  changed:

   What|Removed |Added

   Keywords||patch-ready

-- 
You are receiving this mail because:
You are the assignee for the bug.


[Bug 265632] sound: add Lenovo Legion 5 speakers sound output

2022-08-28 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265632

Nuno Teixeira  changed:

   What|Removed |Added

Summary|sound: add patch for Lenovo |sound: add Lenovo Legion 5
   |Legion 5 Intel  |speakers sound output

-- 
You are receiving this mail because:
You are the assignee for the bug.