[OS-BUILD PATCH] Enable CONFIG_CIFS_SMB_DIRECT for ARK

2021-11-16 Thread Ronnie Sahlberg (via Email Bridge)
From: Ronnie Sahlberg 

Enable CONFIG_CIFS_SMB_DIRECT for ARK

Signed-off-by: Ronnie Sahlberg 

diff --git a/redhat/configs/ark/generic/CONFIG_CIFS_SMB_DIRECT 
b/redhat/configs/ark/generic/CONFIG_CIFS_SMB_DIRECT
new file mode 100644
index blahblah..blahblah 100644
--- /dev/null
+++ b/redhat/configs/ark/generic/CONFIG_CIFS_SMB_DIRECT
@@ -0,0 +1 @@
+CONFIG_CIFS_SMB_DIRECT=y
diff --git a/redhat/rhel_files/filter-modules.sh.rhel 
b/redhat/rhel_files/filter-modules.sh.rhel
index blahblah..blahblah 100755
--- a/redhat/rhel_files/filter-modules.sh.rhel
+++ b/redhat/rhel_files/filter-modules.sh.rhel
@@ -31,7 +31,7 @@ scsidrvs="aacraid aic7xxx aic94xx be2iscsi bfa bnx2i bnx2fc 
csiostor cxgbi esas2
 
 usbdrvs="atm image misc serial wusbcore"
 
-fsdrvs="affs befs coda cramfs ecryptfs hfs hfsplus jfs minix ncpfs nilfs2 
ocfs2 reiserfs romfs squashfs sysv ubifs ufs"
+fsdrvs="affs befs cifs coda cramfs ecryptfs hfs hfsplus jfs minix ncpfs nilfs2 
ocfs2 reiserfs romfs squashfs sysv ubifs ufs"
 
 netprots="6lowpan appletalk atm ax25 batman-adv bluetooth can dccp dsa 
ieee802154 irda l2tp mac80211 mac802154 mpls netrom nfc rds rfkill rose sctp 
smc wireless"
 

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1502
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] redhat/configs/evaluate_configs: Add find dead configs option

2021-11-16 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1498#note_735095272

@prarit - thanks!
/me mumbles something about ark commit id? then runs...
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH] Fix up PREEMPT configs

2021-11-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1500#note_735087605

> Yes, I get that part. I was talking about os-build:50d16ce3099, which has
> a one line commit message and says nothing about keeping VOLUNTARY etc.
>
> The commit message on what you posted here looked more like what I was
> expecting. But maybe I'm just confused...

Those are Fedora commits that have no impact on RHEL and do not go through MRs
so as to not waste RHEL engineers time with review requests which are not
relevant to them. As I had to fix up both pending-common and fedora anyway, it
was done in the same commit, but no RHEL configs outside of pending were
changed.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH] Fix up PREEMPT configs

2021-11-16 Thread Phil Auld
On Tue, Nov 16, 2021 at 09:19:09PM - Justin M. Forbes (via Email Bridge) 
wrote:
> From: Justin M. Forbes on gitlab.com
> https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1500#note_735053347
> 
> > This commit, or something like it, seems to have gone in but without
> > much of a commit message.  Was that on purpsoe?
> 
> 
> Yes, there are files in pending-common that go in without a merge request.
> Typically those are set to default values because they are new config options,
> but in this case it had to be set by hand because these are not new config
> items, but are currently set in a way that is unbuildable. Everything in
> pending-common is set without review, and should have a corresponding MR to
> move the items from pending-common and into the correct place.   If we did not
> do this with pending common, this tree would never build until every single
> config MR was reviewed and merged.  It is a method to keep the tree usable,
> but make sure that nothing makes it into final RHEL configs until reviewed.
> 
> > Also, I noticed that the dynamic preemption is only enabled for x86_64 due
> > to none of the other archs enabling HAVE_PREEMPT_DYNAMIC. There are arm
> > patches posted but not yet merged to enable this there.
> 
> This is true, but as other arches add HAVE_PREEMPT_DYNAMIC, this commit
> message will still be what is seen when people check the values for the
> configs.  In the case of aarch64, I expect that will be soon. In the case of
> power, it may or may not happen, but the commit message still explains the
> situation, as it is PREEMPT_VOLUNTARY, it just can't be overridden unless
> patches are merged to enable dynamic.  S390 is still not dynamic and ships
> with PREEMPT_NONE. In all of these cases, it matches what is currently in RHEL
> 9/RHEL 8, and stable Fedora with the addition of dynamic override allowed
> where the arch supports it.


Yes, I get that part. I was talking about os-build:50d16ce3099, which has
a one line commit message and says nothing about keeping VOLUNTARY etc. 

The commit message on what you posted here looked more like what I was
expecting. But maybe I'm just confused...


Cheers,
Phil

> ___
> kernel mailing list -- kernel@lists.fedoraproject.org
> To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
> Fedora Code of Conduct: 
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: 
> https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
> Do not reply to spam on the list, report it: 
> https://pagure.io/fedora-infrastructure

