Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Thu, May 25, 2023 at 1:30 PM Matthieu Baerts wrote: > > Hi Nick, > > On 24/05/2023 20:56, Nick Desaulniers wrote: > > On Wed, May 24, 2023 at 11:41 AM Nathan Chancellor > > wrote: > >> > >> On Wed, May 24, 2023 at 11:32:32AM -0700, Nick Desaulniers wrote: > >>> On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor > >>> wrote: > > Clang warns: > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: > unannotated fall-through between switch labels > [-Werror,-Wimplicit-fallthrough] > case I915_FORMAT_MOD_X_TILED: > ^ > drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert > 'break;' to avoid fall-through > case I915_FORMAT_MOD_X_TILED: > ^ > break; > 1 error generated. > > Clang is a little more pedantic than GCC, which does not warn when > falling through to a case that is just break or return. Clang's version > is more in line with the kernel's own stance in deprecated.rst, which > states that all switch/case blocks must end in either break, > fallthrough, continue, goto, or return. Add the missing break to silence > the warning. > > Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") > Reported-by: kernel test robot > Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ > Reported-by: Naresh Kamboju > Closes: > https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ > Signed-off-by: Nathan Chancellor > >>> > >>> Thanks for the patch! I've never seen the closes tag before, that's > >>> new to me. Can you tell me more about it? > >> > >> It is new to me (at least in the context of the kernel) as well. I only > >> used it over Link: because checkpatch.pl told me to: > >> > >> WARNING: Reported-by: should be immediately followed by Closes: with a URL > >> to the report > >> #26: > >> Reported-by: kernel test robot > >> Reported-by: Naresh Kamboju > >> > >> WARNING: Reported-by: should be immediately followed by Closes: with a URL > >> to the report > >> #27: > >> Reported-by: Naresh Kamboju > >> Signed-off-by: Nathan Chancellor > >> > >> It was Link: for a bit but commit 44c31888098a ("checkpatch: allow > >> Closes tags with links") changed it to Closes:. Looks odd to me but > >> whatever the linter says I suppose. > >> > >> Thanks for the review! > >> > >> Cheers, > >> Nathan > >> > >>> A few more tags > >>> > >>> Reported-by: Tom Rix > >>> Link: > >>> https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > >>> Reviewed-by: Nick Desaulniers > > > > Ah then I guess my link tag should have been > > > > Closes: > > https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > > > > I hope the author of > > commit 44c31888098a ("checkpatch: allow Closes tags with links") > > has coordinated with the maintainer of b4, so that b4 recognizes Closes > > tags. > > b4 v0.12.2 does not pick up Closes tags. > > I'm sorry for the troubles caused by this series, that was not the > intension. > > When looking at modifying b4 to support the Closes tag, I realised the > Link tag from your previous message [1] was not taken as well. Was it > just me? oh? good find! > > If no, I just sent patches for b4, see [2]. I hope it will help! appreciated! Thank you. > > Cheers, > Matt > > [1] > https://lore.kernel.org/all/CAKwvOd=jzjouunmd3rvc--goa0exphcf6chxua6w1kxjg2a...@mail.gmail.com/ > [2] > https://lore.kernel.org/tools/20230525-closes-tags-v1-0-ed41b1773...@tessares.net/T/ > -- > Tessares | Belgium | Hybrid Access Solutions > www.tessares.net -- Thanks, ~Nick Desaulniers
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Wed, 24 May 2023, Nick Desaulniers wrote: > On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: >> >> Clang warns: >> >> drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated >> fall-through between switch labels [-Werror,-Wimplicit-fallthrough] >> case I915_FORMAT_MOD_X_TILED: >> ^ >> drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert 'break;' >> to avoid fall-through >> case I915_FORMAT_MOD_X_TILED: >> ^ >> break; >> 1 error generated. >> >> Clang is a little more pedantic than GCC, which does not warn when >> falling through to a case that is just break or return. Clang's version >> is more in line with the kernel's own stance in deprecated.rst, which >> states that all switch/case blocks must end in either break, >> fallthrough, continue, goto, or return. Add the missing break to silence >> the warning. >> >> Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") >> Reported-by: kernel test robot >> Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ >> Reported-by: Naresh Kamboju >> Closes: >> https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ >> Signed-off-by: Nathan Chancellor > > Thanks for the patch! I've never seen the closes tag before, that's > new to me. Can you tell me more about it? > > A few more tags > > Reported-by: Tom Rix > Link: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > Reviewed-by: Nick Desaulniers Thanks for the patch and review, pushed to drm-intel-next with s/Link/Closes/. BR, Jani. > > >> --- >> drivers/gpu/drm/i915/display/intel_display.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_display.c >> b/drivers/gpu/drm/i915/display/intel_display.c >> index 0490c6412ab5..6d49e0ab3e85 100644 >> --- a/drivers/gpu/drm/i915/display/intel_display.c >> +++ b/drivers/gpu/drm/i915/display/intel_display.c >> @@ -6008,6 +6008,7 @@ static int intel_async_flip_check_hw(struct >> intel_atomic_state *state, struct in >> plane->base.base.id, >> plane->base.name); >> return -EINVAL; >> } >> + break; >> >> case I915_FORMAT_MOD_X_TILED: >> case I915_FORMAT_MOD_Y_TILED: >> >> --- >> base-commit: 9a2cb1b31c040e2f1b313e2f7921f0f5e6b66d82 >> change-id: >> 20230524-intel_async_flip_check_hw-implicit-fallthrough-c4c40b03802f >> >> Best regards, >> -- >> Nathan Chancellor >> -- Jani Nikula, Intel Open Source Graphics Center
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
Hi Nick, On 24/05/2023 20:56, Nick Desaulniers wrote: > On Wed, May 24, 2023 at 11:41 AM Nathan Chancellor wrote: >> >> On Wed, May 24, 2023 at 11:32:32AM -0700, Nick Desaulniers wrote: >>> On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: Clang warns: drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough] case I915_FORMAT_MOD_X_TILED: ^ drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert 'break;' to avoid fall-through case I915_FORMAT_MOD_X_TILED: ^ break; 1 error generated. Clang is a little more pedantic than GCC, which does not warn when falling through to a case that is just break or return. Clang's version is more in line with the kernel's own stance in deprecated.rst, which states that all switch/case blocks must end in either break, fallthrough, continue, goto, or return. Add the missing break to silence the warning. Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") Reported-by: kernel test robot Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ Reported-by: Naresh Kamboju Closes: https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ Signed-off-by: Nathan Chancellor >>> >>> Thanks for the patch! I've never seen the closes tag before, that's >>> new to me. Can you tell me more about it? >> >> It is new to me (at least in the context of the kernel) as well. I only >> used it over Link: because checkpatch.pl told me to: >> >> WARNING: Reported-by: should be immediately followed by Closes: with a URL >> to the report >> #26: >> Reported-by: kernel test robot >> Reported-by: Naresh Kamboju >> >> WARNING: Reported-by: should be immediately followed by Closes: with a URL >> to the report >> #27: >> Reported-by: Naresh Kamboju >> Signed-off-by: Nathan Chancellor >> >> It was Link: for a bit but commit 44c31888098a ("checkpatch: allow >> Closes tags with links") changed it to Closes:. Looks odd to me but >> whatever the linter says I suppose. >> >> Thanks for the review! >> >> Cheers, >> Nathan >> >>> A few more tags >>> >>> Reported-by: Tom Rix >>> Link: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ >>> Reviewed-by: Nick Desaulniers > > Ah then I guess my link tag should have been > > Closes: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > > I hope the author of > commit 44c31888098a ("checkpatch: allow Closes tags with links") > has coordinated with the maintainer of b4, so that b4 recognizes Closes tags. > b4 v0.12.2 does not pick up Closes tags. I'm sorry for the troubles caused by this series, that was not the intension. When looking at modifying b4 to support the Closes tag, I realised the Link tag from your previous message [1] was not taken as well. Was it just me? If no, I just sent patches for b4, see [2]. I hope it will help! Cheers, Matt [1] https://lore.kernel.org/all/CAKwvOd=jzjouunmd3rvc--goa0exphcf6chxua6w1kxjg2a...@mail.gmail.com/ [2] https://lore.kernel.org/tools/20230525-closes-tags-v1-0-ed41b1773...@tessares.net/T/ -- Tessares | Belgium | Hybrid Access Solutions www.tessares.net
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Wed, May 24, 2023 at 11:41 AM Nathan Chancellor wrote: > > On Wed, May 24, 2023 at 11:32:32AM -0700, Nick Desaulniers wrote: > > On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: > > > > > > Clang warns: > > > > > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated > > > fall-through between switch labels [-Werror,-Wimplicit-fallthrough] > > > case I915_FORMAT_MOD_X_TILED: > > > ^ > > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert > > > 'break;' to avoid fall-through > > > case I915_FORMAT_MOD_X_TILED: > > > ^ > > > break; > > > 1 error generated. > > > > > > Clang is a little more pedantic than GCC, which does not warn when > > > falling through to a case that is just break or return. Clang's version > > > is more in line with the kernel's own stance in deprecated.rst, which > > > states that all switch/case blocks must end in either break, > > > fallthrough, continue, goto, or return. Add the missing break to silence > > > the warning. > > > > > > Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") > > > Reported-by: kernel test robot > > > Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ > > > Reported-by: Naresh Kamboju > > > Closes: > > > https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ > > > Signed-off-by: Nathan Chancellor > > > > Thanks for the patch! I've never seen the closes tag before, that's > > new to me. Can you tell me more about it? > > It is new to me (at least in the context of the kernel) as well. I only > used it over Link: because checkpatch.pl told me to: > > WARNING: Reported-by: should be immediately followed by Closes: with a URL to > the report > #26: > Reported-by: kernel test robot > Reported-by: Naresh Kamboju > > WARNING: Reported-by: should be immediately followed by Closes: with a URL to > the report > #27: > Reported-by: Naresh Kamboju > Signed-off-by: Nathan Chancellor > > It was Link: for a bit but commit 44c31888098a ("checkpatch: allow > Closes tags with links") changed it to Closes:. Looks odd to me but > whatever the linter says I suppose. > > Thanks for the review! > > Cheers, > Nathan > > > A few more tags > > > > Reported-by: Tom Rix > > Link: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > > Reviewed-by: Nick Desaulniers Ah then I guess my link tag should have been Closes: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ I hope the author of commit 44c31888098a ("checkpatch: allow Closes tags with links") has coordinated with the maintainer of b4, so that b4 recognizes Closes tags. b4 v0.12.2 does not pick up Closes tags. -- Thanks, ~Nick Desaulniers
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Wed, May 24, 2023 at 11:32:32AM -0700, Nick Desaulniers wrote: > On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: > > > > Clang warns: > > > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated > > fall-through between switch labels [-Werror,-Wimplicit-fallthrough] > > case I915_FORMAT_MOD_X_TILED: > > ^ > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert > > 'break;' to avoid fall-through > > case I915_FORMAT_MOD_X_TILED: > > ^ > > break; > > 1 error generated. > > > > Clang is a little more pedantic than GCC, which does not warn when > > falling through to a case that is just break or return. Clang's version > > is more in line with the kernel's own stance in deprecated.rst, which > > states that all switch/case blocks must end in either break, > > fallthrough, continue, goto, or return. Add the missing break to silence > > the warning. > > > > Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") > > Reported-by: kernel test robot > > Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ > > Reported-by: Naresh Kamboju > > Closes: > > https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ > > Signed-off-by: Nathan Chancellor > > Thanks for the patch! I've never seen the closes tag before, that's > new to me. Can you tell me more about it? It is new to me (at least in the context of the kernel) as well. I only used it over Link: because checkpatch.pl told me to: WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report #26: Reported-by: kernel test robot Reported-by: Naresh Kamboju WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report #27: Reported-by: Naresh Kamboju Signed-off-by: Nathan Chancellor It was Link: for a bit but commit 44c31888098a ("checkpatch: allow Closes tags with links") changed it to Closes:. Looks odd to me but whatever the linter says I suppose. Thanks for the review! Cheers, Nathan > A few more tags > > Reported-by: Tom Rix > Link: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ > Reviewed-by: Nick Desaulniers > > > > --- > > drivers/gpu/drm/i915/display/intel_display.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > > b/drivers/gpu/drm/i915/display/intel_display.c > > index 0490c6412ab5..6d49e0ab3e85 100644 > > --- a/drivers/gpu/drm/i915/display/intel_display.c > > +++ b/drivers/gpu/drm/i915/display/intel_display.c > > @@ -6008,6 +6008,7 @@ static int intel_async_flip_check_hw(struct > > intel_atomic_state *state, struct in > > plane->base.base.id, > > plane->base.name); > > return -EINVAL; > > } > > + break; > > > > case I915_FORMAT_MOD_X_TILED: > > case I915_FORMAT_MOD_Y_TILED: > > > > --- > > base-commit: 9a2cb1b31c040e2f1b313e2f7921f0f5e6b66d82 > > change-id: > > 20230524-intel_async_flip_check_hw-implicit-fallthrough-c4c40b03802f > > > > Best regards, > > -- > > Nathan Chancellor > > > > > -- > Thanks, > ~Nick Desaulniers
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: > > Clang warns: > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated > fall-through between switch labels [-Werror,-Wimplicit-fallthrough] > case I915_FORMAT_MOD_X_TILED: > ^ > drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert 'break;' > to avoid fall-through > case I915_FORMAT_MOD_X_TILED: > ^ > break; > 1 error generated. > > Clang is a little more pedantic than GCC, which does not warn when > falling through to a case that is just break or return. Clang's version > is more in line with the kernel's own stance in deprecated.rst, which > states that all switch/case blocks must end in either break, > fallthrough, continue, goto, or return. Add the missing break to silence > the warning. > > Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") > Reported-by: kernel test robot > Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ > Reported-by: Naresh Kamboju > Closes: > https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ > Signed-off-by: Nathan Chancellor Thanks for the patch! I've never seen the closes tag before, that's new to me. Can you tell me more about it? A few more tags Reported-by: Tom Rix Link: https://lore.kernel.org/all/20230523125116.1669057-1-t...@redhat.com/ Reviewed-by: Nick Desaulniers > --- > drivers/gpu/drm/i915/display/intel_display.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > b/drivers/gpu/drm/i915/display/intel_display.c > index 0490c6412ab5..6d49e0ab3e85 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -6008,6 +6008,7 @@ static int intel_async_flip_check_hw(struct > intel_atomic_state *state, struct in > plane->base.base.id, > plane->base.name); > return -EINVAL; > } > + break; > > case I915_FORMAT_MOD_X_TILED: > case I915_FORMAT_MOD_Y_TILED: > > --- > base-commit: 9a2cb1b31c040e2f1b313e2f7921f0f5e6b66d82 > change-id: > 20230524-intel_async_flip_check_hw-implicit-fallthrough-c4c40b03802f > > Best regards, > -- > Nathan Chancellor > -- Thanks, ~Nick Desaulniers
Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
On Wed, 24 May 2023, Nathan Chancellor wrote: > Clang warns: > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated > fall-through between switch labels [-Werror,-Wimplicit-fallthrough] > case I915_FORMAT_MOD_X_TILED: > ^ > drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert 'break;' > to avoid fall-through > case I915_FORMAT_MOD_X_TILED: > ^ > break; > 1 error generated. > > Clang is a little more pedantic than GCC, which does not warn when > falling through to a case that is just break or return. Clang's version > is more in line with the kernel's own stance in deprecated.rst, which > states that all switch/case blocks must end in either break, > fallthrough, continue, goto, or return. Add the missing break to silence > the warning. > > Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") > Reported-by: kernel test robot > Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ > Reported-by: Naresh Kamboju > Closes: > https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ > Signed-off-by: Nathan Chancellor Yeah, I think this is the right quick fix. See also [1]. Reviewed-by: Jani Nikula [1] https://lore.kernel.org/r/874jo3kwl6@intel.com > --- > drivers/gpu/drm/i915/display/intel_display.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > b/drivers/gpu/drm/i915/display/intel_display.c > index 0490c6412ab5..6d49e0ab3e85 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -6008,6 +6008,7 @@ static int intel_async_flip_check_hw(struct > intel_atomic_state *state, struct in > plane->base.base.id, > plane->base.name); > return -EINVAL; > } > + break; > > case I915_FORMAT_MOD_X_TILED: > case I915_FORMAT_MOD_Y_TILED: > > --- > base-commit: 9a2cb1b31c040e2f1b313e2f7921f0f5e6b66d82 > change-id: > 20230524-intel_async_flip_check_hw-implicit-fallthrough-c4c40b03802f > > Best regards, -- Jani Nikula, Intel Open Source Graphics Center
[PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()
Clang warns: drivers/gpu/drm/i915/display/intel_display.c:6012:3: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough] case I915_FORMAT_MOD_X_TILED: ^ drivers/gpu/drm/i915/display/intel_display.c:6012:3: note: insert 'break;' to avoid fall-through case I915_FORMAT_MOD_X_TILED: ^ break; 1 error generated. Clang is a little more pedantic than GCC, which does not warn when falling through to a case that is just break or return. Clang's version is more in line with the kernel's own stance in deprecated.rst, which states that all switch/case blocks must end in either break, fallthrough, continue, goto, or return. Add the missing break to silence the warning. Fixes: 937859485aef ("drm/i915: Support Async Flip on Linear buffers") Reported-by: kernel test robot Closes: https://lore.kernel.org/202305241902.uvhtmoxa-...@intel.com/ Reported-by: Naresh Kamboju Closes: https://lore.kernel.org/CA+G9fYv68V3ewK0Qj-syQj7qX-hQr0H1MFL=qfnudoe_j2z...@mail.gmail.com/ Signed-off-by: Nathan Chancellor --- drivers/gpu/drm/i915/display/intel_display.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 0490c6412ab5..6d49e0ab3e85 100644 --- a/drivers/gpu/drm/i915/display/intel_display.c +++ b/drivers/gpu/drm/i915/display/intel_display.c @@ -6008,6 +6008,7 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in plane->base.base.id, plane->base.name); return -EINVAL; } + break; case I915_FORMAT_MOD_X_TILED: case I915_FORMAT_MOD_Y_TILED: --- base-commit: 9a2cb1b31c040e2f1b313e2f7921f0f5e6b66d82 change-id: 20230524-intel_async_flip_check_hw-implicit-fallthrough-c4c40b03802f Best regards, -- Nathan Chancellor