* Re: linux-next: Tree for Sep 2 (exfat) [not found] <20190902224310.208575dc@canb.auug.org.au> @ 2019-09-02 17:39 ` Randy Dunlap 2019-09-02 17:42 ` Greg KH 2019-09-02 17:46 ` Greg KH 0 siblings, 2 replies; 6+ messages in thread From: Randy Dunlap @ 2019-09-02 17:39 UTC (permalink / raw) To: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On 9/2/19 5:43 AM, Stephen Rothwell wrote: > Hi all, > > News: I will only be doing 2 more releases before I leave for Kernel > Summit (there may be some reports on Thursday, but I doubt I will have > time to finish the full release) and then no more until Sept 30. > > Changes since 20190830: > Hi, I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. Maybe its Kconfig should also say depends on BLOCK ? -- ~Randy ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: Tree for Sep 2 (exfat) 2019-09-02 17:39 ` linux-next: Tree for Sep 2 (exfat) Randy Dunlap @ 2019-09-02 17:42 ` Greg KH 2019-09-02 17:46 ` Greg KH 1 sibling, 0 replies; 6+ messages in thread From: Greg KH @ 2019-09-02 17:42 UTC (permalink / raw) To: Randy Dunlap Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: > On 9/2/19 5:43 AM, Stephen Rothwell wrote: > > Hi all, > > > > News: I will only be doing 2 more releases before I leave for Kernel > > Summit (there may be some reports on Thursday, but I doubt I will have > > time to finish the full release) and then no more until Sept 30. > > > > Changes since 20190830: > > > > Hi, > I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. > Maybe its Kconfig should also say > depends on BLOCK > ? Yes it should, sorry, will go fix that up now... greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: Tree for Sep 2 (exfat) 2019-09-02 17:39 ` linux-next: Tree for Sep 2 (exfat) Randy Dunlap 2019-09-02 17:42 ` Greg KH @ 2019-09-02 17:46 ` Greg KH 2019-09-02 18:07 ` Randy Dunlap 1 sibling, 1 reply; 6+ messages in thread From: Greg KH @ 2019-09-02 17:46 UTC (permalink / raw) To: Randy Dunlap Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: > On 9/2/19 5:43 AM, Stephen Rothwell wrote: > > Hi all, > > > > News: I will only be doing 2 more releases before I leave for Kernel > > Summit (there may be some reports on Thursday, but I doubt I will have > > time to finish the full release) and then no more until Sept 30. > > > > Changes since 20190830: > > > > Hi, > I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. > Maybe its Kconfig should also say > depends on BLOCK > ? Here's what I committed to my tree: From e2b880d3d1afaa5cad108c29be3e307b1917d195 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Date: Mon, 2 Sep 2019 19:45:06 +0200 Subject: staging: exfat: make exfat depend on BLOCK This should fix a build error in some configurations when CONFIG_BLOCK is not selected. Also properly set the dependancy for no FAT support at the same time. Reported-by: Randy Dunlap <rdunlap@infradead.org> Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- drivers/staging/exfat/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/exfat/Kconfig b/drivers/staging/exfat/Kconfig index f52129c67f97..290dbfc7ace1 100644 --- a/drivers/staging/exfat/Kconfig +++ b/drivers/staging/exfat/Kconfig @@ -1,11 +1,13 @@ config EXFAT_FS tristate "exFAT fs support" + depends on BLOCK select NLS help This adds support for the exFAT file system. config EXFAT_DONT_MOUNT_VFAT bool "Prohibit mounting of fat/vfat filesysems by exFAT" + depends on EXFAT_FS default y help By default, the exFAT driver will only mount exFAT filesystems, and refuse -- 2.23.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: linux-next: Tree for Sep 2 (exfat) 2019-09-02 17:46 ` Greg KH @ 2019-09-02 18:07 ` Randy Dunlap 2019-09-02 18:11 ` Joe Perches 0 siblings, 1 reply; 6+ messages in thread From: Randy Dunlap @ 2019-09-02 18:07 UTC (permalink / raw) To: Greg KH Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On 9/2/19 10:46 AM, Greg KH wrote: > On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: >> On 9/2/19 5:43 AM, Stephen Rothwell wrote: >>> Hi all, >>> >>> News: I will only be doing 2 more releases before I leave for Kernel >>> Summit (there may be some reports on Thursday, but I doubt I will have >>> time to finish the full release) and then no more until Sept 30. >>> >>> Changes since 20190830: >>> >> >> Hi, >> I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. >> Maybe its Kconfig should also say >> depends on BLOCK >> ? > > Here's what I committed to my tree: > > > From e2b880d3d1afaa5cad108c29be3e307b1917d195 Mon Sep 17 00:00:00 2001 > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Date: Mon, 2 Sep 2019 19:45:06 +0200 > Subject: staging: exfat: make exfat depend on BLOCK > > This should fix a build error in some configurations when CONFIG_BLOCK > is not selected. Also properly set the dependancy for no FAT support at > the same time. > > Reported-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> That works. Thanks. Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested > --- > drivers/staging/exfat/Kconfig | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/staging/exfat/Kconfig b/drivers/staging/exfat/Kconfig > index f52129c67f97..290dbfc7ace1 100644 > --- a/drivers/staging/exfat/Kconfig > +++ b/drivers/staging/exfat/Kconfig > @@ -1,11 +1,13 @@ > config EXFAT_FS > tristate "exFAT fs support" > + depends on BLOCK > select NLS > help > This adds support for the exFAT file system. > > config EXFAT_DONT_MOUNT_VFAT > bool "Prohibit mounting of fat/vfat filesysems by exFAT" > + depends on EXFAT_FS > default y > help > By default, the exFAT driver will only mount exFAT filesystems, and refuse > -- ~Randy ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: Tree for Sep 2 (exfat) 2019-09-02 18:07 ` Randy Dunlap @ 2019-09-02 18:11 ` Joe Perches 2019-09-02 18:44 ` Greg KH 0 siblings, 1 reply; 6+ messages in thread From: Joe Perches @ 2019-09-02 18:11 UTC (permalink / raw) To: Randy Dunlap, Greg KH Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On Mon, 2019-09-02 at 11:07 -0700, Randy Dunlap wrote: > On 9/2/19 10:46 AM, Greg KH wrote: > > On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: > > > On 9/2/19 5:43 AM, Stephen Rothwell wrote: > > > > Hi all, > > > > > > > > News: I will only be doing 2 more releases before I leave for Kernel > > > > Summit (there may be some reports on Thursday, but I doubt I will have > > > > time to finish the full release) and then no more until Sept 30. > > > > > > > > Changes since 20190830: > > > > > > > > > > Hi, > > > I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. > > > Maybe its Kconfig should also say > > > depends on BLOCK > > > ? > > > > Here's what I committed to my tree: > > > > > > From e2b880d3d1afaa5cad108c29be3e307b1917d195 Mon Sep 17 00:00:00 2001 > > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Date: Mon, 2 Sep 2019 19:45:06 +0200 > > Subject: staging: exfat: make exfat depend on BLOCK > > > > This should fix a build error in some configurations when CONFIG_BLOCK > > is not selected. Also properly set the dependancy for no FAT support at > > the same time. > > > > Reported-by: Randy Dunlap <rdunlap@infradead.org> > > Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu> > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > That works. Thanks. > Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested > > > --- > > drivers/staging/exfat/Kconfig | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/staging/exfat/Kconfig b/drivers/staging/exfat/Kconfig > > index f52129c67f97..290dbfc7ace1 100644 > > --- a/drivers/staging/exfat/Kconfig > > +++ b/drivers/staging/exfat/Kconfig > > @@ -1,11 +1,13 @@ > > config EXFAT_FS > > tristate "exFAT fs support" > > + depends on BLOCK > > select NLS > > help > > This adds support for the exFAT file system. > > > > config EXFAT_DONT_MOUNT_VFAT > > bool "Prohibit mounting of fat/vfat filesysems by exFAT" > > + depends on EXFAT_FS > > default y > > help > > By default, the exFAT driver will only mount exFAT filesystems, and refuse I think this last one is backwards and should be config EXFAT_ALLOW_MOUNT_VFAT and default n > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: linux-next: Tree for Sep 2 (exfat) 2019-09-02 18:11 ` Joe Perches @ 2019-09-02 18:44 ` Greg KH 0 siblings, 0 replies; 6+ messages in thread From: Greg KH @ 2019-09-02 18:44 UTC (permalink / raw) To: Joe Perches Cc: Randy Dunlap, Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Valdis Kletnieks, devel@driverdev.osuosl.org, Linux FS Devel On Mon, Sep 02, 2019 at 11:11:45AM -0700, Joe Perches wrote: > On Mon, 2019-09-02 at 11:07 -0700, Randy Dunlap wrote: > > On 9/2/19 10:46 AM, Greg KH wrote: > > > On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: > > > > On 9/2/19 5:43 AM, Stephen Rothwell wrote: > > > > > Hi all, > > > > > > > > > > News: I will only be doing 2 more releases before I leave for Kernel > > > > > Summit (there may be some reports on Thursday, but I doubt I will have > > > > > time to finish the full release) and then no more until Sept 30. > > > > > > > > > > Changes since 20190830: > > > > > > > > > > > > > Hi, > > > > I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. > > > > Maybe its Kconfig should also say > > > > depends on BLOCK > > > > ? > > > > > > Here's what I committed to my tree: > > > > > > > > > From e2b880d3d1afaa5cad108c29be3e307b1917d195 Mon Sep 17 00:00:00 2001 > > > From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > Date: Mon, 2 Sep 2019 19:45:06 +0200 > > > Subject: staging: exfat: make exfat depend on BLOCK > > > > > > This should fix a build error in some configurations when CONFIG_BLOCK > > > is not selected. Also properly set the dependancy for no FAT support at > > > the same time. > > > > > > Reported-by: Randy Dunlap <rdunlap@infradead.org> > > > Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu> > > > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > > That works. Thanks. > > Acked-by: Randy Dunlap <rdunlap@infradead.org> # build-tested > > > > > --- > > > drivers/staging/exfat/Kconfig | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/staging/exfat/Kconfig b/drivers/staging/exfat/Kconfig > > > index f52129c67f97..290dbfc7ace1 100644 > > > --- a/drivers/staging/exfat/Kconfig > > > +++ b/drivers/staging/exfat/Kconfig > > > @@ -1,11 +1,13 @@ > > > config EXFAT_FS > > > tristate "exFAT fs support" > > > + depends on BLOCK > > > select NLS > > > help > > > This adds support for the exFAT file system. > > > > > > config EXFAT_DONT_MOUNT_VFAT > > > bool "Prohibit mounting of fat/vfat filesysems by exFAT" > > > + depends on EXFAT_FS > > > default y > > > help > > > By default, the exFAT driver will only mount exFAT filesystems, and refuse > > I think this last one is backwards and should be > > config EXFAT_ALLOW_MOUNT_VFAT > and > default n This is just a "bandage" for now, it should be fixed properly by ripping out the code wherever possible and making this the default functionality anyway. This just keeps the code from breaking existing working systems if users accidentally build this module. thanks, greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-09-02 18:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20190902224310.208575dc@canb.auug.org.au>
2019-09-02 17:39 ` linux-next: Tree for Sep 2 (exfat) Randy Dunlap
2019-09-02 17:42 ` Greg KH
2019-09-02 17:46 ` Greg KH
2019-09-02 18:07 ` Randy Dunlap
2019-09-02 18:11 ` Joe Perches
2019-09-02 18:44 ` Greg KH
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).