-- 
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[OS-BUILD PATCH] kasan: test: Silence intentional read overflow warnings

2021-11-16 Thread Justin M. Forbes (via Email Bridge)
From: Kees Cook 

kasan: test: Silence intentional read overflow warnings

As done in commit d73dad4eb5ad ("kasan: test: bypass __alloc_size checks")
for __write_overflow warnings, also silence some more cases that trip
the __read_overflow warnings seen in 5.16-rc1[1]:

In file included from /kisskb/src/include/linux/string.h:253,
 from /kisskb/src/include/linux/bitmap.h:10,
 from /kisskb/src/include/linux/cpumask.h:12,
 from /kisskb/src/include/linux/mm_types_task.h:14,
 from /kisskb/src/include/linux/mm_types.h:5,
 from /kisskb/src/include/linux/page-flags.h:13,
 from /kisskb/src/arch/arm64/include/asm/mte.h:14,
 from /kisskb/src/arch/arm64/include/asm/pgtable.h:12,
 from /kisskb/src/include/linux/pgtable.h:6,
 from /kisskb/src/include/linux/kasan.h:29,
 from /kisskb/src/lib/test_kasan.c:10:
In function 'memcmp',
inlined from 'kasan_memcmp' at /kisskb/src/lib/test_kasan.c:897:2:
/kisskb/src/include/linux/fortify-string.h:263:25: error: call to 
'__read_overflow' declared with attribute error: detected read beyond size of 
object (1st parameter)
  263 | __read_overflow();
  | ^
In function 'memchr',
inlined from 'kasan_memchr' at /kisskb/src/lib/test_kasan.c:872:2:
/kisskb/src/include/linux/fortify-string.h:277:17: error: call to 
'__read_overflow' declared with attribute error: detected read beyond size of 
object (1st parameter)
  277 | __read_overflow();
  | ^

[1] http://kisskb.ellerman.id.au/kisskb/buildresult/14660585/log/

Cc: Marco Elver 
Cc: Andrey Ryabinin 
Cc: Alexander Potapenko 
Cc: Andrey Konovalov 
Cc: Dmitry Vyukov 
Cc: kasan-...@googlegroups.com
Fixes: d73dad4eb5ad ("kasan: test: bypass __alloc_size checks")
Signed-off-by: Kees Cook 
Reviewed-by: Andrey Konovalov 

diff --git a/lib/test_kasan.c b/lib/test_kasan.c
index blahblah..blahblah 100644
--- a/lib/test_kasan.c
+++ b/lib/test_kasan.c
@@ -869,6 +869,7 @@ static void kasan_memchr(struct kunit *test)
ptr = kmalloc(size, GFP_KERNEL | __GFP_ZERO);
KUNIT_ASSERT_NOT_ERR_OR_NULL(test, ptr);
 
+   OPTIMIZER_HIDE_VAR(size);
KUNIT_EXPECT_KASAN_FAIL(test,
kasan_ptr_result = memchr(ptr, '1', size + 1));
 
@@ -894,6 +895,7 @@ static void kasan_memcmp(struct kunit *test)
KUNIT_ASSERT_NOT_ERR_OR_NULL(test, ptr);
memset(arr, 0, sizeof(arr));
 
+   OPTIMIZER_HIDE_VAR(size);
KUNIT_EXPECT_KASAN_FAIL(test,
kasan_int_result = memcmp(ptr, arr, size+1));
kfree(ptr);

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1501
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH] Fix up PREEMPT configs

2021-11-16 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1500#note_735053347

> This commit, or something like it, seems to have gone in but without
> much of a commit message.  Was that on purpsoe?


Yes, there are files in pending-common that go in without a merge request.
Typically those are set to default values because they are new config options,
but in this case it had to be set by hand because these are not new config
items, but are currently set in a way that is unbuildable. Everything in
pending-common is set without review, and should have a corresponding MR to
move the items from pending-common and into the correct place.   If we did not
do this with pending common, this tree would never build until every single
config MR was reviewed and merged.  It is a method to keep the tree usable,
but make sure that nothing makes it into final RHEL configs until reviewed.

> Also, I noticed that the dynamic preemption is only enabled for x86_64 due
> to none of the other archs enabling HAVE_PREEMPT_DYNAMIC. There are arm
> patches posted but not yet merged to enable this there.

This is true, but as other arches add HAVE_PREEMPT_DYNAMIC, this commit
message will still be what is seen when people check the values for the
configs.  In the case of aarch64, I expect that will be soon. In the case of
power, it may or may not happen, but the commit message still explains the
situation, as it is PREEMPT_VOLUNTARY, it just can't be overridden unless
patches are merged to enable dynamic.  S390 is still not dynamic and ships
with PREEMPT_NONE. In all of these cases, it matches what is currently in RHEL
9/RHEL 8, and stable Fedora with the addition of dynamic override allowed
where the arch supports it.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] redhat/configs/evaluate_configs: Add find dead configs option

