Thanks Jessica for all the reviews. I guess we can let this die here
and I'll carry it downstream. At least it's all here for others to
see.
Thanks,
Will
On Wed, Feb 3, 2021 at 7:54 AM Greg Kroah-Hartman
wrote:
>
> On Wed, Feb 03, 2021 at 04:46:16PM +0100, Jessica Yu wro
Hi Christoph,
Did you see Saravana's response to you in version 5? I'd love to hear
your thoughts.
Thanks,
Will
On Mon, Feb 1, 2021 at 11:42 PM Christoph Hellwig wrote:
>
> Just to deposit my opposition again: I don't think this hack for your
> out of tree builds has any place in the kernel tr
Hi Jessica and Masahiro,
Could I get a final look at the latest patchset please? All the issues
have been addressed thus far.
Thanks,
Will
On Thu, Jan 21, 2021 at 1:36 PM Will McVicker wrote:
>
> Config MODULE_SCMVERSION introduces a new module attribute --
> `scmversion` -- which ca
:
> cat /sys/module/MODULENAME/scmversion
Signed-off-by: Will McVicker
---
Changelog since v5:
- Simplified scripts/Makefile.modpost to not check for a relative M= path
- Added space between -v and $(module_scmversion)
- Updated modpost.c to not check for a missing argument to -v
Documentati
les in baz, and create objects in foo/bar/baz.
>
> Currently, it works like that because of the presence of VPATH and
> --include-dir=$(abs_srctree), but this behavior is subtle and cannot
> be guaranteed in the future. I do not know if it was even officially
> supported ever.
>
calversion to make sure
everything was defined as expected.
Thanks,
Will
On Tue, Jan 19, 2021 at 1:48 PM Will McVicker wrote:
>
> Thanks for the review Masahiro! I'll upload v6 shortly.
>
> --Will
>
> On Sun, Jan 17, 2021 at 7:21 AM Masahiro Yamada wrote:
> >
> >
Thanks for the review Masahiro! I'll upload v6 shortly.
--Will
On Sun, Jan 17, 2021 at 7:21 AM Masahiro Yamada wrote:
>
> On Fri, Jan 8, 2021 at 9:30 AM Will McVicker wrote:
> >
> > Config MODULE_SCMVERSION introduces a new module attribute --
> > `scmversion` --
Thanks Jessica for the reviews!
--Will
On Tue, Jan 12, 2021 at 7:55 AM Jessica Yu wrote:
>
> +++ Will McVicker [08/01/21 00:30 +]:
> >Config MODULE_SCMVERSION introduces a new module attribute --
> >`scmversion` -- which can be used to identify a given module's SCM
&g
Hi Jiri,
I noticed this hasn't merged yet. So just sending a friendly reminder.
Thanks,
Will
On Thu, Dec 17, 2020 at 10:42 AM Will McVicker wrote:
>
> Great! Thanks for the reply.
>
> --Will
>
> On Thu, Dec 17, 2020 at 2:19 AM Jiri Kosina wrote:
> >
> >
:
> cat /sys/module/MODULENAME/scmversion
Signed-off-by: Will McVicker
---
Documentation/ABI/stable/sysfs-module | 18 ++
include/linux/module.h| 1 +
init/Kconfig | 14 ++
kernel/module.c | 2 ++
dated the Documentation in v5.
Thanks,
Will
On Thu, Jan 7, 2021 at 6:31 AM Jessica Yu wrote:
>
> +++ Will McVicker [06/01/21 10:44 -0800]:
> >Thanks for the vacation notice Jessica! I'm just letting you know I'm
> >back as well and am happy to respond to any concerns
Thanks for the vacation notice Jessica! I'm just letting you know I'm
back as well and am happy to respond to any concerns regarding v4 when
you get all caught up.
I hope you had a relaxing holiday :)
Thanks,
Will
On Fri, Dec 18, 2020 at 4:01 AM Jessica Yu wrote:
>
> +++ Will
Great! Thanks for the reply.
--Will
On Thu, Dec 17, 2020 at 2:19 AM Jiri Kosina wrote:
>
> On Mon, 14 Dec 2020, Will McVicker wrote:
>
> > > The HID subsystem allows an "HID report field" to have a different
> > > number of "values" and "usa
:
> cat /sys/module/MODULENAME/scmversion
Signed-off-by: Will McVicker
---
Changelog since v3:
- Dropped [PATCH v2 1/2] scripts/setlocalversion: allow running in a subdir
Documentation/ABI/stable/sysfs-module | 18 ++
include/linux/module.h
On Fri, Dec 11, 2020 at 04:33:59PM +0100, Jessica Yu wrote:
> Hi Will,
>
> +++ Will McVicker [08/12/20 20:05 +]:
> > Getting the scmversion using scripts/setlocalversion currently only
> > works when run at the root of a git or mecurial project. This was
> > introd
On Sat, Dec 05, 2020 at 12:48:48AM +, Will McVicker wrote:
> The HID subsystem allows an "HID report field" to have a different
> number of "values" and "usages" when it is allocated. When a field
> struct is created, the size of the usage array is guar
:
> cat /sys/module/MODULENAME/scmversion
Signed-off-by: Will McVicker
---
Documentation/ABI/stable/sysfs-module | 18 ++
include/linux/module.h| 1 +
init/Kconfig | 12
kernel/module.c | 2 ++
found.
Thanks,
Will
Will McVicker (2):
scripts/setlocalversion: allow running in a subdir
modules: introduce the MODULE_SCMVERSION config
Documentation/ABI/stable/sysfs-module | 18 ++
include/linux/module.h| 1 +
init/Kconfig | 12
ALVERSION_AUTO is not set" arch/* | wc -l
89
So let's bring back this functionality so that we can use
scripts/setlocalversion to capture the SCM version of external modules
that reside within subdirectories of an SCM project.
Signed-off-by: Will McVicker
---
scripts/setlocalversion | 5
On Mon, Dec 07, 2020 at 07:24:16PM +0100, Greg KH wrote:
> On Mon, Dec 07, 2020 at 09:55:48AM -0800, Will McVicker wrote:
> > On Sat, Dec 05, 2020 at 09:59:57AM +0100, Greg KH wrote:
> > > On Sat, Dec 05, 2020 at 12:48:48AM +0000, Will McVicker wrote:
> > > > Th
On Sat, Dec 05, 2020 at 09:59:57AM +0100, Greg KH wrote:
> On Sat, Dec 05, 2020 at 12:48:48AM +0000, Will McVicker wrote:
> > The HID subsystem allows an "HID report field" to have a different
> > number of "values" and "usages" when it is allocated. Wh
is leads to
a potential out-of-bounds write in
__hidinput_change_resolution_multipliers() and an out-of-bounds read in
hidinput_count_leds().
To fix this, let's make sure that both the usage and value arrays are
the same size.
Signed-off-by: Will McVicker
---
drivers/hid/hid-core.c | 6 +++---
On Fri, Dec 04, 2020 at 06:18:08PM +, Christoph Hellwig wrote:
> On Fri, Dec 04, 2020 at 10:13:56AM -0800, Will McVicker wrote:
> > On Fri, Dec 04, 2020 at 07:51:59AM +, Christoph Hellwig wrote:
> > > I think your decription still shows absolutely no benefit for the
>
On Fri, Dec 04, 2020 at 07:51:59AM +, Christoph Hellwig wrote:
> I think your decription still shows absolutely no benefit for the
> kernel, so I'not sure why anyone would want to waste time on this.
Hi Christoph,
Did you get a chance to read my earlier responses regarding the uses for
in-tree
anges")
Signed-off-by: Edmond Chung
Signed-off-by: Andrew Chant
Signed-off-by: Will McVicker
---
drivers/gpio/gpiolib.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index 089ddcaa9bc6..6e3c4d7a7d14 100644
--- a/drivers/gpio/gpiolib.c
+++
This patch fixes commit 2ab73c6d8323 ("gpio: Support GPIO controllers
without pin-ranges") where an imbalance of requests and frees to the
pinctrl driver were introduced.
Edmond Chung (1):
gpiolib: Don't free if pin ranges are not defined
drivers/gpio/gpiolib.c | 5 +
1 file changed, 5 ins
t; cat /sys/module//scmversion
Signed-off-by: Will McVicker
---
Documentation/ABI/stable/sysfs-module | 17 +
include/linux/module.h| 1 +
kernel/module.c | 2 ++
scripts/Makefile.modpost | 20
scrip
ALVERSION_AUTO is not set" arch/* | wc -l
89
So let's bring back this functionality so that we can use
scripts/setlocalversion to capture the SCM version of external modules
that reside within subdirectories of an SCM project.
Signed-off-by: Will McVicker
---
scripts/setlocalversion | 5
loaded. With that being said,
adding this scmversion attribute ensures that you can *always* get the correct
SCM version of the module that loaded.
Please take a look at the updated patch and provide any comments you find.
Thanks,
Will
Will McVicker (2):
scripts/setlocalversion: allow running in a
323b9
vermagic: 5.10.0-rc4-00110-gd83461f36865 SMP mod_unload
$ modinfo simple_module.ko | egrep 'scmversion|vermagic'
scmversion: svn1
vermagic: 5.10.0-rc4-00110-gd83461f36865 SMP mod_unload
Will McVicker (2):
scripts/setlocalversion: allow running in a subdir
ALVERSION_AUTO is not set" arch/* | wc -l
89
So let's bring back this functionality so that we can use
scripts/setlocalversion to capture the SCM version of external modules
that reside within subdirectories of an SCM project.
Signed-off-by: Will McVicker
---
scripts/setlocalversion | 5
t; cat /sys/module//scmversion
Signed-off-by: Will McVicker
---
include/linux/module.h | 1 +
kernel/module.c | 20
scripts/Makefile.modpost | 19 +--
scripts/mod/modpost.c| 28 +++-
4 files changed, 65 insertions
sigreturn trampolines from vDSO")
Signed-off-by: Will McVicker
---
arch/arm64/include/asm/vdso.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/include/asm/vdso.h b/arch/arm64/include/asm/vdso.h
index f99dcb94b438..a7384379e8e1 100644
--- a/arch/arm64/include/a
Hi Pablo,
> This patch is much smaller and if you confirm this is address the
> issue, then this is awesome.
Yes, I can confirm the updated patch does fix the kernel panic. I have retested
on the Pixel 4 XL with version 4.14.180. Please see the updated patchset v3.
Thanks,
Will
Will Mc
0x31c/0x4d4
netlink_rcv_skb+0x100/0x184
nfnetlink_rcv+0xf4/0x180
netlink_unicast+0x360/0x770
netlink_sendmsg+0x5a0/0x6a4
___sys_sendmsg+0x314/0x46c
SyS_sendmsg+0xb4/0x108
el0_svc_naked+0x34/0x38
Fixes: c1d10adb4a521 ("[NETFILTER]: Add ctnetlink port for nf_conntrack")
Signed-off-by
nfnetlink_rcv_msg+0x31c/0x4d4
netlink_rcv_skb+0x100/0x184
nfnetlink_rcv+0xf4/0x180
netlink_unicast+0x360/0x770
netlink_sendmsg+0x5a0/0x6a4
___sys_sendmsg+0x314/0x46c
SyS_sendmsg+0xb4/0x108
el0_svc_naked+0x34/0x38
Signed-off-by: Will McVicker
---
net/ipv4/netfilter/nf_nat_l3proto_ipv4.c | 6
struct"), but the 4.19 and below LTS branches remain vulnerable.
I have verifed the OOB kernel panic is fixed with this patch on both the
4.19 and 4.14 kernels using the approariate hardware.
Please review the fix and apply to branches 4.19.y, 4.14.y, 4.9.y and
4.4.y.
Thanks,
Will
Will Mc
37 matches
Mail list logo