2021-11-16 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1498#note_735002771

Sorry about that, put that into the "why aren't you thinking what I'm
thinking" basket.  I've updated the description on the 3rd commit.  I looked
at separating things out and it just got messy for no reason.  It was cleaner
to show things this way IMO.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH] Fix up PREEMPT configs

2021-11-16 Thread Phil Auld

Hi Justin,

On Mon, Nov 15, 2021 at 06:28:11PM - Justin M. Forbes (via Email Bridge) 
wrote:
> From: Justin M. Forbes 
> 
> Fix up PREEMPT configs
> 
> While preempt_dynamic was greatly improved at the beginning of the merge
> window, the config to make things work was messy and the new BEHAVIOUR
> configs were confusing to some. With commit
> a8b76910e465d718effce0cad306a21fa4f3526b the BEHAVIOUR config options
> are gone and we now default to whichever CONFIG_PREEMPT is selected.
> This MR moves us to the new config options, but should leave the kernel
> behaving as before with PREEMPT_VOLUNTARY as the default with
> PREEMPT_DYNAMIC enabled for all arches but s390 which does not enable
> DYNAMIC and builds with PREEMPT_NONE.
>


This commit, or something like it, seems to have gone in but without
much of a commit message.  Was that on purpsoe?

Also, I noticed that the dynamic preemption is only enabled for x86_64 due
to none of the other archs enabling HAVE_PREEMPT_DYNAMIC. There are arm
patches posted but not yet merged to enable this there.


Cheers,
Phil


> Signed-off-by: Justin M. Forbes 
> 
> diff --git a/redhat/configs/common/generic/CONFIG_PREEMPT 
> b/redhat/configs/common/generic/CONFIG_PREEMPT
> index blahblah..blahblah 100644
> --- a/redhat/configs/common/generic/CONFIG_PREEMPT
> +++ b/redhat/configs/common/generic/CONFIG_PREEMPT
> @@ -1 +1 @@
> -CONFIG_PREEMPT=y
> +# CONFIG_PREEMPT is not set
> diff --git a/redhat/configs/common/generic/CONFIG_PREEMPT_BEHAVIOUR 
> b/redhat/configs/common/generic/CONFIG_PREEMPT_BEHAVIOUR
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/common/generic/CONFIG_PREEMPT_BEHAVIOUR
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT_BEHAVIOUR is not set
> diff --git a/redhat/configs/common/generic/CONFIG_PREEMPT_NONE 
> b/redhat/configs/common/generic/CONFIG_PREEMPT_NONE
> index blahblah..blahblah 100644
> --- a/redhat/configs/common/generic/CONFIG_PREEMPT_NONE
> +++ b/redhat/configs/common/generic/CONFIG_PREEMPT_NONE
> @@ -1 +1 @@
> -CONFIG_PREEMPT_NONE=y
> +# CONFIG_PREEMPT_NONE is not set
> diff --git a/redhat/configs/common/generic/CONFIG_PREEMPT_NONE_BEHAVIOUR 
> b/redhat/configs/common/generic/CONFIG_PREEMPT_NONE_BEHAVIOUR
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/common/generic/CONFIG_PREEMPT_NONE_BEHAVIOUR
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT_NONE_BEHAVIOUR is not set
> diff --git a/redhat/configs/common/generic/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR 
> b/redhat/configs/common/generic/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/common/generic/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR
> +++ /dev/null
> @@ -1 +0,0 @@
> -CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR=y
> diff --git 
> a/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_NONE_BEHAVIOUR 
> b/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_NONE_BEHAVIOUR
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_NONE_BEHAVIOUR
> +++ /dev/null
> @@ -1 +0,0 @@
> -CONFIG_PREEMPT_NONE_BEHAVIOUR=y
> diff --git 
> a/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR 
> b/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT_VOLUNTARY_BEHAVIOUR is not set
> diff --git a/redhat/configs/pending-common/generic/CONFIG_PREEMPT 
> b/redhat/configs/pending-common/generic/CONFIG_PREEMPT
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/pending-common/generic/CONFIG_PREEMPT
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT is not set
> diff --git a/redhat/configs/pending-common/generic/CONFIG_PREEMPT_NONE 
> b/redhat/configs/pending-common/generic/CONFIG_PREEMPT_NONE
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/pending-common/generic/CONFIG_PREEMPT_NONE
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT_NONE is not set
> diff --git a/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_NONE 
> b/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_NONE
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_NONE
> +++ /dev/null
> @@ -1 +0,0 @@
> -CONFIG_PREEMPT_NONE=y
> diff --git 
> a/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY 
> b/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY
> deleted file mode 100644
> index blahblah..blahblah 0
> --- a/redhat/configs/pending-common/generic/s390x/CONFIG_PREEMPT_VOLUNTARY
> +++ /dev/null
> @@ -1 +0,0 @@
> -# CONFIG_PREEMPT_VOLUNTARY is not set
> 
> --
> https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1500
> ___
> kernel ma